keith-turner commented on issue #4246:
URL: https://github.com/apache/accumulo/issues/4246#issuecomment-2253389086
Looking at the history of this I found the following.
* This issue was opened about a test the verifies the UserFateStore will
retry when the random id it picked for a new fate id exists. It retries up to
5 times and then fails, the test that only exists for UserFateStore checks
this.
* The MetaFateStore does not retry and does not have a test for the case of
a collision.
* These changes referenced in this issue were made before #4388 which
changed the fate id type from 64bit to 128bit. This change dramatically
decreases the chance of collision to the point where maybe it should not be
expected.
With the change from 64bit to 128bit wondering if instead of implementing
and testing the behavior in both stores if we should simplify the expectation
to fail on any collision (instead of allowing 5) and test that in both stores.
--
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]
