ctubbsii commented on a change in pull request #868: Make replication services 
start when configured
URL: https://github.com/apache/accumulo/pull/868#discussion_r245128301
 
 

 ##########
 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:
   As I understand it, the current design doesn't make a single 
`replication.enabled` option as simple to do as it first appears. The elements 
in the master that are replication related appear to be part of the replica 
"send" code path.... the elements in the tserver that we're talking about 
appear related to the "AccumuloReplicaSystem", which is specifically a tserver 
feature for handling received replication data at the destination (the replica 
"receive" code path). It should be entirely possible to enable these features 
independently (to support basically any use case that isn't bi-directional 
replication between two Accumulo instances).

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