Laszlo Gaal has posted comments on this change. ( http://gerrit.cloudera.org:8080/15229 )
Change subject: IMPALA-9390: Change ant and maven download URLs to valid locations ...................................................................... Patch Set 1: (1 comment) > Patch Set 1: > > (1 comment) http://gerrit.cloudera.org:8080/#/c/15229/1/bin/bootstrap_build.sh File bin/bootstrap_build.sh: http://gerrit.cloudera.org:8080/#/c/15229/1/bin/bootstrap_build.sh@50 PS1, Line 50: -nv > I think we can remove the "-nv" option to improve debuggability. The output I prefer to keep it: the default progress indicator is too chatty (at least to me), and -nv still prints a one-line summary of what happened even in a successful case. Ultimately it's a matter of personal taste, so I don't feel too strongly about it, but my personal opinion is that some of our run logs are still too verbose in a couple of places. -- To view, visit http://gerrit.cloudera.org:8080/15229 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I299bdd9cbe999ff4d483c022fa69ec88b37612f6 Gerrit-Change-Number: 15229 Gerrit-PatchSet: 1 Gerrit-Owner: Laszlo Gaal <laszlo.g...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Comment-Date: Thu, 20 Feb 2020 15:06:07 +0000 Gerrit-HasComments: Yes