Alex Behm has posted comments on this change.

Change subject: IMPALA-4171: Remove JAR from repo.
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4499/2/testdata/bin/copy-udfs-udas.sh
File testdata/bin/copy-udfs-udas.sh:

Line 58:   find . -type f -name '*.java' -execdir bash -c "sed -i s/'Old 
UDF'/'New UDF'/g '{}'" \;
Seems cleaner to mention the expected .java file by name and not wildcard.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icee7b8c32b08e064dea4a14624acff6021ef5ce1
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Jim Apple <jbap...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to