cshannon commented on PR #4049: URL: https://github.com/apache/accumulo/pull/4049#issuecomment-1859265131
@keith-turner - I did some refactoring and now FateIT is abstract and there are test classes to run the 4 tests for both Zookeeper and Accumulo stores. One thing I noticed was one of the tests seems to have a race condition that I think needs fixing as I saw an occasional failure with the accumuo store version and i added a comment [here](https://github.com/apache/accumulo/pull/4049/files#diff-2d3ed905a65d84cd0a4ad053128a6506bd88de4abcdb8b8a452600b73815f1dfR134-R143). This should be close to being able to go out of draft status if we are ok doing some follow ons for things like conditional mutations. -- 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]
