Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19612 )

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
Reviewed-on: http://gerrit.cloudera.org:8080/19612
Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
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, 43 insertions(+), 3 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
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: merged
Gerrit-Change-Id: Ie184a0c2404f36a3ee28296336f6545615a5c6ca
Gerrit-Change-Number: 19612
Gerrit-PatchSet: 9
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