keith-turner commented on issue #3512:
URL: https://github.com/apache/accumulo/issues/3512#issuecomment-1613632758

   One way to fix this is to have single method like the following in 
SessionManager that is used for all session cleanup.
   
   ```java
   void cleanup(Collection<Session> sessions){
    for(var session : sessions){
       if(!session.cleanup()){
           // Queue session for defered clean up at later time.  This could be 
done by scheduling a 
           // task to run later or adding it a collection that is periodically 
inspected by a scheduled 
           // task.  The sweep() method in session manager method handles 
deferred cleanup for 
           // itself it, this could be generalized for the entire class.  The 
sweep() method is already
           // called periodically.
      }
   }
   ```


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