[GitHub] milleruntime commented on a change in pull request #868: Make tserver start repl services when configured

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org


With regards,
Apache Git Services