From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: tableam: Don't assume that every AM uses md.c style storage. |
Date: | 2019-05-18 02:10:46 |
Message-ID: | E1hRooA-0006mT-73@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
tableam: Don't assume that every AM uses md.c style storage.
Previously various parts of the code routed size requests through
RelationGetNumberOfBlocks[InFork]. That works if md.c is used by the
AM, but not otherwise.
Add a tableam callback to return the size of the table. As not every
AM will use postgres' BLCKSZ, have it return bytes, and have
RelationGetNumberOfBlocksInFork() round the byte size up into blocks.
To allow code outside of the AM to determine the actual relation size
map InvalidForkNumber the total size of a relation, as not every AM
might just need the postgres defined forks.
A few users of RelationGetNumberOfBlocks() ought to be converted away
from that. One case, the use of it to determine whether a tid is
valid, will be fixed in a follow up commit. Others will have to wait
for v13.
Author: Andres Freund
Discussion: https://postgr.es/m/20190423225201.3bbv6tbqzkb5w7cw@alap3.anarazel.de
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/7f44ede5941499c4cee13b812dd93335f4005095
Modified Files
--------------
src/backend/access/heap/heapam_handler.c | 27 ++++++++++++++++++++
src/backend/access/table/tableamapi.c | 3 +++
src/backend/storage/buffer/bufmgr.c | 43 +++++++++++++++++++++++++++++---
src/include/access/tableam.h | 40 +++++++++++++++++++++++++++++
4 files changed, 110 insertions(+), 3 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2019-05-18 04:40:51 | pgsql: "A void function may not return a value". |
Previous Message | Tom Lane | 2019-05-17 23:44:43 | pgsql: Restructure creation of run-time pruning steps. |