Todd Lipcon has posted comments on this change. Change subject: Fix RAT issues for 1.3 release ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/6330/1//COMMIT_MSG Commit Message: PS1, Line 9: * Forgot to add x509_check_host to the RAT exclude list. It's already : mentioned in LICENSE.txt > Consider updating this part -- as I understand, those were there, but under Done 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 corresp Done 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 Done -- 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-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
