dlmarion commented on issue #2124: URL: https://github.com/apache/accumulo/issues/2124#issuecomment-852405108
I'm thinking that we can just remove the UnknownCompactionIdException. It's thrown in 4 cases, 3 in the Coordinator (`compactionCompleted`, `compactionFailed`, and `updateCompactionStatus`) when the compaction does not exist in the RUNNING list. If the compaction is not in the RUNNING list the first time it's called, then it's likely never going to exist. However, in `compactionCompleted` and `compactionFailed`, the compactionFinalizer is called before the error is thrown. The 4th location is in the Compactor, when the Coordinator is trying to cancel the compaction, I think we should leave that one. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
