[ 
https://issues.apache.org/jira/browse/HIVE-17468?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16157180#comment-16157180
 ] 

slim bouguerra commented on HIVE-17468:
---------------------------------------

[~jcamachorodriguez] 
1 looking at the deps i see calcite-druid is a dependency for hive exec anyway 
thus it is always on the classpath if am not wrong 
https://github.com/b-slim/hive/blob/453c8ed4ed17d215e367fbdd22fb082d1b0f48e4/ql/pom.xml#L397.
2 that usage you pointed is actually very minimal (a constant) thus we can 
either remove it or and use the dependency from hive exec that is on the tez 
class path as well.


> Shade and package appropriate jackson version for druid storage handler
> -----------------------------------------------------------------------
>
>                 Key: HIVE-17468
>                 URL: https://issues.apache.org/jira/browse/HIVE-17468
>             Project: Hive
>          Issue Type: Bug
>            Reporter: slim bouguerra
>            Assignee: Jesus Camacho Rodriguez
>             Fix For: 3.0.0
>
>         Attachments: HIVE-17468.2.patch, HIVE-17468.patch, hive-druid-deps.txt
>
>
> Currently we are excluding all the jackson core dependencies coming from 
> druid. This is wrong in my opinion since this will lead to the packaging of 
> unwanted jackson library from other projects.
> As you can see the file hive-druid-deps.txt currently jacskon core is coming 
> from calcite and the version is 2.6.3 which is very different from 2.4.6 used 
> by druid. This patch exclude the unwanted jars and make sure to bring in 
> druid jackson dependency from druid it self.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to