couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Davis <paul.joseph.da...@gmail.com>
Subject Re: waitForRestart() double request()
Date Wed, 29 Jun 2011 16:07:23 GMT
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 have an accidental
line duplication:
>
> function waitForRestart() {
>  var waiting = true;
>  while (waiting) {
>    try {
>      CouchDB.request("GET", "/");
>      CouchDB.request("GET", "/");
>      waiting = false;
>    } catch(e) {
>      // the request will fail until restart completes
>    }
>  }
> };
>
> 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/ed7e7c686fae7f1d2e3f149c2f2ed8854c4f95c8#L0R152

Mime
View raw message