keith-turner commented on code in PR #3482:
URL: https://github.com/apache/accumulo/pull/3482#discussion_r1227073278
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -808,6 +814,11 @@ public enum Property {
+ " global setting to take effect. However, you must use the API or
the shell"
+ " to change properties in zookeeper that are set on a table.",
"1.3.5"),
+ TABLE_ASSIGNMENT_GROUP("table.assignment.group",
ServiceDescriptor.DEFAULT_GROUP_NAME,
+ PropertyType.STRING,
+ "Tablets for this table will be assigned to TabletServers that have a
corresponding"
Review Comment:
This statement may not be true, it depends on the behavior of the balancer?
Unless the balancer is limited to only assigning to tserver within the group?
We could limit balancing choices to a group, so a balancer could only chose
from tservers within this group. That may require some changes to the SPI, not
sure. Or this could be provided as information to the balancer, but it does
not have to honor it. If the balancer does not have to honor it then would
need to change the description.
--
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]
