ddanielr commented on PR #5966: URL: https://github.com/apache/accumulo/pull/5966#issuecomment-3554778081
Moving the range property to the service definition inside `cluster.yaml` is a solid improvement as I've been bitten before by forgetting to also increase port ranges after raising the `compactors_per_host` value in `cluster.yaml`. If adding this property to `cluster.yaml` isn't desired then we might be able to use the [include](https://github.com/apache/accumulo/blob/main/assemble/conf/accumulo.properties#L37) option to have service specific property files IF property file contents collapse with a "last one wins" override behavior. If each file has to contain distinct property definitions then the existing `accumulo.properties` file would need to be separated into multiple files like `general.properties`, `tserver.properties`,`manager.properties`, etc. -- 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]
