David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/15537 )
Change subject: IMPALA-9107: Add timestamp to maven logging options. ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/15537/1/bin/impala-config.sh File bin/impala-config.sh: http://gerrit.cloudera.org:8080/#/c/15537/1/bin/impala-config.sh@328 PS1, Line 328: export IMPALA_MAVEN_OPTIONS=${IMPALA_MAVEN_OPTIONS-} : : # If enabled, debug symbols are added to cross-comp > Maybe it's better to just to this at the same place in mvn-quiet.sh? OK, there was a typo in this comment, but you get the idea. -- To view, visit http://gerrit.cloudera.org:8080/15537 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I10fbe9eb76b66e6ba00db9f95c91063410dd1b4e Gerrit-Change-Number: 15537 Gerrit-PatchSet: 2 Gerrit-Owner: David Knupp <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Comment-Date: Tue, 24 Mar 2020 00:36:25 +0000 Gerrit-HasComments: Yes
