[I] FateId's tid should be replaced with 128bit tid [accumulo]

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

kevinrr888 opened a new issue, #4277:
URL: https://github.com/apache/accumulo/issues/4277

   **Is your feature request related to a problem? Please describe.**
   Prior to [issue#4044](https://github.com/apache/accumulo/issues/4044) and the related PRs https://github.com/apache/accumulo/pull/4191, https://github.com/apache/accumulo/pull/4228, https://github.com/apache/accumulo/pull/4247, and https://github.com/apache/accumulo/pull/4258 for elasticity, fate transaction ids were a long. Now, fate transaction ids are an object (FateId) consisting of the long and the FateInstanceType. Now that the long id is centralized, it would be nice to change this from a 64 bit long to a 128 bit UUID. Further, FateIds should be sortable by UUID. The UUID should include the creation time, and first be sorted by the creation time. For example FateId1 < FateId2 means FateId1 was created before FateId2. This can be achieved with UUIDv7.
   
   The benefits of this are:
   - Collisions will be much less likely
   - FateIds will be sortable by timestamp allowing for easier determination of the order in which Fate operations occur
   
   **Additional context**
   Further UUIDv7 context: https://github.com/apache/accumulo/issues/4044#issuecomment-1932572797
   


-- 
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.apache.org

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

Re: [I] FateId's tid should be replaced with 128bit tid [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner closed issue #4277: FateId's tid should be replaced with 128bit tid
URL: https://github.com/apache/accumulo/issues/4277


-- 
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: [I] FateId's tid should be replaced with 128bit tid [accumulo]

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

kevinrr888 commented on issue #4277:
URL: https://github.com/apache/accumulo/issues/4277#issuecomment-1954841486

   I would like to work on this


-- 
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