Skip to content

Conversation

@pestretsov
Copy link
Contributor

Hi, PhpStorm here! We're working on improving indexing performance and saw an optimization opportunity. It's generally recommended to avoid using recursive visitors, since they traverse whole trees which might be very deep. In case of Symfony Plugin we can replace them with other techniques like traversing only needed elements and/or ControlFlow (which is a linear data structure and contains less nodes than trees)

This change results into at least 10% indexing speed up on our internal tests.

@pestretsov
Copy link
Contributor Author

cc @slapoguzov

@Haehnchen
Copy link
Owner

thank you, very welcome this changes to replace recursive visitors.

i was already trying to narrow the scope where ever possible. but its helps even more 👍

@Haehnchen Haehnchen merged commit 65d5fa1 into Haehnchen:master Mar 23, 2023
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