New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
reduce the number of pruning operations #2881
base: main
Are you sure you want to change the base?
Conversation
Instead of checking a path against the entire list of paths every time, we only check against the list of already accepted paths.
|
✅ Deploy Preview for apollo-federation-docs canceled.
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
there's another potnetial optimization here: making federation/query-graphs-js/src/graphPath.ts Line 153 in 3f7392b
Since it is only used to be filled and the only part where it is read is in isOverridenBy`: federation/query-graphs-js/src/graphPath.ts Lines 770 to 777 in 3f7392b
|
I am not sure this PR is correct. In federation/query-graphs-js/src/graphPath.ts Lines 753 to 768 in 3f7392b
we add the |
Instead of checking a path against the entire list of paths every time, we only check against the list of already accepted paths.