ctubbsii commented on PR #5708: URL: https://github.com/apache/accumulo/pull/5708#issuecomment-3050400250
> When testing 2.1.4-rc0 I tried to set `compactor.wait.time.job.max` in the shell and that was not allowed. Looking at the code, its constantly reading that prop. So it seems like it would have been ok to set that prop in the shell. Mentioning because it seems related to the goals of this change, also I need to open an issue about that. I remember discussing this at some point about compactor configs being explicitly excluded because we have no way to configure resource groups for compactors independently in ZooKeeper. The same is technically true for other server types in resource groups, but other server types don't depend as heavily on resource groups as compactors, and also predate resource groups, so we're kinda stuck with the history of those. I think we need the ability to configure resource groups independently in ZK without clobbering per-resource group configs with a global setting, which would help with this. If we had that, we could safely allow compactor configs in ZK too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
