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

Stephan Ewen commented on FLINK-20098:
--------------------------------------

[~aljoscha] and [~jark] What is the conclusion about the issue that Jark 
raised? Meaning the regression that some SQL dependencies are now not available 
out of the box?

> Don't add flink-connector-files to flink-dist, make dependencies explicit
> -------------------------------------------------------------------------
>
>                 Key: FLINK-20098
>                 URL: https://issues.apache.org/jira/browse/FLINK-20098
>             Project: Flink
>          Issue Type: Improvement
>          Components: API / DataStream
>            Reporter: Aljoscha Krettek
>            Assignee: Aljoscha Krettek
>            Priority: Blocker
>              Labels: pull-request-available
>             Fix For: 1.12.0
>
>
> We currently add both {{flink-connector-files}} and {{flink-connector-base}} 
> to {{flink-dist}}. 
> This implies, that users should use the dependency like this:
> {code}
> <dependency>
>       <groupId>org.apache.flink</groupId>
>       <artifactId>flink-connector-files</artifactId>
>       <version>${project.version}</version>
>       <scope>provided</scope>
> </dependency>
> {code}
> which differs from other connectors where users don't need to specify 
> {{<scope>provided</scope>}}.
> Also, {{flink-connector-files}} has {{flink-connector-base}} as a provided 
> dependency, which means that examples that use this dependency will not run 
> out-of-box in IntelliJ because transitive provided dependencies will not be 
> considered.
> I propose to just remove the dependencies from {{flink-dist}} and let users 
> use the File Connector like any other connector.
> I believe the initial motivation for "providing" the File Connector in 
> {{flink-dist}} was to allow us to use the File Connector under the hood in 
> methods such as {{StreamExecutionEnvironment.readFile(...)}}. We could decide 
> to deprecate and remove those methods or re-add the File Connector as an 
> explicit (non-provided) dependency again in the future.



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

Reply via email to