impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthew Jacobs (Code Review)" <ger...@cloudera.org>
Subject [Toolchain-CR] Versioning of build artifacts
Date Tue, 18 Oct 2016 01:01:36 GMT
Matthew Jacobs has posted comments on this change.

Change subject: Versioning of build artifacts
......................................................................


Patch Set 1:

> My feeling was that we generally be referencing a fixed version of
 > the toolchain artifacts in most cases (rather than using "latest").
 > I think it could be useful to have that pointer, but I'm not sure
 > that it's the right thing to use for many use cases (beyond people
 > experimenting with a new toolchain build).
 > 
 > E.g. I was thinking we would probably bump toolchain versions along
 > with component versions in most cases.

Hm OK, that seems fine, just kind of a hassle if you want to look at the published toolchain
since the build id doesn't indicate any ordering. I've done that a few times but I get that's
probably not very common (e.g. to download and inspect particular bits manually).
Either way, this isn't a big deal -- this looks much better.

-- 
To view, visit http://gerrit.cloudera.org:8080/4742
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb8774a7bd4bcae0c135684078f5ce89a28f6bc2
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-HasComments: No

Mime
View raw message