keith-turner opened a new issue, #3314: URL: https://github.com/apache/accumulo/issues/3314
**Describe the bug** While working on #3284 I noticed this code : https://github.com/apache/accumulo/blob/15c9f55280f4ce28a80b800f1e05128f33c85db3/server/base/src/main/java/org/apache/accumulo/server/manager/state/MetaDataStateStore.java#L140-L151 Tracing back what calls this method, AFAICT its only passed TabletLocationState objects with a non-null suspend. If my analysis is correct then this method ignores all of its input. Seems like the code should be : ```java for (TabletLocationState tls : tablets) { if (tls.suspend != null) { tabletsMutator.mutateTablet(tls.extent).deleteSuspension().mutate(); } } ``` This need a bit more investigation. Took a quick look at 1.10 code and it seems to be the same there. **Expected behavior** The suspension is properly cleared. -- 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]
