search

apache / accumulo-proxy

Apache Accumulo Proxy
https://accumulo.apache.org
Apache License 2.0
9 stars 19 forks source link

Fix CompactionSelecor test #72

Closed DomGarguilo closed 1 year ago

DomGarguilo commented 1 year ago

Fixes #71

DomGarguilo commented 1 year ago

Currently, the changes in this PR make it so that the condition that was trying to be assessed, that the compation selector is working, is actually being assessed. This is causing the test to fail because something is wrong and the file count is not changing as expected.

The message that prints out when the tests fails is: Expected to find 13 files but found 26

DomGarguilo commented 1 year ago

Marking this as ready for review. I changed the Selector used in the test to the TooManyDeletesSelector. I modified the test to use this selector and based it off of CompactionExecutorIT.testTooManyDeletes. The point of the test is to make sure a selector can be specified from the proxy client and will take affect during compactions.