accumulo
accumulo copied to clipboard
Fixes 'admin fate --print/--summary' no node exception bug
closes https://github.com/apache/accumulo/issues/4462
- Fixes bug with admin fate print and summary commands failing due to ZK NoNodeException when a transaction completes between getting the list of transactions and probing for info on that transaction.
- Added test testFatePrintAndSummaryCommandsWithInProgressTxns() to FateSummaryIT to test changes
- Small changes to testFateSummaryCommandWithSlowCompaction() in FateSummaryIT
- Stop the COMPACTOR to avoid seeing unnexpected transactions in the summary
- Wait for the flush to complete
- delete the table at the end of the test
This issue reminded me of the FateOpsCommandsIT written for elasticity (tests all the admin fate commands). This test was an expansion of FateSummaryIT and removed FateSummaryIT in elasticity. Made me think that it might be good to have the FateOpsCommandsIT written for 2.1/3.1. If desired, I could rewrite the test for 2.1/3.1 in a separate PR.