DomGarguilo commented on PR #5540: URL: https://github.com/apache/accumulo/pull/5540#issuecomment-2949971314
> Why would this no longer throw TableNotFoundException? That seems odd. @ctubbsii I added a new helper method, `modifyPropertiesUnwrapped()` which wraps `modifyProperties()` catches `AccumuloException`, and throws `TableNotFoundException` if thats its cause. I used this new helper method in only the methods that I changed here (setSamplerConfiguration, clearSamplerConfiguration, setLocalityGroups). This adds back the TableNotFoundException that had been removed by a previous change in this PR. It should also restore the exception handling functionality that was in place before converting these methods to use `modifyProperties`. -- 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-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
