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

Rui Li commented on FLINK-13400:
--------------------------------

The reason why we introduced hive deps into sql-client was because we don't 
have e2e tests for hive. So I guess we should figure out a way to add hive e2e 
test (perhaps using docker?) then we can remove these dependencies from 
sql-client.

> Remove Hive and Hadoop dependencies from SQL Client
> ---------------------------------------------------
>
>                 Key: FLINK-13400
>                 URL: https://issues.apache.org/jira/browse/FLINK-13400
>             Project: Flink
>          Issue Type: Bug
>          Components: Table SQL / Client
>            Reporter: Timo Walther
>            Priority: Critical
>              Labels: auto-unassigned, stale-critical
>             Fix For: 1.14.0
>
>
> 340/550 lines in the SQL Client {{pom.xml}} are just around Hive and Hadoop 
> dependencies.  Hive has nothing to do with the SQL Client and it will be hard 
> to maintain the long list of  exclusion there. Some dependencies are even in 
> a {{provided}} scope and not {{test}} scope.
> We should remove all dependencies on Hive/Hadoop and replace catalog-related 
> tests by a testing catalog. Similar to how we tests source/sinks.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to