Login  Register

[jira] [Commented] (FLINK-931) Remove redundant (old) operator tests

Posted by Shang Yuanchun (Jira) on Jun 12, 2014; 6:33pm
URL: http://deprecated-apache-flink-mailing-list-archive.368.s1.nabble.com/jira-Commented-FLINK-931-Remove-redundant-old-operator-tests-tp127.html


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

Fabian Hueske commented on FLINK-931:
-------------------------------------

The old tests are NOT subsumed by the new tests.
The tests under eu.stratosphere.test.operators test all physical execution strategies for each (basic) operator. The tests under eu.stratosphere.test.japiOperators basically test that the transformation before the optimization and the optimizer postpass are working correctly (assembly of program, serializers and comparators). They do not run all physical strategies.

I would leave the tests in place as they are.

> Remove redundant (old) operator tests
> -------------------------------------
>
>                 Key: FLINK-931
>                 URL: https://issues.apache.org/jira/browse/FLINK-931
>             Project: Flink
>          Issue Type: Bug
>            Reporter: Stephan Ewen
>            Assignee: Stephan Ewen
>            Priority: Minor
>
> I would remove all tests in the package "eu.stratosphere.test.operators".
> There are many tests in "eu.stratosphere.test.javaApiOperators" that cover the same operator code paths.
> The redundant tests take a lot of time without really adding test coverage.



--
This message was sent by Atlassian JIRA
(v6.2#6252)