DomGarguilo commented on PR #4242: URL: https://github.com/apache/accumulo/pull/4242#issuecomment-1946795277
In d0e44d2 I added test coverage to all 3 of the methods that were changed in AccumuloStore. The test makes sure only the expected statuses allow the operation to succeed. For some reason, `pop()` is succeeding when the status is `NEW` even though we are requiring that only `FAILED_IN_PROGRESS` is present. I am not sure where the bug is. It could be in my test logic or elsewhere. Still looking into that but thought I would push my changes so far in case someone else spots whats going on and so others can see the test. -- 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]
