accumulo icon indicating copy to clipboard operation
accumulo copied to clipboard

Functionality to run ScanConsistencyIT standalone

Open kevinrr888 opened this issue 1 year ago • 6 comments

Closes #3646. Refactored ScanConsistencyIT to be able to be run standalone. Can now be run as a test or from main.

kevinrr888 avatar Dec 11 '23 16:12 kevinrr888

Added requested changes:

  • Removed "@ throws Exception" from javadoc. Hopefully fixes build failure from "Checkstyle violations".
  • Shutdown the executor in a finally block
  • Table is now deleted when test is done
  • Check args length using Preconditions.checkArgument()
  • Moved client into try-with-resources block
  • Messages are now logged in debug mode if in testing context, info mode otherwise

kevinrr888 avatar Dec 11 '23 18:12 kevinrr888

The same can be achieved using the instructions here. Just curious, why would we need this?

dlmarion avatar Dec 11 '23 18:12 dlmarion

The same can be achieved using the instructions here. Just curious, why would we need this?

We may not, I had forgotten about that, wonder if the following work

  1. Setup accumulo in uno
  2. copy test jar to uno lib dir, this is needed because the test has server side code it runs
  3. restart accumulo
  4. run test using mvn passing in uno accumulo props file

If it does work may still want some of the changes in this PR like deleting the table.

keith-turner avatar Dec 11 '23 18:12 keith-turner

When running this test manually against an existing instance, would like to be able to adjust this sleep. Would like to be able to run this test against an uno instance for hours.

keith-turner avatar Dec 11 '23 18:12 keith-turner

See this for running an IT against an existing instance. Caveat: I've never tried it.

dlmarion avatar Dec 11 '23 19:12 dlmarion

More requested changes:

  • Table deletion is no longer in finally block
  • Use log.info
  • Main now accepts sleep time

Since we should be taking in a sleep time parameter, not sure how this would be done using mvn.

Let me know if the approach using main is sufficient.

kevinrr888 avatar Dec 11 '23 19:12 kevinrr888