-
Notifications
You must be signed in to change notification settings - Fork 230
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
Add version properties to integration tests modules #3907
Add version properties to integration tests modules #3907
Conversation
Signed-off-by: Peixin Li <pxli@nyu.edu>
build |
udf-examples/pom.xml
Outdated
@@ -74,6 +74,32 @@ | |||
</resource> | |||
</resources> | |||
<plugins> | |||
<plugin> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be getting picked up by the pluginManagement
section of the parent pom. I verified that removing the change to this file doesn't seem to affect the contents of the udf-examples jar.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch, thx! removed
build |
Signed-off-by: Peixin Li pxli@nyu.edu
related to #3652
It's difficult to fetch artifacts by git commit from maven repo.
let's add build info to all artifacts in integration tests for better debugging context first,
and I will try adjust our nightly build triggering graph later