Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

rongou
Copy link
Contributor

@rongou rongou commented Feb 16, 2021

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
Copy link
Contributor Author

rongou commented Feb 17, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit d180213 into rapidsai: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