You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by GitBox <gi...@apache.org> on 2021/03/06 16:19:20 UTC

[GitHub] [groovy] danielsun1106 opened a new pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

danielsun1106 opened a new pull request #1511:
URL: https://github.com/apache/groovy/pull/1511


   …ation for Gradle build scripts


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [groovy] danielsun1106 commented on pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

Posted by GitBox <gi...@apache.org>.
danielsun1106 commented on pull request #1511:
URL: https://github.com/apache/groovy/pull/1511#issuecomment-792047854


   I tested with "project0/build.gradle" file  locally, about 10% time saved.
   As for the goal, tune the performance as good as we could...


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [groovy] danielsun1106 commented on pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

Posted by GitBox <gi...@apache.org>.
danielsun1106 commented on pull request #1511:
URL: https://github.com/apache/groovy/pull/1511#issuecomment-797818936


   It is not expected... but parser suffers from ambiguities while choosing alternatives... Maybe we have to find better way to improve the parsing performance...


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [groovy] eric-milles commented on pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

Posted by GitBox <gi...@apache.org>.
eric-milles commented on pull request #1511:
URL: https://github.com/apache/groovy/pull/1511#issuecomment-792016064


   Are there any performance measurements for this change?  Nothing is mentioned in the linked issue about what is taking the time or how much improvement to expect.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [groovy] danielsun1106 closed pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

Posted by GitBox <gi...@apache.org>.
danielsun1106 closed pull request #1511:
URL: https://github.com/apache/groovy/pull/1511


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [groovy] paulk-asert commented on pull request #1511: GROOVY-9964: Groovy 3 compilation is much slower than Groovy 2 compil…

Posted by GitBox <gi...@apache.org>.
paulk-asert commented on pull request #1511:
URL: https://github.com/apache/groovy/pull/1511#issuecomment-797796447


   So, we have to eliminate the lambda chaining to gain the performance? I.e. GROOVY-8991 would be broken?


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org