dlmarion commented on code in PR #5692:
URL: https://github.com/apache/accumulo/pull/5692#discussion_r2178192748
##########
minicluster/src/main/java/org/apache/accumulo/miniclusterImpl/MiniAccumuloClusterControl.java:
##########
@@ -213,16 +215,20 @@ public synchronized void start(ServerType server,
Map<String,String> configOverr
Map<String,Integer> compactorGroups =
cluster.getConfig().getClusterServerConfiguration().getCompactorConfiguration();
for (Entry<String,Integer> e : compactorGroups.entrySet()) {
+ final String rg = e.getKey();
Review Comment:
The problem is that the call to set the server class,
`MiniAccumuloConfiImpl.setServerClass(ServerType, Class)`, does not take the
resource group as a parameter. When this method is used it applies to all
groups, including DEFAULT. I think we either have to overwrite the default as
is done here, or modify the `setServerClass` method to accept a resource group
parameter.
--
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]
