Re: WIP: BRIN multi-range indexes
Tomas Vondra <tomas.vondra@2ndquadrant.com>
From: Tomas Vondra <tomas.vondra@2ndquadrant.com>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: John Naylor <john.naylor@2ndquadrant.com>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2020-09-09T19:35:30Z
Lists: pgsql-hackers
On Wed, Sep 09, 2020 at 03:40:41PM -0300, Alvaro Herrera wrote: >On 2020-Sep-09, John Naylor wrote: > >> create index on t using brin (a); >> CREATE INDEX >> Time: 1631.452 ms (00:01.631) > >> create index on t using brin (a int8_minmax_multi_ops); >> CREATE INDEX >> Time: 6521.026 ms (00:06.521) > >It seems strange that the multi-minmax index takes so much longer to >build. I wonder if there's some obvious part of the algorithm that can >be improved? > There are some minor optimizations possible - for example I noticed we call minmax_multi_get_strategy_procinfo often because it happens in a loop, and we could easily do it just once. But that saves only about 10% or so, it's not a ground-breaking optimization. The main reason for the slowness is that we pass the values one by one to brin_minmax_multi_add_value - and on each call we need to deserialize (and then sometimes also serialize) the summary, which may be quite expensive. The regular minmax does not have this issue, it just swaps the Datum value and that's it. I see two possible optimizations - firstly, adding some sort of batch variant of the add_value function, which would get a bunch of values instead of just a single one, amortizing the serialization costs. Another option would be to teach add_value to keep the deserialized summary somewhere, and then force serialization at the end of the BRIN page range. The end result would be roughly the same, I think. regards -- Tomas Vondra http://www.2ndQuadrant.com PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
BRIN minmax-multi indexes
- ab596105b55f 14.0 landed
-
BRIN bloom indexes
- 77b88cd1bb90 14.0 landed
-
Support the old signature of BRIN consistent function
- a681e3c107aa 14.0 landed
-
Remove unnecessary pg_amproc BRIN minmax entries
- a68dfa27d42f 14.0 landed
-
Optimize allocations in bringetbitmap
- 8e4b332e88b8 14.0 landed
-
Move IS [NOT] NULL handling from BRIN support functions
- 72ccf55cb99c 14.0 landed
-
Pass all scan keys to BRIN consistent function at once
- a1c649d889bd 14.0 landed
-
Properly detoast data in brin_form_tuple
- d2d3a4bd33d2 9.5.24 landed
- bae31e75f777 9.6.20 landed
- 0b96fc977c5b 10.15 landed
- 895d0f0e8218 11.10 landed
- 8149e9f9a0d6 12.5 landed
- 6a7b55f3716f 13.1 landed
- 7577dd84807a 14.0 landed