[PR] Added new Locked state [accumulo]

["ArbaazKhan1 (via GitHub)" <gi...@apache.org>]

ArbaazKhan1 opened a new pull request, #6210:
URL: https://github.com/apache/accumulo/pull/6210

   closes [issue #6151 ](https://github.com/apache/accumulo/issues/6151) 
   
   Added a new Enum LOCKED to `TableState` and `InitialTableState` and deprecated OFFLINE. I have so far only added in the new usage of LOCKED into the basic functionality, I still need to update all the test for the new state. But I figured to make the draft now so to get feedback on its direction. I have also left in the usage of OFFLINE state to not break anything just yet, but have had to add in a couple supresswarnings to ensure it builds. I did not do anything with upgrade behavior to support transition from OFFLINE to LOCKED and I also left OfflineIterator  alone for the most part, but I'm not entirely sure how I should handle them going forward. 
   
   The new LOCKED state blocks all operations, except delete, on a table. It does allow for table export (to replace OFFLINE) and dose NOT unhost tablets (stay in their current hosting state). The thought was that LOCKED is a "frozen" state and not a "shutdown" state. If you need tablets to be unhosted, you can use `TabletAvailability` settings in combination with `lock()`. The new state slow is as follows: 
   
   NEW -> (ONLINE | OFFLINE | LOCKED) <--> DELETING
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Added new Locked state [accumulo]

["DomGarguilo (via GitHub)" <gi...@apache.org>]

DomGarguilo commented on PR #6210:
URL: https://github.com/apache/accumulo/pull/6210#issuecomment-5095789367

   What is the status of this PR? I see it is in draft. Are there other changes to be made before this is ready for review?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org