dlmarion commented on code in PR #5749:
URL: https://github.com/apache/accumulo/pull/5749#discussion_r2324930202
##########
server/base/src/main/java/org/apache/accumulo/server/ServerContext.java:
##########
@@ -182,6 +191,10 @@ public SiteConfiguration getSiteConfiguration() {
@Override
public AccumuloConfiguration getConfiguration() {
+ return serverConfFactory.get().getResourceGroupConfiguration();
Review Comment:
My original implementation did what you are describing. @keith-turner
suggested that we should separate them and provide access to each, discussion
at https://github.com/apache/accumulo/pull/5749#discussion_r2231627609. You can
see how I had the code originally looking at the ServerConfigurationFactory
class in
https://github.com/apache/accumulo/commit/febdd079e8235d9f83058c3db09b349e15779517.
--
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]
