You're right
On Apr 10, 2014 12:05 PM, "Clément Garnier" <notifications@github.com>
wrote:

> In
> nrv-zookeeper/src/it/scala/com/wajam/nrv/zookeeper/consistency/TestZookeeperConsistencyPersistence.scala:
>
> >      // Should be rate limited
> >      consistency.updateReplicationLagSeconds(token, slave, firstUpdate)
> >
> > +    // Wait a little while and then ensure the lag haven't been persisted
> > +    Thread.sleep(500L)
> >      eventually {
> >        checkCachedAndPersistedLagValues(service, token, slave, initialLag)
> >      }
> >
> >      // Advance time to get past threshold
>
> I think this comment is wrong. We advance time to get past *spacing*.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/wajam/nrv/pull/298/files#r11492400>
> .
>