Uploaded image for project: 'Spymemcached Java Client'
  1. Spymemcached Java Client
  2. SPY-88

OperationState as tracked in BaseOperationImpl should be volatile

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: .next
    • Component/s: None
    • Security Level: Public
    • Labels:
      None

      Description

      Since the OperationState is accessed by multiple threads, there can be a coherency issue if the application code is trying to reference the OperationState after an OperationFuture has completed, but a previous value of the OperationState is not seen on the thread of the application blocking on that OperationFuture.

      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        Hide
        ingenthr Matt Ingenthron added a comment -

        Found that the fix here was introducing test failures in multigets. The problem isn't with the fix posted in http://review.couchbase.org/#change,16255 but rather that it now correctly asserts.

        Show
        ingenthr Matt Ingenthron added a comment - Found that the fix here was introducing test failures in multigets. The problem isn't with the fix posted in http://review.couchbase.org/#change,16255 but rather that it now correctly asserts.
        Hide
        daschl Michael Nitschinger added a comment -

        This is also quite old, does that mean its already fixed Matt?

        Show
        daschl Michael Nitschinger added a comment - This is also quite old, does that mean its already fixed Matt?

          People

          • Assignee:
            ingenthr Matt Ingenthron
            Reporter:
            ingenthr Matt Ingenthron
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes