| From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
|---|---|
| To: | Pg Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | list of TransactionIds |
| Date: | 2022-05-13 08:30:12 |
| Message-ID: | 202205130830.g5ntonhztspb@alvherre.pgsql |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
We didn't have any use of TransactionId as members of List, until
RelationSyncEntry->streamed_txns was introduced (464824323e57, pg14).
It's currently implemented as a list of int. This is not wrong at
present, but it may soon be, and I'm sure it rubs some people the wrong
way.
But is the rubbing way wrong enough to add support for TransactionId in
pg_list.h, including, say, T_XidList?
The minimal patch (attached) is quite small AFAICS.
--
Álvaro Herrera 48°01'N 7°57'E — https://www.EnterpriseDB.com/
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Implement-List-support-for-TransactionId.patch | text/x-diff | 6.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | houzj.fnst@fujitsu.com | 2022-05-13 08:48:33 | RE: Perform streaming logical transactions by background workers and parallel apply |
| Previous Message | Aleksander Alekseev | 2022-05-13 08:08:53 | Re: [PATCH] Compression dictionaries for JSONB |