Hello Zoltan Borok-Nagy, Joe McDonnell, Csaba Ringhofer, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#5).

Change subject: IMPALA-11992: Support setting query options in Hive JDBC's 
connection URL
......................................................................

IMPALA-11992: Support setting query options in Hive JDBC's connection URL

Hive JDBC support setting hive configurations in Connection URL's query
part, thus we can map impala's query options to hive configurations to
support setting query options on the hive JDBC client side. But hive
JDBC will add a "set:hiveconf:" prefix to the configuration key when
request OpenSession() thrift API, we should remove the configuration
key prefix in OpenSession() so that we can set valid query options.

Testing:
 - update the 'TestHS2::test_open_session_query_options' EE test.
 - add a FE test 'testSetQueryOptionsInConnectionURL' to JdbcTest.

Change-Id: Ie184a0c2404f36a3ee28296336f6545615a5c6ca
---
M be/src/service/impala-hs2-server.cc
M fe/src/test/java/org/apache/impala/service/JdbcTest.java
M fe/src/test/java/org/apache/impala/testutil/ImpalaJdbcClient.java
M tests/hs2/test_hs2.py
4 files changed, 42 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/12/19612/5
--
To view, visit http://gerrit.cloudera.org:8080/19612
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie184a0c2404f36a3ee28296336f6545615a5c6ca
Gerrit-Change-Number: 19612
Gerrit-PatchSet: 5
Gerrit-Owner: Xiang Yang <yx91...@126.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Xiang Yang <yx91...@126.com>
Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>

Reply via email to