ctubbsii commented on pull request #1819: URL: https://github.com/apache/accumulo/pull/1819#issuecomment-736805760
> One big difference between this change and #1812 is that here we know exactly why we are making a call to ZooCache. For almost of all these occurrences, the table has to already exist. We won't ever call `new Tablet()` on a table that doesn't exist yet. But these aren't just checking for properties existence of the table itself. We're also checking for properties which could be created later and whose non-existence could be cached if the watcher isn't set... like a per-table context classpath, which can change the behavior of how the next iterators are loaded for that table. If its non-existence is cached, and that cache is never updated, we will never see that this context classpath has been updated and could load the wrong iterator (or fail to load the iterator) the next time we try to read that property to load an iterator class. ---------------------------------------------------------------- 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]
