Re: Ivy performance fix
Hi Josh, yes I think we can merge this in the 2.4.0 final release, but we'll have to review a bit in more detail first. However, could you create a JIRA issue containing a link to the patch? thanks! Maarten Van: Antoine Levy Lambert anto...@gmx.de Aan: Ant Developers List dev@ant.apache.org Verzonden: dinsdag 1 april 3:58 2014 Onderwerp: Re: Ivy performance fix Hello Josh, thanks for contacting the Ant/Ivy community. I think we can work with pull requests on github, as long as we see clearly the diffs with the code in the svn repository we should be fine. @Maarten, Nicolas, and others in the Ivy community, do you think we can merge this before building the ivy 2.4.0 final ? Regards, Antoine On Mar 31, 2014, at 4:15 PM, Josh Suereth joshua.suer...@gmail.com wrote: So, for the sbt project we noticed significant resolution time improvements with the following patch: https://github.com/sbt/ivy/pull/1 The reasoning: - A lot of artifacts being resolved use Maven's dependencyManagement conventions - Ivy appears to turn these into exact matcher rules - A ton of resolution time is spent filtering through these rules - The existing solution is O(n) for all overrides What the patch does: - Creates a key'd store for all exact matcher rules - When executing rules, ensure that we only traverse what we have to (non-exact, exact specific to our key and default). As I said, this represents a significant speed bump for sbt builds using Ivy. All existing tests pass, and I think they cover this aspect of ivy pretty well, from what I could see. What's the best mechanism to submit this back? Do you accept pull requests on github? Thanks! - Josh Suereth Tools Lead Typesafe, Inc. - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org
Re: Ivy performance fix
Done: https://issues.apache.org/jira/browse/IVY-1465 On Tue, Apr 1, 2014 at 3:40 PM, Josh Suereth joshua.suer...@gmail.comwrote: Can do. I'll have to find a jira account. On Tue, Apr 1, 2014 at 3:36 PM, Maarten Coene maarten_co...@yahoo.comwrote: Hi Josh, yes I think we can merge this in the 2.4.0 final release, but we'll have to review a bit in more detail first. However, could you create a JIRA issue containing a link to the patch? thanks! Maarten Van: Antoine Levy Lambert anto...@gmx.de Aan: Ant Developers List dev@ant.apache.org Verzonden: dinsdag 1 april 3:58 2014 Onderwerp: Re: Ivy performance fix Hello Josh, thanks for contacting the Ant/Ivy community. I think we can work with pull requests on github, as long as we see clearly the diffs with the code in the svn repository we should be fine. @Maarten, Nicolas, and others in the Ivy community, do you think we can merge this before building the ivy 2.4.0 final ? Regards, Antoine On Mar 31, 2014, at 4:15 PM, Josh Suereth joshua.suer...@gmail.com wrote: So, for the sbt project we noticed significant resolution time improvements with the following patch: https://github.com/sbt/ivy/pull/1 The reasoning: - A lot of artifacts being resolved use Maven's dependencyManagement conventions - Ivy appears to turn these into exact matcher rules - A ton of resolution time is spent filtering through these rules - The existing solution is O(n) for all overrides What the patch does: - Creates a key'd store for all exact matcher rules - When executing rules, ensure that we only traverse what we have to (non-exact, exact specific to our key and default). As I said, this represents a significant speed bump for sbt builds using Ivy. All existing tests pass, and I think they cover this aspect of ivy pretty well, from what I could see. What's the best mechanism to submit this back? Do you accept pull requests on github? Thanks! - Josh Suereth Tools Lead Typesafe, Inc. - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org
Re: Ivy performance fix
Thanks! I'll try to have a look at it soon. Maarten Van: Josh Suereth joshua.suer...@gmail.com Aan: Ant Developers List dev@ant.apache.org; Maarten Coene maarten_co...@yahoo.com Verzonden: dinsdag 1 april 21:47 2014 Onderwerp: Re: Ivy performance fix Done: https://issues.apache.org/jira/browse/IVY-1465 On Tue, Apr 1, 2014 at 3:40 PM, Josh Suereth joshua.suer...@gmail.com wrote: Can do. I'll have to find a jira account. On Tue, Apr 1, 2014 at 3:36 PM, Maarten Coene maarten_co...@yahoo.com wrote: Hi Josh, yes I think we can merge this in the 2.4.0 final release, but we'll have to review a bit in more detail first. However, could you create a JIRA issue containing a link to the patch? thanks! Maarten Van: Antoine Levy Lambert anto...@gmx.de Aan: Ant Developers List dev@ant.apache.org Verzonden: dinsdag 1 april 3:58 2014 Onderwerp: Re: Ivy performance fix Hello Josh, thanks for contacting the Ant/Ivy community. I think we can work with pull requests on github, as long as we see clearly the diffs with the code in the svn repository we should be fine. @Maarten, Nicolas, and others in the Ivy community, do you think we can merge this before building the ivy 2.4.0 final ? Regards, Antoine On Mar 31, 2014, at 4:15 PM, Josh Suereth joshua.suer...@gmail.com wrote: So, for the sbt project we noticed significant resolution time improvements with the following patch: https://github.com/sbt/ivy/pull/1 The reasoning: - A lot of artifacts being resolved use Maven's dependencyManagement conventions - Ivy appears to turn these into exact matcher rules - A ton of resolution time is spent filtering through these rules - The existing solution is O(n) for all overrides What the patch does: - Creates a key'd store for all exact matcher rules - When executing rules, ensure that we only traverse what we have to (non-exact, exact specific to our key and default). As I said, this represents a significant speed bump for sbt builds using Ivy. All existing tests pass, and I think they cover this aspect of ivy pretty well, from what I could see. What's the best mechanism to submit this back? Do you accept pull requests on github? Thanks! - Josh Suereth Tools Lead Typesafe, Inc. - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org
Ivy performance fix
So, for the sbt project we noticed significant resolution time improvements with the following patch: https://github.com/sbt/ivy/pull/1 The reasoning: - A lot of artifacts being resolved use Maven's dependencyManagement conventions - Ivy appears to turn these into exact matcher rules - A ton of resolution time is spent filtering through these rules - The existing solution is O(n) for all overrides What the patch does: - Creates a key'd store for all exact matcher rules - When executing rules, ensure that we only traverse what we have to (non-exact, exact specific to our key and default). As I said, this represents a significant speed bump for sbt builds using Ivy. All existing tests pass, and I think they cover this aspect of ivy pretty well, from what I could see. What's the best mechanism to submit this back? Do you accept pull requests on github? Thanks! - Josh Suereth Tools Lead Typesafe, Inc.
Re: Ivy performance fix
Hello Josh, thanks for contacting the Ant/Ivy community. I think we can work with pull requests on github, as long as we see clearly the diffs with the code in the svn repository we should be fine. @Maarten, Nicolas, and others in the Ivy community, do you think we can merge this before building the ivy 2.4.0 final ? Regards, Antoine On Mar 31, 2014, at 4:15 PM, Josh Suereth joshua.suer...@gmail.com wrote: So, for the sbt project we noticed significant resolution time improvements with the following patch: https://github.com/sbt/ivy/pull/1 The reasoning: - A lot of artifacts being resolved use Maven's dependencyManagement conventions - Ivy appears to turn these into exact matcher rules - A ton of resolution time is spent filtering through these rules - The existing solution is O(n) for all overrides What the patch does: - Creates a key'd store for all exact matcher rules - When executing rules, ensure that we only traverse what we have to (non-exact, exact specific to our key and default). As I said, this represents a significant speed bump for sbt builds using Ivy. All existing tests pass, and I think they cover this aspect of ivy pretty well, from what I could see. What's the best mechanism to submit this back? Do you accept pull requests on github? Thanks! - Josh Suereth Tools Lead Typesafe, Inc. - To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org For additional commands, e-mail: dev-h...@ant.apache.org