zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From enixon <...@git.apache.org>
Subject [GitHub] zookeeper issue #184: ZOOKEEPER-236: SSL Support for Atomic Broadcast protoc...
Date Sat, 15 Sep 2018 16:56:29 GMT
Github user enixon commented on the issue:

    @anmolnar, We strongly agree that this PR should be merged. It has performed well in our
testing and has received extensive review at this point. I'm pretty confident in its robustness.

    Our ideal scenario would be that 184 gets merged and then 627 gets a proper review. This
would be made easier if we could stack pull requests or gracefully extract @ivmaykov 's bug
fix from the rest of the commits but in the absence of those, I'd move for committing 184
and awareness that we have contributions to add on top. 


View raw message