You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/02/17 15:08:49 UTC

[GitHub] [accumulo] milleruntime commented on pull request #2169: Cancel compactions when delete table is called

milleruntime commented on pull request #2169:
URL: https://github.com/apache/accumulo/pull/2169#issuecomment-1043050205


   @dlmarion @keith-turner I noticed a shared static method between 2 separate FATE operations that was added in this PR and was wondering if this could cause concurrency problems. https://github.com/apache/accumulo/blob/3b2cdb99aeb60dfcc58eba26b6620825aadb560b/server/manager/src/main/java/org/apache/accumulo/manager/tableOps/compact/cancel/CancelCompactions.java#L67
   
   I am not sure of what the situation would be but the fact that these two operations mutate the same node in ZK (compact-cancel-id) makes me curious. If they are both running and hit ZK problems, I wonder if a cancel compaction may cause a delete table to fail.
   Delete Table (PreDeleteTable)
   Cancel compaction (CancelCompactions)


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org