impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3223: Update path to Postgres JDBC driver
Date Fri, 27 May 2016 01:13:51 GMT
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 <kwho@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message