Skip to content
Snippets Groups Projects
Commit c9ceb4f60256 authored by Pierre-Yves David's avatar Pierre-Yves David :octopus:
Browse files

phases: avoid N² behavior in `advanceboundary`

We allowed duplicated entries in the deque, which each entry could potentially
insert all its ancestors. So advancing boundary for the full repository would
mean each revision would walk all its ancestors, resulting in O(N²) iteration.
For repository of any decent size, N² is quickly insane.

We introduce a simple set to avoid this and get back to reasonable performance.
parent 6b2aeeec3ed0
No related branches found
No related tags found
2 merge requests!812Merge stable into default,!811phases: avoid N² behavior in `advanceboundary`
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment