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

Posted to dev@curator.apache.org by GitBox <gi...@apache.org> on 2021/06/14 07:52:06 UTC

[GitHub] [curator] Randgalt edited a comment on pull request #384: CURATOR-592 Initial commit, no enough tests

Randgalt edited a comment on pull request #384:
URL: https://github.com/apache/curator/pull/384#issuecomment-860149800


   While I understand why this might be useful for you I'm concerned about adding such a bespoke feature to Curator. The lock code is already complicated and introducing this shim makes it much harder to reason about. We can't predict what this proc will do in the context of the lock mechanics. I'm sorry but I'm -1 on this change. But maybe one of the other committers will accept it.


-- 
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