EdColeman

How does this make future changes harder? - It moves the instantiation of instances to one place. - Instead of multiple instances running in scheduled executors there is one. -...

The rename was a suggestion by @milleruntime ``` ...I think the name of this class is confusing. Some reference to Java or the JVM in the name would greatly help...

Maybe I am reading you code wrong, but I don't see where your code is functionally equivalent to the current code that you posted here. The current code seems to...

Looking at the method in 'Key' the method that performs the equality check on a byte array private static boolean isEqual(byte[] a1, byte[] a2) is already optimized for Accumulo row...

I am confused with the in the context of #2811 are any changes needed? If not, changing the full depth equals to deleted, timestamp, row, col fam, col qual and...

Besides the TabletFileTest, It looks like any change causes additional Test failures. Either using a "normal" equals that includes the parent equals or just using the stored path for the...

Should be handled with Update TabletFile methods #2854. The issue as found by @ctubbsii: ` Bulk v2 LoadFiles reads creates a Map and puts StoredTabletFiles in it. Then, it calls...

I am not sure if it this problem, or a similar issue with getting the instance id from hdfs but running the full tests with github actions, sometimes there are...

Is this a valid utility to expose to general use? It seems to address a narrow use-case that might have significant performance issues if arbitrarily run on a large table....

Extending KeywordExecutable will display the command as help - when you execute accumulo with no arguments, it shows the available commands - that would raise the visibility of this command...