Couchbase Java Client

race condition during startup

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.0, 1.0.1, 1.1dp, 1.1.0
  • Fix Version/s: 1.1.3
  • Component/s: library
  • Security Level: Public
  • Labels:
    None
  • Flagged:
    Release Note

Description

During startup, if there is no authentication (and thus no authentication latch) we can reply with errors before we get the configuration back and settled in with the node locator. This should be more reliable.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Matt Ingenthron added a comment -
See http://www.couchbase.com/forums/thread/fast-computer-race-condition-java-client
Show
Matt Ingenthron added a comment - See http://www.couchbase.com/forums/thread/fast-computer-race-condition-java-client
Hide
Permalink
Matt Ingenthron added a comment -
I think part of the solution on this is to poll the configuration until it transitions from warmup to healthy.
Show
Matt Ingenthron added a comment - I think part of the solution on this is to poll the configuration until it transitions from warmup to healthy.
Raghavan Srinivas made changes -
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 1.1beta [ 10370 ]
Resolution Incomplete [ 4 ]
Raghavan Srinivas made changes -
Resolution Incomplete [ 4 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Michael Nitschinger made changes -
Assignee Raghavan Srinivas [ rags ] Michael Nitschinger [ daschl ]
Hide
Permalink
Matt Ingenthron added a comment -
The idea is that there is a section of the code that walks the URIs, finds the bucket, then after finding it sets up the stream for the configuration. When it first finds the bucket, if it's in a "warmup" state, (easy to simulate by restarting a server) it will show that it is and will not have a vbucket map. At that point, we should loop without setting up the stream *or* we should set up the stream and let anything handling reconfigure handle the transition from warmup to warmed up.
Show
Matt Ingenthron added a comment - The idea is that there is a section of the code that walks the URIs, finds the bucket, then after finding it sets up the stream for the configuration. When it first finds the bucket, if it's in a "warmup" state, (easy to simulate by restarting a server) it will show that it is and will not have a vbucket map. At that point, we should loop without setting up the stream *or* we should set up the stream and let anything handling reconfigure handle the transition from warmup to warmed up.
Michael Nitschinger made changes -
Priority Major [ 3 ] Blocker [ 1 ]
Hide
Permalink
Michael Nitschinger added a comment -
http://review.couchbase.com/#/c/22933/1
Show
Michael Nitschinger added a comment - http://review.couchbase.com/#/c/22933/1
Michael Nitschinger made changes -
Status Reopened [ 4 ] In Progress [ 3 ]
Hide
Permalink
Matt Ingenthron added a comment -
Still working out the flow here. Based on our current understanding, this can be deferred to 1.1.0 or even post since it's an enhancement for reliable operation in a secondary or tertiary circumstance. Should be release noted though.
Show
Matt Ingenthron added a comment - Still working out the flow here. Based on our current understanding, this can be deferred to 1.1.0 or even post since it's an enhancement for reliable operation in a secondary or tertiary circumstance. Should be release noted though.
Matt Ingenthron made changes -
Fix Version/s 1.1.0 [ 10274 ]
Fix Version/s 1.1-beta [ 10370 ]
Flagged [Release Note]
Matt Ingenthron made changes -
Fix Version/s 1.1.1 [ 10430 ]
Fix Version/s 1.1.0 [ 10274 ]
Affects Version/s 1.1.0 [ 10274 ]
Michael Nitschinger made changes -
Fix Version/s 1.1.2 [ 10480 ]
Fix Version/s 1.1.1 [ 10430 ]
Michael Nitschinger made changes -
Fix Version/s 1.1.3 [ 10496 ]
Fix Version/s 1.1.2 [ 10480 ]
Hide
Permalink
Matt Ingenthron added a comment -
Determined that the proposed approach is a good change, but better change is needed. That's tracked under JCBC-255.
Show
Matt Ingenthron added a comment - Determined that the proposed approach is a good change, but better change is needed. That's tracked under JCBC-255 .
Michael Nitschinger made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Michael Nitschinger
    Reporter:
    Matt Ingenthron
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: