accumulo icon indicating copy to clipboard operation
accumulo copied to clipboard

Fixes 'admin fate --print/--summary' no node exception bug

Open kevinrr888 opened this issue 9 months ago • 1 comments

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

kevinrr888 avatar May 13 '24 19:05 kevinrr888

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.

kevinrr888 avatar May 13 '24 19:05 kevinrr888