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