kevinrr888 commented on issue #4044: URL: https://github.com/apache/accumulo/issues/4044#issuecomment-1921770882
The end goal is to have the stronger type FateId replace the current representation of a transaction id (which is just a long). This was brought about from the addition of the AccumuloStore class - there are now two fate instance types associated with a transaction - META (for ZooStore) or USER (for AccumuloStore). FateId is a new class which includes the FateInstanceType and the transaction id. TODO list for this issue: - [x] Create new class FateId to replace FateTxId - [x] Change the stores to use FateId. Start with ReadOnlyFateStores. Resolve issues stemming from these changes. - [x] Change Fate to use FateId. Resolve issues stemming from these changes. (the above have been completed and merged in by [PR#4191](https://github.com/apache/accumulo/pull/4191)) - [ ] Change Repo to use FateId. Resolve issues stemming from these changes. - [ ] AdminUtil and Admin need to use FateId. Deferred for now. Need [#4168](https://github.com/apache/accumulo/issues/4168) completed first. - [ ] Delete FateTxId when all uses have been replaced with FateId -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
