Todd Lipcon has posted comments on this change.

Change subject: Fix and improve code coverage
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2448/2/build-support/jenkins/build-and-test.sh
File build-support/jenkins/build-and-test.sh:

Line 306:       -r $SOURCE_ROOT/src \
> So what does this mean? It'll recurse over the entire source root (includin
yea. the problem was with src/ as the root, it would generate the file names as 
'kudu/util/slice.h' instead of 'src/kudu/util/slice.h.' Sonar then got confused 
when it couldn't find these patches relative to the repository root, and I 
couldn't see an easy way to fix Sonar, so I changed it here.


-- 
To view, visit http://gerrit.cloudera.org:8080/2448
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If0273fe095d8f160f0068ec421fceb54eda588de
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to