[
https://issues.apache.org/jira/browse/DRILL-5568?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16054922#comment-16054922
]
ASF GitHub Bot commented on DRILL-5568:
---------------------------------------
Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/849#discussion_r122847079
--- Diff:
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillFactory.java ---
@@ -37,6 +38,24 @@
protected final int major;
protected final int minor;
+ static {
+ Properties prop = new Properties();
--- End diff --
Moved it to SecurityConfiguration.java. All unit tests are also passing and
the application is also running fine.
> Include hadoop-common jars inside drill-jdbc-all.jar
> ----------------------------------------------------
>
> Key: DRILL-5568
> URL: https://issues.apache.org/jira/browse/DRILL-5568
> Project: Apache Drill
> Issue Type: Bug
> Components: Client - JDBC
> Reporter: Sorabh Hamirwasia
> Assignee: Sorabh Hamirwasia
>
> With Sasl support in 1.10 the authentication using username/password was
> moved to Plain Mechanism of Sasl Framework. There are couple of Hadoop
> classes like Configuration.java and UserGroupInformation.java defined in
> hadoop-common package which were used in DrillClient for security mechanisms
> like Plain/Kerberos mechanisms. Due to this we need to add hadoop dependency
> inside _drill-jdbc-all.jar_ Without it the application using this driver
> will fail to connect to Drill with authentication enabled.
> Today this jar (which is JDBC driver for Drill) already has lots of other
> dependencies which DrillClient relies on like Netty, etc. But the way we add
> these dependencies are under *oadd* namespace so that the application using
> this driver won't end up in conflict with it's own version of same
> dependencies. As part of this JIRA it will include hadoop-common dependencies
> under same namespace. This will allow an application to connect to Drill
> using this driver with security enabled.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
