ivakegg commented on code in PR #3142:
URL: https://github.com/apache/accumulo/pull/3142#discussion_r1059473711


##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -726,6 +726,14 @@ public enum Property {
       "The number of threads on each tablet server available to retrieve"
           + " summary data, that is not currently in cache, from RFiles.",
       "2.0.0"),
+  TSERV_LAST_LOCATION_MODE("tserver.last.location.mode", "compact", 
PropertyType.LAST_LOCATION_MODE,
+      "Describes how the system will assign tablets initially by defining how 
the 'last' location is updated."
+          + " If 'compact' is the mode, then the system will assign tablets 
based on the data locality (e.g. the last compaction location)."
+          + " If 'assign' is the mode, then tablets will be initially assigned 
to the last place they were assigned."
+          + " If 'unload' is the mode, then tablets will be initially assigned 
to the last place they were unloaded from (i.e. requires a clean shutdown)."

Review Comment:
   ok, that is reasonable argument.  I was briefly thinking that one would want 
only one or the other to be able to see the discrepency between the last 
location and the new locations when scanning the metadata and using the 
"unload" mode.



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