dlmarion commented on issue #4232:
URL: https://github.com/apache/accumulo/issues/4232#issuecomment-1931969315

   ```
   1. When a walog is created include the tablet server and its current RG in 
its name. Currently the name only includes the tserver
   2. If a log ever needs recovery perfer tablet servers in the same RG
   3. If there are no tservers present in the RG, then use any server 
(tsever,compactor, scan server)
   ```
   
   I think to do this efficiently we would need to modify the way that 
TabletServers and ScanServers are registered in ZooKeeper. The 
[Compactor](https://github.com/apache/accumulo/blob/elasticity/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java#L250)
 ZooKeeper path has the resource group in it, but the 
[TabletServer](https://github.com/apache/accumulo/blob/elasticity/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java#L494)
 and 
[ScanServer](https://github.com/apache/accumulo/blob/elasticity/server/tserver/src/main/java/org/apache/accumulo/tserver/ScanServer.java#L305)
 do not. I'm not sure of the consequences of making this change.
   


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