You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Matthew Jacobs (Code Review)" <ge...@cloudera.org> on 2017/03/29 16:51:54 UTC

[native-toolchain-CR] Bump Kudu version to 16dd6e4 and remove historical versions

Matthew Jacobs has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/6509

Change subject: Bump Kudu version to 16dd6e4 and remove historical versions
......................................................................

Bump Kudu version to 16dd6e4 and remove historical versions

The historical versions weren't ever being built, so there's
no point in keeping the growing list of Kudu git hashes. The
versions are all in this git repository's history, anyway.

Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
---
M buildall.sh
1 file changed, 5 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/09/6509/1
-- 
To view, visit http://gerrit.cloudera.org:8080/6509
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
Gerrit-PatchSet: 1
Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>

[native-toolchain-CR] Bump Kudu version to 16dd6e4 and remove historical versions

Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has submitted this change and it was merged.

Change subject: Bump Kudu version to 16dd6e4 and remove historical versions
......................................................................


Bump Kudu version to 16dd6e4 and remove historical versions

The historical versions weren't ever being built, so there's
no point in keeping the growing list of Kudu git hashes. The
versions are all in this git repository's history, anyway.

Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
---
M buildall.sh
1 file changed, 5 insertions(+), 15 deletions(-)

Approvals:
  Matthew Jacobs: Verified
  Tim Armstrong: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
Gerrit-PatchSet: 1
Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>

[native-toolchain-CR] Bump Kudu version to 16dd6e4 and remove historical versions

Posted by "Tim Armstrong (Code Review)" <ge...@cloudera.org>.
Tim Armstrong has posted comments on this change.

Change subject: Bump Kudu version to 16dd6e4 and remove historical versions
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
Gerrit-PatchSet: 1
Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-HasComments: No

[native-toolchain-CR] Bump Kudu version to 16dd6e4 and remove historical versions

Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has posted comments on this change.

Change subject: Bump Kudu version to 16dd6e4 and remove historical versions
......................................................................


Patch Set 1: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I291116d9c05d85d96879001ea6a89dfd6bc00661
Gerrit-PatchSet: 1
Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <ta...@cloudera.com>
Gerrit-HasComments: No