It is out of date. Try 1 is now treated like low RDY within balance.

On Saturday, December 21, 2013, Matt Reiferson wrote:

> In lib/readerrdy.coffee:
>
> > +
> > +      @balance()
> > +      if @current_state_name is 'ZERO'
> > +        @goto 'MAX'
> > +      else if @current_state_name in ['TRY_ONE', 'MAX']
> > +        connectionRdy.bump()
> > +
> > +  removeConnection: (conn) ->
> > +    @connections.splice @connections.indexOf(conn), 1
> > +    @roundRobinConnections.remove conn
> > +
> > +    if @connections.length is 0
> > +      @goto 'ZERO'
> > +
> > +  bump: ->
> > +    # RDY 1 to each connection to test the waters.
>
> this comment seems out of date and, if it's not, the name of the function
> seems incorrect as it doesn't then have the same semantics as the other
> bump(), right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dudleycarr/nsqjs/pull/5/files#r8518083>
> .
>