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

Posted to issues@flink.apache.org by zentol <gi...@git.apache.org> on 2017/07/26 09:12:34 UTC

[GitHub] flink pull request #4400: [FLINK-7253] [tests] Remove CommonTestUtils#assume...

GitHub user zentol opened a pull request:

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

    [FLINK-7253] [tests] Remove CommonTestUtils#assumeJava8

    This PR is based on #4388, #4398 and #4399 .
    
    It removes all usages of `CommonTestUtils#assumeJava8` and the method itself since we no longer need it.

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

    $ git pull https://github.com/zentol/flink 7253

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

    https://github.com/apache/flink/pull/4400.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 #4400
    
----
commit 3aa97b9d0ede97f04fd33a24c8c8776cc96551fb
Author: zentol <ch...@apache.org>
Date:   2017-07-24T10:34:17Z

    [FLINK-7247] [travis] Replace java 7 build profiles

commit 52729e7148668505e5c57dd6872898f38c96b03e
Author: zentol <ch...@apache.org>
Date:   2017-07-24T11:13:34Z

    [FLINK-7249] [build] Bump java.version property to 1.8

commit e16ef3dca8b631df3c3533453ff031c5b47687cb
Author: zentol <ch...@apache.org>
Date:   2017-07-24T11:14:33Z

    [FLINK-7250] [build] Remove jdk8 profile

commit bf3dd1a1b08ce2d19f51a76a69e92f090187a5de
Author: zentol <ch...@apache.org>
Date:   2017-07-24T11:24:30Z

    [FLINK-7253] [tests] Remove CommonTestUtils#assumeJava8

----


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

[GitHub] flink issue #4400: [FLINK-7253] [tests] Remove CommonTestUtils#assumeJava8

Posted by zentol <gi...@git.apache.org>.

Github user zentol commented on the issue:

    https://github.com/apache/flink/pull/4400
  
    will fix the checkstyle violations while merging.


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

[GitHub] flink issue #4400: [FLINK-7253] [tests] Remove CommonTestUtils#assumeJava8

Posted by greghogan <gi...@git.apache.org>.

Github user greghogan commented on the issue:

    https://github.com/apache/flink/pull/4400
  
    Changes look good but unused imports checkstyle violation.


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

[GitHub] flink pull request #4400: [FLINK-7253] [tests] Remove CommonTestUtils#assume...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

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


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