dlmarion commented on code in PR #5899:
URL: https://github.com/apache/accumulo/pull/5899#discussion_r2367990829


##########
test/src/main/java/org/apache/accumulo/test/conf/ResourceGroupConfigIT.java:
##########
@@ -313,118 +310,4 @@ public void testPermissions() throws Exception {
       test_user_client.resourceGroupOperations().create(rgid);
     }
   }
-
-  @Test
-  public void testMultipleConfigurations() throws Exception {

Review Comment:
   > Would still be useful to have a test that creates multiple RGs and set 
config on them.
   
   I re-added the testMultipleConfigurations test in 1d54fbe that used the now 
removed ResourceGroupOperations.getConfiguration. The new test creates a Thrift 
client and calls the thrift operation directly.



-- 
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]

Reply via email to