milleruntime commented on issue #2968: URL: https://github.com/apache/accumulo/issues/2968#issuecomment-1260741652
> The switch to using the ReadOnlyRepo was to enforce places that were reading FATEs for status / metrics were not performing updates. As long as the type declared is `ReadOnlyTStore` and not the fully functioning `TStore`, then we are fine. > the null returned by top in ZooStore will need to be correctly handled or we will have just moved the NPE to another location. This is true. It appears the code previously just printed the output of `top()` and didn't try to use it. I am not sure if this is still true. -- 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]
