You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "cstamas (via GitHub)" <gi...@apache.org> on 2023/01/31 18:20:28 UTC
[GitHub] [maven-resolver] cstamas opened a new pull request, #237: [MRESOLVER-32] Parallel deploy
cstamas opened a new pull request, #237:
URL: https://github.com/apache/maven-resolver/pull/237
Applied very same logic that download has, uses same pool as well.
---
https://issues.apache.org/jira/browse/MRESOLVER-32
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-resolver] cstamas commented on pull request #237: [MRESOLVER-32] Parallel deploy and more
Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas commented on PR #237:
URL: https://github.com/apache/maven-resolver/pull/237#issuecomment-1420774926
Closing this one out, this PR was split in 3:
* https://github.com/apache/maven-resolver/pull/238
* https://github.com/apache/maven-resolver/pull/239
* https://github.com/apache/maven-resolver/pull/240
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-resolver] michael-o commented on pull request #237: [MRESOLVER-32] Parallel deploy and more
Posted by "michael-o (via GitHub)" <gi...@apache.org>.
michael-o commented on PR #237:
URL: https://github.com/apache/maven-resolver/pull/237#issuecomment-1415763278
> Ok, will do it, but sequentially, as I don't see how to create these 3 PRs simultaneously without conflicts against each other, and I would not like to spend time on fixing conflicts after each merge, just for "purity". So, please then, be prepared for snappy reviews 😉
>
> In other words, having these 3 commits separate is okay, but will not allow their initial purpose (for example to roll back one of them) as all 3 are quite interleaving with each other.
Layered, sequentically, of course. Not in parallel.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-resolver] cstamas closed pull request #237: [MRESOLVER-32] Parallel deploy and more
Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas closed pull request #237: [MRESOLVER-32] Parallel deploy and more
URL: https://github.com/apache/maven-resolver/pull/237
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-resolver] michael-o commented on pull request #237: [MRESOLVER-32] Parallel deploy and more
Posted by "michael-o (via GitHub)" <gi...@apache.org>.
michael-o commented on PR #237:
URL: https://github.com/apache/maven-resolver/pull/237#issuecomment-1415763904
Why make parallel put an option? I mean if we do parallel downloads by default, why not do the same with uploads? Do you see a risk here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [maven-resolver] cstamas commented on pull request #237: [MRESOLVER-32] Parallel deploy and more
Posted by "cstamas (via GitHub)" <gi...@apache.org>.
cstamas commented on PR #237:
URL: https://github.com/apache/maven-resolver/pull/237#issuecomment-1413324838
Ok, will do it, but sequentially, as I don't see how to create these 3 PRs simultaneously without conflicts against each other, and I would not like to spend time on fixing conflicts after each merge, just for "purity". So, please then, be prepared for snappy reviews :wink:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org