From: | John Naylor <john(dot)naylor(at)postgresql(dot)org> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Use bump context for TID bitmaps stored by vacuum |
Date: | 2024-04-08 07:44:28 |
Message-ID: | E1rtjg4-0015mL-8M@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Use bump context for TID bitmaps stored by vacuum
Vacuum does not pfree individual entries, and only frees the entire
storage space when finished with it. This allows using a bump context,
eliminating the chunk header in each leaf allocation. Most leaf
allocations will be 16 to 32 bytes, so that's a significant savings.
TidStoreCreateLocal gets a boolean parameter to indicate that the
created store is insert-only.
This requires a separate tree context for iteration, since we free
the iteration state after iteration completes.
Discussion: https://postgr.es/m/CANWCAZac%3DpBePg3rhX8nXkUuaLoiAJJLtmnCfZsPEAS4EtJ%3Dkg%40mail.gmail.com
Discussion: https://postgr.es/m/CANWCAZZQFfxvzO8yZHFWtQV+Z2gAMv1ku16Vu7KWmb5kZQyd1w@mail.gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/8a1b31e6e59631807a08a4e9465134c343bbdf5e
Modified Files
--------------
src/backend/access/common/tidstore.c | 15 +++++++++++++--
src/backend/access/heap/vacuumlazy.c | 4 ++--
src/include/access/tidstore.h | 2 +-
src/include/lib/radixtree.h | 11 ++++++++++-
src/test/modules/test_tidstore/test_tidstore.c | 3 ++-
5 files changed, 28 insertions(+), 7 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2024-04-08 08:03:59 | pgsql: Fix the intermittent buildfarm failures in 040_standby_failover_ |
Previous Message | Alexander Korotkov | 2024-04-08 07:36:15 | Re: pgsql: Transform OR clauses to ANY expression |