Skip to content

upgrade maven-antrun-plugin to support maven parallel builds [skip ci] - #7393

Merged
rapids-bot[bot] merged 1 commit into
NVIDIA:branch-0.19from
rongou:upgrade-maven-antrun
Feb 17, 2021
Merged

upgrade maven-antrun-plugin to support maven parallel builds [skip ci]#7393
rapids-bot[bot] merged 1 commit into
NVIDIA:branch-0.19from
rongou:upgrade-maven-antrun

Conversation

@rongou

@rongou rongou commented Feb 16, 2021

Copy link
Copy Markdown
Contributor

When running maven in parallel build mode (mvn -T 1C), getting this warning:

[WARNING] *****************************************************************
[WARNING] * Your build is requesting parallel execution, but project      *
[WARNING] * contains the following plugin(s) that have goals not marked   *
[WARNING] * as @threadSafe to support parallel building.                  *
[WARNING] * While this /may/ work fine, please look for plugin updates    *
[WARNING] * and/or request plugins be made thread-safe.                   *
[WARNING] * If reporting an issue, report it against the plugin in        *
[WARNING] * question, not against maven-core                              *
[WARNING] *****************************************************************
[WARNING] The following plugins are not marked @threadSafe in cudfjni:
[WARNING] org.apache.maven.plugins:maven-antrun-plugin:1.3
[WARNING] Enable debug to see more precisely which goals are not marked @threadSafe.
[WARNING] *****************************************************************

Upgrading the org.apache.maven.plugins:maven-antrun-plugin got rid of the warning.

@jlowe @revans2

@rongou rongou added 3 - Ready for Review Ready for review by team Java Affects Java cuDF API. Spark Functionality that helps Spark RAPIDS tech debt improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Feb 16, 2021
@rongou
rongou requested a review from a team as a code owner February 16, 2021 20:28
@rongou rongou added 5 - Ready to Merge Testing and reviews complete, ready to merge and removed 3 - Ready for Review Ready for review by team labels Feb 17, 2021
@rongou

rongou commented Feb 17, 2021

Copy link
Copy Markdown
Contributor Author

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit d180213 into NVIDIA:branch-0.19 Feb 17, 2021
@rongou
rongou deleted the upgrade-maven-antrun branch February 17, 2021 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to Merge Testing and reviews complete, ready to merge improvement Improvement / enhancement to an existing function Java Affects Java cuDF API. non-breaking Non-breaking change Spark Functionality that helps Spark RAPIDS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants