accumulo
accumulo copied to clipboard
apache
Refactor scan consistency IT so that it can also be run standalone
Running the scan consistency test added in #3639 standalone for longer periods would be useful to stress test accumulo. The test could be refactored to support this by adding a main method that takes an accumulo properties file and runs the test. This would allow running against a standalone instance for long periods of time where its easier to inspect different parts of accumulo.
As a long running test, would this be more fitting to move to AccumuloTesting? Or do you think it should stay here?
As a long running test, would this be more fitting to move to AccumuloTesting? Or do you think it should stay here?
I think its nice to have as an IT that stresses a lot of functionality together that's always expected to work (that is why I added it as a sunny day test). So I think its nice to leave it here.
As a long running test, would this be more fitting to move to AccumuloTesting? Or do you think it should stay here?
I think its nice to have as an IT that stresses a lot of functionality together that's always expected to work (that is why I added it as a sunny day test). So I think its nice to leave it here.
I think the point of the sunny day test suite is that they test base functionality, and is something that gives us a quick "yes" or "no" on mergability for any particular patch or contribution. If one of those tests fails, it means Accumulo stopped doing basic things, like starting up, or communicating between server types. These are tests we expect a contributor to have not broken, and should be fast enough to run the whole batch of them on a modest developer hardware, like a contributor's personal laptop. I don't think we should be putting stress tests in there, or edge case coverage, or anything that is nondeterministic or flaky. Of those, I'd consider this test "edge case coverage", and don't think it warrants making sunny take longer to execute. But, I think it's fine to leave here, rather than move it to the testing repo.
