Adar Dembo has posted comments on this change.

Change subject: KUDU-733: Fix javadoc warnings and fail build on warnings
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3043/1/docs/support/scripts/make_site.sh
File docs/support/scripts/make_site.sh:

Line 65: if mvn clean javadoc:aggregate | tee /dev/tty | fgrep -q "Javadoc 
Warnings"; then
Does redirecting to /dev/tty do the right thing when make_site.sh is invoked 
from a non-interactive shell (i.e. from Jenkins or perhaps from ssh -c), and 
from a pseudo-tty (i.e. ssh)?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iaf466d99d57ac2ba11f2af457f09bb5c7853f699
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to