You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/11/30 14:12:56 UTC

[GitHub] [accumulo] ctubbsii commented on pull request #1812: Stop passing watcher to ZooCache exists()

ctubbsii commented on pull request #1812:
URL: https://github.com/apache/accumulo/pull/1812#issuecomment-735809919


   > @EdColeman and I were hoping we could at least fix #1809 until it can be rewritten.
   
   Given the fact that this breaks a unit test (ZooCacheTest is now failing), I don't have a high degree of confidence that this isn't going to break other things. Given this code is as complicated as it is, it seems like it's too risky to make such a change, and we should probably just proceed to removing the need for watching so many nodes in the first place, as previously discussed in other issues for 2.1.


----------------------------------------------------------------
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:
users@infra.apache.org