impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bharath Vissapragada (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5184: build fe against both Hive 1 & 2 APIs
Date Tue, 11 Apr 2017 18:02:42 GMT
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

File bin/

Line 377: # a complete Hive build.
> I ran into the dreaded sticky config variable problem with some of these va
Thanks for fixing it.
File common/thrift/.gitignore:

Line 4: hive-2-api/TCLIService.thrift
> The other one is checked in, so any changes to it shouldn't be ignored.
Ok I wasn't aware of that. Thanks for clarifying.
File common/thrift/CMakeLists.txt:

Line 189: # The thrift-generated java classes defined in TCLIService are also downloaded via
> TCLIService.thrift from Hive 2 adds additional RPC endpoints that we don't 
Makes sense. My only concern was that if the spec is updated, we may have to fix this logic
again. However that looks less likely, so fine by me.
File fe/src/compat-hive-1/java/org/apache/hive/service/rpc/thrift/

Line 1: package org.apache.hive.service.rpc.thrift;
Apache license in the newly added classes?
File fe/src/compat-hive-1/java/org/apache/impala/compat/

Line 16: // under the License.
nit: \n after license.
File fe/src/compat-hive-2/java/org/apache/impala/compat/

Line 100
> The thrift classes moved between packages. The code is identical but the cl
Ah, got it.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbc265281c04fe3136bc3c920dbac966742ce09a
Gerrit-PatchSet: 8
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <>
Gerrit-Reviewer: Bharath Vissapragada <>
Gerrit-Reviewer: Tim Armstrong <>
Gerrit-HasComments: Yes

View raw message