[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: I wonder if it's worth embedding a specific version into our release at all. It

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: > Patch Set 3: > > (3 comments) > > > Patch Set 3: > > > > > It's not just for

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: (3 comments) > Patch Set 3: > > > It's not just for dev builds. What prompted

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/9280/3/src/kudu/util/versio

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: > It's not just for dev builds. What prompted the change was that I > saw a

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: (2 comments) > Patch Set 3: > > (2 comments) > > > This is only an issue in de

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: (2 comments) > This is only an issue in development while working on unrelea

[kudu-CR] util: fix logged Kudu URL

2018-02-22 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/9280 ) Change subject: util: fix logged Kudu URL .. Patch Set 3: This is only an issue in development while working on unreleased code right? E

[kudu-CR] util: fix logged Kudu URL

2018-02-21 Thread Andrew Wong (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9280 to look at the new patch set (#3). Change subject: util: fix logged Kudu URL .. util: fix logged Kudu

[kudu-CR] util: fix logged Kudu URL

2018-02-21 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9280 to look at the new patch set (#2). Change subject: util: fix logged Kudu URL .. util: fix logged Kudu URL Previousl

[kudu-CR] util: fix logged Kudu URL

2018-02-09 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9280 Change subject: util: fix logged Kudu URL .. util: fix logged Kudu URL Previously we would log urls like: http://kudu.apache.org/releases/