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

Previously we would log urls like:
http://kudu.apache.org/releases/1.7.0-SNAPSHOT/docs/...
Instead of:
http://kudu.apache.org/releases/1.7.0/docs/...

This patch parses a number out of the supplied version string. If a
version number can't be parsed out, a default URL will be used:
http://kudu.apache.org/docs/...

Change-Id: I357693122d27337183cb85c677a85e7b8d63fe48
---
M src/kudu/util/version_info.cc
M src/kudu/util/version_info.h
M src/kudu/util/website_util.cc
3 files changed, 29 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/9280/2
--
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: newpatchset
Gerrit-Change-Id: I357693122d27337183cb85c677a85e7b8d63fe48
Gerrit-Change-Number: 9280
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to