Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-19319

Fix bug in synchronizing over ProcedureEvent

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-beta-1, 2.0.0
    • Component/s: None
    • Labels:
      None

      Description

      Following synchronizes over local variable rather than the original ProcedureEvent object. Clearly a bug since this code block won't follow exclusion with many of the synchronized methods in ProcedureEvent class.

       @Override
        public void wakeEvents(final int count, final ProcedureEvent... events) {
          final boolean traceEnabled = LOG.isTraceEnabled();
          schedLock();
          try {
            int waitingCount = 0;
            for (int i = 0; i < count; ++i) {
              final ProcedureEvent event = events[i];
              synchronized (event) {
                if (!event.isReady()) {
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                appy Appy
                Reporter:
                appy Appy
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: