ivakegg commented on code in PR #3139:
URL: https://github.com/apache/accumulo/pull/3139#discussion_r1058375491
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -851,6 +851,14 @@ public enum Property {
+ " the tablet server this duration to revive before reassigning its
tablets"
+ " to other tablet servers."),
+ TABLE_LOCATION_MODE("table.location.mode", "locality",
PropertyType.LAST_LOCATION_MODE,
Review Comment:
I thought a per table propery might be useful, but perhaps not. I can go
either way on this one.
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -851,6 +851,14 @@ public enum Property {
+ " the tablet server this duration to revive before reassigning its
tablets"
+ " to other tablet servers."),
+ TABLE_LOCATION_MODE("table.location.mode", "locality",
PropertyType.LAST_LOCATION_MODE,
+ "Describes how the system will assign tablets initially."
+ + " If 'locality' is the mode, then the system will assign tablets
based on the data locality (e.g. the last major compaction location)."
+ + " If 'assignment' is the mode, then tablets will be initially
assigned to the last place they were assigned which could be"
+ + " different then where they were last compacted given balancing
activities."
+ + " Also note that master.startup.tserver properties might need to
be set as well to ensure"
Review Comment:
of course
--
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]
