accumulo-testing icon indicating copy to clipboard operation
accumulo-testing copied to clipboard

Create a tool for verifying WAL assumptions

Open billierinaldi opened this issue 5 years ago • 3 comments

Suggestions for additional verifications that could be performed or other improvements are welcome!

billierinaldi avatar Mar 26 '20 23:03 billierinaldi

Looking at the test its use of internal accumulo config and volume manager are likely susceptible to breakage between accumulo versions as those areas of the code are changing as we pay down tech debt. Ideally the test could only ref the pluggable log closer which is more stable. However not sure if that is possible, will look into it more next week.

keith-turner avatar Mar 27 '20 22:03 keith-turner

Thanks for taking a look, @ctubbsii and @keith-turner. That's a good point. One option would be for this to be a standalone tool in Accumulo itself, like the troubleshooting tools. Regarding technical debt, working on this has made me think it would be a lot easier if we had an interface that had all the WAL file system operations in the same place. That is, if the LogCloser also opened the files in addition to closing them (closing meaning lease recovery in this case). That might allow us to wrap a file system that doesn't have leases with our own custom lease handling, if we wanted.

billierinaldi avatar Mar 28 '20 02:03 billierinaldi

This is quite old. Has anybody looked at this recently and interested in picking it back up?

ctubbsii avatar Jun 28 '23 18:06 ctubbsii