dlmarion commented on issue #5162: URL: https://github.com/apache/accumulo/issues/5162#issuecomment-2555166403
> Another way to approach this would be to remove the tablet servers ZooKeeper entry on a normal shutdown. In the ServiceLock.unlock code (below) in the case of Compactors, Scan Servers, and TabletServers `path` is `/accumulo/<instanceId>/(compactors|sservers|tservers)/<resourceGroup>/<address>` and `localLock` is the ephemeral node that is created. The code is currently only deleting the ephemeral node. We could modify `ServiceLock.unlock` to accept a boolean, which when `true` could perform a recursive delete on `path`. We would only set it to `true` in the case of Compactors, ScanServers, and TabletServers when ServiceLock.unlock is called. This would mean that the LiveTServerSet running in the Manager would not find any TabletServers without locks when they are shut down gracefully. Graceful shutdown is being addressed in #5193. https://github.com/apache/accumulo/blob/e5370d1d5cd436d86170d16b0e8c0986d3275e15/core/src/main/java/org/apache/accumulo/core/lock/ServiceLock.java#L531-L558 -- 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]
