Alexey Serbin has posted comments on this change. Change subject: Fix RAT issues for 1.3 release ......................................................................
Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6330/1/build-support/release/rat_exclude_files.txt File build-support/release/rat_exclude_files.txt: Line 145: src/kudu/security/x509_check_host.cc It seems we have these moved from src/kudu/util. Consider removing corresponding lines. PS1, Line 165: src/kudu/util/x509_check_host.cc : src/kudu/util/x509_check_host.h remove this -- now they are in src/kudu/security -- To view, visit http://gerrit.cloudera.org:8080/6330 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I23afc5acaa7543e6f777be9d8cbc247f2bbc1f22 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-HasComments: Yes
