Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-4574

Modify TableOperations online to check for table online before executing fate operation.

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7.2
    • Fix Version/s: 1.7.3, 1.8.1, 2.0.0
    • Component/s: None
    • Labels:
      None

      Description

      The table operations online operation executes as a fate operation. If a transaction lock for the table is currently held (say by table compactions) the online operation will block.

      This modification essentially changes the behavior of the online operation to a NOOP if the table is currently online. If the current table state is online, then the operation returns immediately without queuing a fate operation to set the online table to online. This eliminates the blocking behavior of the operation if the table is already online.

        Issue Links

          Activity

          Hide
          mjwall Michael Wall added a comment -

          Leaving this one in 1.8.1 after talking with Ed Coleman

          Show
          mjwall Michael Wall added a comment - Leaving this one in 1.8.1 after talking with Ed Coleman
          Hide
          etcoleman Ed Coleman added a comment -

          Should be able to wrap this up this weekend - I need to merge Keith Turner fate changes so I can use the new FATE class.

          Show
          etcoleman Ed Coleman added a comment - Should be able to wrap this up this weekend - I need to merge Keith Turner fate changes so I can use the new FATE class.
          Hide
          kturner Keith Turner added a comment -

          Ed Coleman I refactored the new FATE code some more in PR #209 so I could call it from other ITs. Hopefully, these changes should make it easier to use in your test.

          Show
          kturner Keith Turner added a comment - Ed Coleman I refactored the new FATE code some more in PR #209 so I could call it from other ITs. Hopefully, these changes should make it easier to use in your test.
          Hide
          etcoleman Ed Coleman added a comment -

          I believe pr:204 is complete - pending additional review comments.

          I updated the pull request 204 to include the changes in pull request 209 - it should merge cleanly - but we probably should merge 209 first so the history is cleaner.

          Show
          etcoleman Ed Coleman added a comment - I believe pr:204 is complete - pending additional review comments. I updated the pull request 204 to include the changes in pull request 209 - it should merge cleanly - but we probably should merge 209 first so the history is cleaner.
          Hide
          mjwall Michael Wall added a comment -
          Show
          mjwall Michael Wall added a comment - Ed Coleman https://github.com/apache/accumulo/pull/209 has been merged.
          Hide
          etcoleman Ed Coleman added a comment -

          Keith Turner - with your merge of this into 1.7.3 and 1.8.1 can this be closed (what about 2.0?)

          Show
          etcoleman Ed Coleman added a comment - Keith Turner - with your merge of this into 1.7.3 and 1.8.1 can this be closed (what about 2.0?)
          Hide
          kturner Keith Turner added a comment -

          I merged this to 2.0.0 also

          Show
          kturner Keith Turner added a comment - I merged this to 2.0.0 also

            People

            • Assignee:
              etcoleman Ed Coleman
              Reporter:
              etcoleman Ed Coleman
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 24h
                24h
                Remaining:
                Time Spent - 9h 50m Remaining Estimate - 14h 10m
                14h 10m
                Logged:
                Time Spent - 9h 50m Remaining Estimate - 14h 10m
                9h 50m

                  Development