You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by StephanEwen <gi...@git.apache.org> on 2016/08/02 14:40:13 UTC

[GitHub] flink issue #2298: [FLINK-4247] [table] CsvTableSource.getDataSet() expects ...

Github user StephanEwen commented on the issue:

    https://github.com/apache/flink/pull/2298
  
    Would be great to reflect the "do not use" with proper visibility. A JavaDoc comment is usually not helping at all.
    
    Also, the dependency on the java API classes in the Scala classes does not seem right. Usually the Scala classes hide the Java API streams and environments.
    
    Both of these things are usually strong indicators that there are some abstraction issues.
    Would be good to get that right, while the API is fresh and malleable.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---