flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zentol <...@git.apache.org>
Subject [GitHub] flink pull request: [FLINK-2901] Remove Record API dependencies fr...
Date Wed, 28 Oct 2015 10:45:09 GMT
GitHub user zentol opened a pull request:

    https://github.com/apache/flink/pull/1306

    [FLINK-2901] Remove Record API dependencies from flink-tests

    This PR removes all dependencies to the Record API in the flink-tests project.
    
    For this, tests were either ported or removed.
    
    The following files were NOT ported:
    * recordJobs/*
     * except kmeans/udfs CoordVector, PointInFormat (moved & ported)
     * except util/InfiniteIntegerInputFormat (moved & ported)
    * recordJobTests/*
    * operators/*
     *  except CoGroupSortITCase, MapPartitionITCase, ObjectReuseITCase (moved)
    * accumulators/AccumulatorIterativeITCase
     * Unfinished test, should be addressed in a separate issue
    * IterationTerminationWithTwoTails
     * nigh identical with IterationTerminationWithTerminationTail (ported version also failed)
    * iterative/DeltaPageRankITCase, IterativeKMeansITCase, KMeansITCase
     * behaved like they belong into /recordJobTests/ (as in they simply execute a recordJob),
thus removed
    * optimizer/iterations/IterativeKMeansITCase
     * overlaps with ConnectedComponentsITCase (need a second opinion on this!)
    * util/FailingTestBase
     * integrated into TaskFailureITCase (only user of the class)
    * testPrograms/util/tests/*
    
    cancelling/MatchJoinCancellingITCase was ported but disabled since it is unstable, and
also disabled before. Separate issue seems appropriate.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zentol/flink 2901_record_tests

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/1306.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1306
    
----
commit ae3e5d64240feb0f2d9f5b74b709567ee3c66ec4
Author: Fabian Hueske <fhueske@apache.org>
Date:   2015-10-22T19:13:59Z

    Remove ITCases for Record API operators

commit 88fcd3419774a48bc42c21fc46a745129bc4c0ac
Author: Fabian Hueske <fhueske@apache.org>
Date:   2015-10-22T19:15:38Z

    Move operator ITCases into correct package

commit 4b9cecbe747d1131849070223821cbb4446a2a48
Author: Fabian Hueske <fhueske@apache.org>
Date:   2015-10-22T19:18:04Z

    Remove Record API dependencies from CC iteration tests

commit 75052028dc8f29b4280d9c29d61fcb6f6c00ff0a
Author: Fabian Hueske <fhueske@apache.org>
Date:   2015-10-22T19:19:34Z

    Remove Record API dependencies from WordCount compiler test

commit 91ca56c0df8a07cd142fd764b13af6ea3d7dae8a
Author: Fabian Hueske <fhueske@apache.org>
Date:   2015-10-22T19:20:34Z

    Remove Record API program tests

commit 491afa1ea220f5292c36910d5c4f0c2491edc859
Author: zentol <s.motsu@web.de>
Date:   2015-10-25T14:07:37Z

    Remove deprecation warning suppressions

commit a32a0d1b33c7ffcd4a997fa1cdd11fe299bc7e60
Author: zentol <s.motsu@web.de>
Date:   2015-10-27T20:22:38Z

    [FLINK-2901] Remove Record API dependencies from flink-tests #1

commit 0600da3154665aaf1ee02510e24f8222ca12af5f
Author: zentol <s.motsu@web.de>
Date:   2015-10-27T20:22:45Z

    [FLINK-2901] Remove Record API dependencies from flink-tests #2

----


---
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.
---

Mime
View raw message