Skip to content

Deterministic publish order for gradle and ivy #68

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

Merged
merged 1 commit into from
Oct 14, 2015

Conversation

arothian
Copy link
Contributor

Currently the build extractors for Ivy and Gradle store the resulting list of artifacts to publish in a HashSet. This causes the artifacts to be published in a random order. This updates that to use a TreeSet and provides comparable implementations for related objects so that the published artifacts occur in a deterministic order. This is really helpful for us to determine when a build is complete. This would solve https://www.jfrog.com/jira/browse/GAP-181 in a more predictable manner than is currently implemented.

Sort the deploy artifacts using natural comparators so that they
get published to artifactory in a deterministic order.
@arothian
Copy link
Contributor Author

This is the same as #59 , I just rebased to resolve conflicts and squashed the commits down.

shikloshi pushed a commit that referenced this pull request Oct 14, 2015
Deterministic publish order for gradle and ivy (BI-204)
@shikloshi shikloshi merged commit 6db7945 into jfrog:master Oct 14, 2015
@shikloshi
Copy link
Contributor

Hi @arothian ,

Thank you for your contribution. This is now merged and will be part of build-info next release.

For keeping track on this issue please see Jira issue.

Aviad.

@arothian arothian deleted the deterministic_publish_order branch October 14, 2015 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants