Allow setting some compactor properties in zookeeper

[keith-turner]

Is your feature request related to a problem? Please describe.

When testing Accumulo 2.1.4rc0 attempted to set compactor.wait.time.job.max in the shell and it failed.

Describe the solution you'd like

Looking at the compactor code it seems like if this property were set in zookeerper that it would naturally be picked up by compactors. Wondering if there is a subset of compactor props that could be allowed to be set in zookeeper.

[dlmarion]

This is related to #5695 and is not working because the compactor prefix (or specific compactor properties) are not in Property.isValidZooPropertyKey. I think this should be fixed for the 2.1.4 release. Marking as blocker.

[keith-turner]

@dlmarion would #5749 allow closing this? or do you think these props should be settable at the system level?

[dlmarion]

#5749 doesn't fix it for 2.1.4. To fix this for 2.1.4 we would have to modify Property.java's valid ZooKeeper property list. IIRC there was a decision not do that because setting the property in ZK would override any property set in accumulo.properties on specific nodes for different groups of compactors. I think #5749 fixes the issue for main though.

[ddanielr]

Yep @dlmarion is right, I had run into the global override issue when testing with different compactors on different queues and not being able to separate property changes.