[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-31 Thread Harrison Sheinblatt (Code Review) 
Harrison Sheinblatt has submitted this change and it was merged.

Change subject: Bump Impala version to 2.8
..


Bump Impala version to 2.8

Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Reviewed-on: http://gerrit.cloudera.org:8080/4170
Tested-by: Internal Jenkins
Reviewed-by: Alex Behm 
---
M bin/save-version.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Internal Jenkins: Verified
  Alex Behm: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Harrison Sheinblatt 
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-31 Thread Alex Behm (Code Review) 
Alex Behm has posted comments on this change.

Change subject: Bump Impala version to 2.8
..


Patch Set 2: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Alex Behm 
Gerrit-Reviewer: Harrison Sheinblatt 
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-31 Thread Internal Jenkins (Code Review) 
Internal Jenkins has posted comments on this change.

Change subject: Bump Impala version to 2.8
..


Patch Set 2: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Harrison Sheinblatt 
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-30 Thread Thomas Tauber-Marshall (Code Review) 
Thomas Tauber-Marshall has posted comments on this change.

Change subject: Bump Impala version to 2.8
..


Patch Set 1: Code-Review+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Harrison Sheinblatt 
Gerrit-Reviewer: Jim Apple 
Gerrit-Reviewer: Thomas Tauber-Marshall 
Gerrit-HasComments: No

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-30 Thread Harrison Sheinblatt (Code Review) 
Harrison Sheinblatt has posted comments on this change.

Change subject: Bump Impala version to 2.8
..


Patch Set 1:

(1 comment)

Responded

http://gerrit.cloudera.org:8080/#/c/4170/1/bin/save-version.sh
File bin/save-version.sh:

PS1, Line 24: cdh5
> Can this be removed?
I think it would break our build.  I'd like to get the build working on master 
without changing packaging code and then work on how to remove this dependency 
from any scripts.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Harrison Sheinblatt 
Gerrit-Reviewer: Jim Apple 
Gerrit-HasComments: Yes

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-30 Thread Jim Apple (Code Review) 
Jim Apple has posted comments on this change.

Change subject: Bump Impala version to 2.8
..


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4170/1/bin/save-version.sh
File bin/save-version.sh:

PS1, Line 24: cdh5
Can this be removed?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt 
Gerrit-Reviewer: Jim Apple 
Gerrit-HasComments: Yes

[Impala-ASF-CR] Bump Impala version to 2.8

2016-08-30 Thread Harrison Sheinblatt (Code Review) 
Harrison Sheinblatt has uploaded a new change for review.

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

Change subject: Bump Impala version to 2.8
..

Bump Impala version to 2.8

Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
---
M bin/save-version.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/70/4170/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4170
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5b147dee9bb7fcb159e98a008c01659a4cb7473b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Harrison Sheinblatt