flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-7350) only execute japicmp in one build profile
Date Wed, 02 Aug 2017 10:15:00 GMT

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

ASF GitHub Bot commented on FLINK-7350:

GitHub user zentol opened a pull request:


    [FLINK-7350] [travis] Only execute japicmp in misc profile

    Based on #4460.
    ## What is the purpose of the change
    Improve travis build times and stability by skipping redundant japicmp executions.
    Note that the japicmp 0.7 does not allow skipping the execution form the command-line
(only added in 0.10), so I worked around that with a profile that is activated by specifying
`-Djapicmp.skip=true`, which is how it _would_ work in 0.10.
    ## Brief change log
      - add `japicmp.skip` property to skip execution
      - add `skip-japicmp` profile to skip execution, activated by specifying "-Djapicmp.skip=true"
      - add `-Djapicmp.skip=true` to core/libraries/connector/tests profile options
    ## Verifying this change
    Remove the `@Public` annotation from any class and run it on travis, should only fail
1 profile (i.e. 2 builds)
    ## Does this pull request potentially affect one of the following parts:
      - Dependencies (does it add or upgrade a dependency): (no)
      - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (no)
      - The serializers: (no)
      - The runtime per-record code paths (performance sensitive): (no)
      - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing,
Yarn/Mesos, ZooKeeper: (no)
    ## Documentation
      - Does this pull request introduce a new feature? (no)
      - If yes, how is the feature documented? (not applicable)

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

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

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


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

    This closes #4461
commit 4730355b9efa4169f4739681999c16000d9eb3bd
Author: zentol <chesnay@apache.org>
Date:   2017-08-02T10:05:56Z

    [FLINK-7349] [travis] Only execute checkstyle in misc profile

commit 855e4effb5dfae27a56fb703ddd1fc3785814538
Author: zentol <chesnay@apache.org>
Date:   2017-08-02T10:06:56Z

    [FLINK-7350] [travis] Only execute japicmp in misc profile


> only execute japicmp in one build profile
> -----------------------------------------
>                 Key: FLINK-7350
>                 URL: https://issues.apache.org/jira/browse/FLINK-7350
>             Project: Flink
>          Issue Type: Improvement
>          Components: Travis
>    Affects Versions: 1.4.0
>            Reporter: Chesnay Schepler
>            Assignee: Chesnay Schepler
>             Fix For: 1.4.0
> Similarly to FLINK-7349 we improve build times (and stability!) by only executing the
japicmp plugin in the build profile that builds the all of flink.

This message was sent by Atlassian JIRA

View raw message