Mailing-List: contact dev-help@couchdb.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@couchdb.apache.org
Received-SPF: pass (athena.apache.org: domain of paul.joseph.davis@gmail.com
 designates 209.85.212.52 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <A588B71B-78B2-4736-81EF-D3C4D3E94894@apache.org>
References: <A588B71B-78B2-4736-81EF-D3C4D3E94894@apache.org>
From: Paul Davis <paul.joseph.davis@gmail.com>
Date: Wed, 29 Jun 2011 12:07:23 -0400
Message-ID: <BANLkTin81_v_87TjjXYozUbizwnT43n4wg@mail.gmail.com>
Subject: Re: waitForRestart() double request()
To: dev@couchdb.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Wed, Jun 29, 2011 at 9:45 AM, Jan Lehnardt <jan@apache.org> wrote:
> Heya,
>
> I may be missing something, but our weitForRestart() function seems to ha=
ve an accidental line duplication:
>
> function waitForRestart() {
> =A0var waiting =3D true;
> =A0while (waiting) {
> =A0 =A0try {
> =A0 =A0 =A0CouchDB.request("GET", "/");
> =A0 =A0 =A0CouchDB.request("GET", "/");
> =A0 =A0 =A0waiting =3D false;
> =A0 =A0} catch(e) {
> =A0 =A0 =A0// the request will fail until restart completes
> =A0 =A0}
> =A0}
> };
>
> If this is intentional, I'd like to know what the intent here is :)
>
> Cheers
> Jan
> --
>
>

Someone's sick idea of a sleep statement?

FWIW, it's been doubled up since it was committed a long time ago:

https://github.com/apache/couchdb/commit/ed7e7c686fae7f1d2e3f149c2f2ed8854c=
4f95c8#L0R152