Chervontsev Mikhail
Chervontsev Mikhail
Nice change for 8.1 compatibility, can we have this merged please @makasim
It seems like we fixed requested changes, am i missing something?
Have this as some type of property test `RouteCollectorTest::testCreatingHttpRouteWithExistingPathShouldBeLinear` it took 320ms vs 5s 940ms with new and old implementation on my machine.
I extract duplicate checks and storage logic to separate class, it looks better to me.