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

        ingenthr Matt Ingenthron created issue -
        ingenthr Matt Ingenthron made changes -
        Field Original Value New Value
        Assignee Raghavan Srinivas [ rags ] Matt Ingenthron [ ingenthr ]
        ingenthr Matt Ingenthron made changes -
        Project Couchbase Java Client [ 10080 ] Spymemcached Java Client [ 10047 ]
        Key JCBC-52 SPY-88
        mikew Mike Wiederhold made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        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.
        ingenthr Matt Ingenthron made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        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?
        daschl Michael Nitschinger made changes -
        Fix Version/s .next [ 10205 ]
        ingenthr Matt Ingenthron made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Done [ 6 ]

          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