-
Notifications
You must be signed in to change notification settings - Fork 576
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
[WIP] Revert #1344 and use in place index #1355
base: master
Are you sure you want to change the base?
[WIP] Revert #1344 and use in place index #1355
Conversation
Results evaluating commit a3cc4b4 (merged with base 5add9a5 as 2d7351f). See run details. PostgreSQL DB size in MB: 2954 ⇒ 2952 (-0.1% change)
expand for details...
|
HI @frodrigo, |
Hi @frodrigo, Master
This PR
|
9152088
to
a3cc4b4
Compare
#1344 try to fix transportation name update issue taking too long.
But it does not address the usage of the index.
Following tests on 1000 random highway in France.
Before #1344
Time: too long
Now, with #1344
It is not so much better on explain but execution time is.
Time: 28266.749 ms
Insert
Time: too long
Actual plus new index
Time: 7385.498 ms
Insert
Time: 106943.798 ms
Fixed query using existing index
The existing Index
Time: 7186.550 ms
Insert
The existing Index
Time: 95274.221 ms
This PR is a bit faster than fixing index of current code.
cc @lazaa32 @nyurik @ZeLonewolf