keith-turner commented on code in PR #3496:
URL: https://github.com/apache/accumulo/pull/3496#discussion_r1244184039
##########
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:
If we scope the config to plugin, it could allow TableLoadBalancer to have
more complex configuration that is tightly coupled to it. For example
TableLoadBalancer could support the property `table.custom.assignment.group` or
`table.custom.assignment.rangedGroups` where rangedGroups allows a user to set
a json value that specifies something like the folllowing.
* For tablets in range `(<row1>,<row2>]` use tablet server group X
* For tablets in range `(<row3>,<row4>]` use tablet server group Y
* For all other tablet use tablet server group Z
So one property or the other could be set, if both are set it throws an
error.
--
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]
