milleruntime commented on a change in pull request #868: Make tserver start 
repl services when configured
URL: https://github.com/apache/accumulo/pull/868#discussion_r245072223
 
 

 ##########
 File path: core/src/main/java/org/apache/accumulo/core/conf/Property.java
 ##########
 @@ -456,6 +456,8 @@
       "The amount of time an assignment can run before the server will print a"
           + " warning along with the current stack trace. Meant to help debug 
stuck"
           + " assignments"),
+  TSERV_REPLICATION_ENABLED("tserver.replication.services.enabled", "false", 
PropertyType.BOOLEAN,
 
 Review comment:
   Look at the services that the Master starts up for replication, it is a bit 
trickier.  And I am not sure how much we save since there should only be one 
Master running versus X TabletServers.  I think making it a tserver property 
makes the most sense here.  There may also be a use case where you want certain 
tservers to participate in replication and disabled on others.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to