[GitHub] [maven-resolver] caiwei-ebay2 commented on pull request #158: MRESOLVER-247: Introduce skipper to avoid unnecessary dependency reso…

[GitBox <gi...@apache.org>]

caiwei-ebay2 commented on PR #158:
URL: https://github.com/apache/maven-resolver/pull/158#issuecomment-1120358732

   @michael-o @cstamas @weibo1995 @ibabiankou 
   
   Echoed weibo1995's feedback in https://github.com/apache/maven-resolver/pull/172
   
   > At present, 5 or 6 long tail applications have been piloted, and the compilation time can be reduced by half, that is, about 100% performance gain. Later, we will use BF for more applications, and then feedback and communicate.
   thanks.
   
   Really glad that this patch could help more developers.
   
   Also I'm currently working on parallel pom downloading, the idea is a bit different than https://github.com/ibabiankou/maven-resolver/pull/2/files:
   
   1. only parallelize VersionRange and ArtifactDesciptor resolutioon part (poms & metadata.xml downloading), please check resolveArtifactDescriptorAsync for more details.
   
   Here is the commit for preview and still evolving, I will certify the approach by dry-running large numbers of apps in our company, please share your comments with the idea.
   https://github.com/parallel-pom-downloading/maven-resolver/commit/2c675527666f7bc0cec9543d7b3b1fd916c6fe0d
   
   Note: I'm resetting the 2FA auth of original "caiwei-ebay" account and unable to fire PR with "caiwei-ebay2" (getting message saying: **It looks like this is your first time opening a pull request in this project!**).  Will fire draft PR once 2FA auth is reset and we can discuss there later,
   
   


-- 
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