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

Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2020/01/20 10:07:25 UTC

[GitHub] [zookeeper] ztzg commented on issue #1134: ZOOKEEPER-1112: Add (Cyrus) SASL authentication support to C client library

ztzg commented on issue #1134: ZOOKEEPER-1112: Add (Cyrus) SASL authentication support to C client library
URL: https://github.com/apache/zookeeper/pull/1134#issuecomment-576200803
 
 
   Hi @eolivelli, @anmolnar, @nkalmar, @phunt:
   
   I understand that you are all busy, but am wondering: is there anything else I can do to facilitate reviewing this PR?
   
   I don't know if the ["review guide"](https://github.com/apache/zookeeper/pull/1134#issuecomment-569935984) I posted above is useful.  I don't know if you do your reviews using the GitHub interface, or something else; in any case, let me know if you would prefer me to re-split the patches along the lines of the "guide."
   
   What do you think?

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


With regards,
Apache Git Services