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

ASF GitHub Bot commented on FLINK-4315:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2637#discussion_r85765311
  
    --- Diff: 
flink-tests/src/test/scala/org/apache/flink/api/scala/hadoop/mapred/WordCountMapredITCase.scala
 ---
    @@ -42,8 +42,8 @@ class WordCountMapredITCase extends JavaProgramTestBase {
       protected def testProgram() {
         val env = ExecutionEnvironment.getExecutionEnvironment
     
    -    val input =
    -      env.readHadoopFile(new TextInputFormat, classOf[LongWritable], 
classOf[Text], textPath)
    +    val input = FlinkHadoopEnvironment.getHadoopEnvironment(env).
    --- End diff --
    
    Please include the deprecated method in the test as done with the Java 
tests.


> Deprecate Hadoop dependent methods in flink-java
> ------------------------------------------------
>
>                 Key: FLINK-4315
>                 URL: https://issues.apache.org/jira/browse/FLINK-4315
>             Project: Flink
>          Issue Type: Task
>          Components: Java API
>            Reporter: Stephan Ewen
>            Assignee: Evgeny Kincharov
>             Fix For: 2.0.0
>
>
> The API projects should be independent of Hadoop, because Hadoop is not an 
> integral part of the Flink stack, and we should have the option to offer 
> Flink without Hadoop dependencies.
> The current batch APIs have a hard dependency on Hadoop, mainly because the 
> API has utility methods like `readHadoopFile(...)`.
> I suggest to deprecate those methods and add helpers in the 
> `flink-hadoop-compatibility` project.
> FLINK-4048 will later remove the deprecated methods.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to