You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2025/01/31 23:07:21 UTC
[PR] Fefactor Fate lock code to use concrete types [accumulo]
cshannon opened a new pull request, #5298:
URL: https://github.com/apache/accumulo/pull/5298
Instead of passing around byte arrays for DistributedReadWriteLock and related classes we can use LockType and FateId to make the code easier to work with and to undertand as those values are serialized as part of the lock. A new FateLockEntry type has been created which is passed around instead and that is serialized/deserialized into the correct byte array format when reading/writing to zookeeeper.
This closes #5264
Note: I didn't add much for new tests because all the existing Fate tests should provide coverage for the changes (at least i think so)
--
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] Refactor Fate lock code to use concrete types [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #5298:
URL: https://github.com/apache/accumulo/pull/5298#issuecomment-2629013166
Merged this so I can merge these changes into #5262
--
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] Refactor Fate lock code to use concrete types [accumulo]
Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner merged PR #5298:
URL: https://github.com/apache/accumulo/pull/5298
--
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