You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Matthew Jacobs (Code Review)" <ge...@cloudera.org> on 2017/02/07 18:45:47 UTC

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

Matthew Jacobs has uploaded a new change for review.

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

Change subject: Bump Kudu version to latest master (cd7b0dd)
......................................................................

Bump Kudu version to latest master (cd7b0dd)

Kudu's cmake no longer needs to be patched to work around a
cmake bug [1] because the patch is now applied in Kudu
(commit f3ec97b).

1. https://gitlab.kitware.com/cmake/cmake/issues/15873

Change-Id: Ifcd4dbb4b7e3c1ddd8c85b00ec443f7289dc1617
---
M buildall.sh
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Toolchain refs/changes/28/5928/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5928
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

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

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

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 latest master (cd7b0dd)
......................................................................


Bump Kudu version to latest master (cd7b0dd)

Kudu's cmake no longer needs to be patched to work around a
cmake bug [1] because the patch is now applied in Kudu
(commit f3ec97b).

1. https://gitlab.kitware.com/cmake/cmake/issues/15873

Change-Id: Ifcd4dbb4b7e3c1ddd8c85b00ec443f7289dc1617
---
M buildall.sh
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Matthew Jacobs: Verified
  Sailesh Mukil: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifcd4dbb4b7e3c1ddd8c85b00ec443f7289dc1617
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

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

Change subject: Bump Kudu version to latest master (cd7b0dd)
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifcd4dbb4b7e3c1ddd8c85b00ec443f7289dc1617
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>
Gerrit-HasComments: No

[Toolchain-CR] Bump Kudu version to latest master (cd7b0dd)

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

Change subject: Bump Kudu version to latest master (cd7b0dd)
......................................................................


Patch Set 1: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifcd4dbb4b7e3c1ddd8c85b00ec443f7289dc1617
Gerrit-PatchSet: 1
Gerrit-Project: Toolchain
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>
Gerrit-HasComments: No