Alex Behm has posted comments on this change.

Change subject: IMPALA-3223: Update path to Postgres JDBC driver
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3232/1/bin/impala-config.sh
File bin/impala-config.sh:

Line 337: 
JDBC_DRIVER=$IMPALA_FE_DIR/target/dependency/postgresql-${IMPALA_POSTGRES_JDBC_DRIVER_VERSION}.jdbc4.jar
Wo we even need JDBC_DRIVER now? Why not directly export POSTGRES_JDBC_DRIVER 
and use that in HIVE_AUX_JARS_PATH as well.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I76bce18fd308890e66615c8d08d5e58f02a8a132
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-HasComments: Yes

Reply via email to