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 production cluster point to http://.../1.7.0-cdh5.12/docs Definitely we want to remove the -cdh mumbo jumbo, but don't we want to keep -SNAPSHOT? The link won't be any more broken now than a link to an unreleased version, and if we do publish the website from master, then I think we should have the -SNAPSHOT in the URL. -- To view, visit http://gerrit.cloudera.org:8080/9280 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I357693122d27337183cb85c677a85e7b8d63fe48 Gerrit-Change-Number: 9280 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Comment-Date: Thu, 22 Feb 2018 20:11:49 +0000 Gerrit-HasComments: No