search

apache / accumulo-proxy

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

Update compaction techniques #47

Closed DomGarguilo closed 1 year ago

DomGarguilo commented 1 year ago

After updating accumulo-proxy to use accumulo 2.1, the usage of compaction also has to be updated. Currently, compaction strategies are being used which is marked for removal and should be updated.

DomGarguilo commented 1 year ago

I am working on this ticket now but am running into an issue. I'm not sure how to package up a custom CompactionSelector into a jar to be used in the proxy. Currently, there is a jar file containing a compaction strategy (which is deprecated and is what we are trying to replace here). In one of the tests, the jar is read and then the class name is sent along with the compact command. I assume we need to do something similar with a CompactionSelector and CompactionConfigurer to make sure things work with the new compaction methods.

https://github.com/apache/accumulo-proxy/blob/d7988f812c50f78bafde4e94ab13f6aaaca92041/src/test/java/org/apache/accumulo/proxy/its/SimpleProxyBase.java#L2707-L2714

https://github.com/apache/accumulo-proxy/blob/d7988f812c50f78bafde4e94ab13f6aaaca92041/src/test/java/org/apache/accumulo/proxy/its/SimpleProxyBase.java#L2736

If anyone could point me in the right direction for creating these jar files that would be helpful.