| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Greg Burd <greg(at)burd(dot)me> |
| Cc: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Support for 8-byte TOAST values, round two |
| Date: | 2026-09-10 07:51:54 |
| Message-ID: | aqJhmsEEiAUKQFXN@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Sep 10, 2026 at 08:18:57AM +0900, Michael Paquier wrote:
> Hmm. Okay. I'll think about something here.
Attached is a rebased v16 following cb298616463d. The patch set has
been slightly re-organized, with the pieces that seem worth
introducing first on top of the rest:
- 0001 for binary upgrades.
- 0002 for the reloption toast_value_type
- 0003 for adding more Oid8 in the TOAST APIs compared to Oid.
- 0004 for pg_column_toast_chunk_id() switched to Oid8
- 0005 for adding toast_value_type=oid8, with all the TOAST changes
not related to the new vartag.
- 0006 is the battery of tests for all the cases found during
development, grouped together: core things, amcheck, test_decoding..
- 0007 adds the new vartag_external.
- 0008 is the tweak for toast_tuple_find_biggest_attribute(), which
should perhaps be merged with 0007.
Most of the previous comments were related 0002 and the new reloption
(double whitespace, doc improvement regarding no effects after
initial creation of the TOAST table, parens in the defaultarg of
rel.h. regression tests). I've addressed all of them, as so as whole
0002 feels kind of OK as-is.
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| v16-0001-Add-support-for-TOAST-chunk_id-type-in-binary-up.patch | text/plain | 9.3 KB |
| v16-0002-Add-relation-option-toast_value_type.patch | text/plain | 9.9 KB |
| v16-0003-Refactor-some-TOAST-value-ID-code-to-use-Oid8-in.patch | text/plain | 16.1 KB |
| v16-0004-Switch-pg_column_toast_chunk_id-return-value-fro.patch | text/plain | 5.7 KB |
| v16-0005-Add-support-for-oid8-TOAST-values.patch | text/plain | 20.3 KB |
| v16-0006-Add-battery-of-tests-related-oid8.patch | text/plain | 37.4 KB |
| v16-0007-Add-support-for-TOAST-pointers-as-oid8.patch | text/plain | 47.4 KB |
| v16-0008-Fix-toast_tuple_find_biggest_attribute-for-OID8-.patch | text/plain | 6.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Chao Li | 2026-09-10 07:53:45 | Re: Review items for EXCEPT TABLE publication |
| Previous Message | Chao Li | 2026-09-10 07:36:18 | Re: Review items for EXCEPT TABLE publication |