You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Chesnay Schepler (JIRA)" <ji...@apache.org> on 2019/02/22 12:41:00 UTC

[jira] [Updated] (FLINK-11657) Remove redundant test path in WordCountMapreduceITCase

     [ https://issues.apache.org/jira/browse/FLINK-11657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chesnay Schepler updated FLINK-11657:
-------------------------------------
    Summary: Remove redundant test path in WordCountMapreduceITCase  (was: Fix deprecated API compatibility issue for WordCountMapreduceITCase)

> Remove redundant test path in WordCountMapreduceITCase
> ------------------------------------------------------
>
>                 Key: FLINK-11657
>                 URL: https://issues.apache.org/jira/browse/FLINK-11657
>             Project: Flink
>          Issue Type: Improvement
>          Components: Batch Connectors and Input/Output Formats, Tests
>    Affects Versions: 1.6.4, 1.7.2, 1.8.0
>            Reporter: vinoyang
>            Assignee: TANG Wen-hui
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> method:
>  
> {code:java}
> protected void testProgram() throws Exception {
>    internalRun(true);
>    postSubmit();
>    resultPath = getTempDirPath("result2");
>    internalRun(false);
>    postSubmit();
> }
> {code}
> Previously, in order to test deprecated API, the following code snippet existed:
>  
> {code:java}
> if (isTestDeprecatedAPI) {
>    input = env.createInput(HadoopInputs.readHadoopFile(new TextInputFormat(),
>       LongWritable.class, Text.class, textPath));
> } else {
>    input = env.createInput(readHadoopFile(new TextInputFormat(),
>       LongWritable.class, Text.class, textPath));
> }
> {code}
> But now, the related APIs have been removed from the DataSet, so the code needs to be refactored to make it clearer.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)