[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. IMPALA-5184: build fe against both Hive 1 & 2 APIs This adds a compatibility shim layer wi

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 11: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5538 To unsubscribe, vi

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-05-08 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 11: Code-Review+2 Fixed JDBC test failure. -- To view, visit http://gerrit.cloudera.org:8080/55

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-05-08 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 11: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/541/ -- To vi

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-05-08 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5538 to look at the new patch set (#11). Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs ..

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 10: Code-Review+2 (2 comments) http://gerrit.cloudera.org:8080/#/c/5538/9/common/thrift/CMakeLists.t

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 10: Code-Review+1 carry +1 -- To view, visit http://gerrit.cloudera.org:8080/5538 To unsubscrib

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/5538/9/fe/src/compat-hive-2/java/org/apache/i

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 9: (8 comments) http://gerrit.cloudera.org:8080/#/c/5538/9/common/thrift/CMakeLists.txt File com

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5538 to look at the new patch set (#10). Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. I

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-17 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 9: (8 comments) Unfortunate that we have to do this, but I can't think of a better way. http://gerr

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-11 Thread Tim Armstrong (Code Review)
Hello Bharath Vissapragada, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5538 to look at the new patch set (#9). Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. IM

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/5538/8/fe/src/compat-hive-1/java/org/apache/

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-11 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 9: Code-Review+1 Carry +1 -- To view, visit http://gerrit.cloudera.org:8080/5538 To unsubscribe

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-11 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 8: Code-Review+1 (6 comments) http://gerrit.cloudera.org:8080/#/c/5538/7/bin/impala-confi

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 7: (11 comments) http://gerrit.cloudera.org:8080/#/c/5538/7//COMMIT_MSG Commit Message: Line 9:

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#8). Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. IMPALA-5184: build fe against both Hive 1 & 2 APIs This adds a compatibility shim layer with Hive 1 and Hive

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-10 Thread Bharath Vissapragada (Code Review)
Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. Patch Set 7: (9 comments) http://gerrit.cloudera.org:8080/#/c/5538/7//COMMIT_MSG Commit Message: L

[Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs

2017-04-10 Thread Tim Armstrong (Code Review)
Tim Armstrong has uploaded a new patch set (#7). Change subject: IMPALA-5184: build fe against both Hive 1 & 2 APIs .. IMPALA-5184: build fe against both Hive 1 & 2 APIs This adds a compatibility shim layer with Hive 1 and Hive