From: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
---|---|
To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: File API cleanup |
Date: | 2023-02-20 09:52:14 |
Message-ID: | 9b7658c6-66de-b636-ec97-e6f6e7470677@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 23.12.22 09:33, Peter Eisentraut wrote:
> On 01.12.22 09:25, Peter Eisentraut wrote:
>> Here are a couple of patches that clean up the internal File API and
>> related things a bit:
>
> Here are two follow-up patches that clean up some stuff related to the
> earlier patch set. I suspect these are all historically related.
Another patch under this theme. Here, I'm addressing the smgr API,
which effectively sits one level above the previously-dealt with "File" API.
Specifically, I'm changing the data buffer to void *, from char *, and
adding const where appropriate. As you can see in the patch, most
callers were unhappy with the previous arrangement and required casts.
(I pondered whether "Page" might be the right data type instead, since
the writers all write values of that type. But the readers don't read
into pages directly. So "Block" seemed more appropriate, and Block is
void * (bufmgr.h), so this makes sense.)
Attachment | Content-Type | Size |
---|---|---|
0001-Update-types-in-smgr-API.patch | text/plain | 11.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Komal Habura | 2023-02-20 09:53:46 | Seek for helper documents to implement WAL with an FDW |
Previous Message | David Rowley | 2023-02-20 09:51:40 | Re: Introduce list_reverse() to make lcons() usage less inefficient |