pgsql: Dissociate btequalimage() from interval_ops, ending its deduplic
От | Noah Misch |
---|---|
Тема | pgsql: Dissociate btequalimage() from interval_ops, ending its deduplic |
Дата | |
Msg-id | E1qroAX-000arw-0A@gemulon.postgresql.org обсуждение исходный текст |
Ответы |
Re: pgsql: Dissociate btequalimage() from interval_ops, ending its deduplic
|
Список | pgsql-committers |
Dissociate btequalimage() from interval_ops, ending its deduplication. Under interval_ops, some equal values are distinguishable. One such pair is '24:00:00' and '1 day'. With that being so, btequalimage() breaches the documented contract for the "equalimage" btree support function. This can cause incorrect results from index-only scans. Users should REINDEX any btree indexes having interval-type columns. After updating, pg_amcheck will report an error for almost all such indexes. This fix makes interval_ops simply omit the support function, like numeric_ops does. Back-pack to v13, where btequalimage() first appeared. In back branches, for the benefit of old catalog content, btequalimage() code will return false for type "interval". Going forward, back-branch initdb will include the catalog change. Reviewed by Peter Geoghegan. Discussion: https://postgr.es/m/20231011013317.22.nmisch@google.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/5f27b5f848a433ba54c521ccb889788b8f4d6ba7 Modified Files -------------- contrib/amcheck/verify_nbtree.c | 13 ++++++++++++- src/include/catalog/catversion.h | 2 +- src/include/catalog/pg_amproc.dat | 2 -- src/include/catalog/pg_opfamily.dat | 2 +- src/test/regress/expected/opr_sanity.out | 3 ++- 5 files changed, 16 insertions(+), 6 deletions(-)
В списке pgsql-committers по дате отправления: