keith-turner commented on code in PR #3496:
URL: https://github.com/apache/accumulo/pull/3496#discussion_r1244221540


##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -1069,6 +1075,11 @@ public enum Property {
           + "also consider configuring the `" + 
NoDeleteConstraint.class.getName() + "` "
           + "constraint.",
       "2.0.0"),
+  TABLE_ASSIGNMENT_GROUP("table.assignment.group", 
Constants.DEFAULT_RESOURCE_GROUP_NAME,

Review Comment:
   > So, it wasn't just the warning in TGW.getAssignmentsFromBalancer. There is 
also this:
   >
   > 
https://github.com/apache/accumulo/pull/3496/files#diff-dd97054ac43737177293a8f3a9231f05f0e4bc5eb0a3b042cad3fb8ce31b6caaR303-R325
   
   For that case maybe we need to add a new method to the balancer so we can 
ask `hey balancer are you happy with the group this tablet is currently 
assigned to?`
   



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