Right; that version were posted accidentally -- I actually only posted the update for the next patch (unit tests), not going to post the update for the first one because it was not ready yet. But gerrit updated this one as it sensed the base has been changed as well.
Please hold on reviewing this version -- I'll post it later, most likely tomorrow in the morning. Thanks! /Alexey On Thu, Jan 12, 2017 at 9:06 PM, Dan Burkert (Code Review) < [email protected]> wrote: > Dan Burkert has posted comments on this change. > > Change subject: [TLS certs management] initial commit > ...................................................................... > > > Patch Set 2: > > crypto_engine and crypto_common are no longer needed, right? > > -- > To view, visit http://gerrit.cloudera.org:8080/5671 > To unsubscribe, visit http://gerrit.cloudera.org:8080/settings > > Gerrit-MessageType: comment > Gerrit-Change-Id: Ic2ce55d38f4d06172fadaaa702f4550997d9bc8f > Gerrit-PatchSet: 2 > Gerrit-Project: kudu > Gerrit-Branch: master > Gerrit-Owner: Alexey Serbin <[email protected]> > Gerrit-Reviewer: Dan Burkert <[email protected]> > Gerrit-Reviewer: Kudu Jenkins > Gerrit-Reviewer: Mike Percy <[email protected]> > Gerrit-Reviewer: Todd Lipcon <[email protected]> > Gerrit-HasComments: No >
