Re: list of TransactionIds

Alvaro Herrera <alvherre@alvh.no-ip.org>

From: Alvaro Herrera <alvherre@alvh.no-ip.org>
To: "houzj.fnst@fujitsu.com" <houzj.fnst@fujitsu.com>
Cc: Pg Hackers <pgsql-hackers@lists.postgresql.org>, Amit Kapila <amit.kapila16@gmail.com>
Date: 2022-10-20T07:43:55Z
Lists: pgsql-hackers
Hello

On 2022-Oct-20, houzj.fnst@fujitsu.com wrote:

> While trying to use the newly introduced list_member_xid(), I noticed that it
> internally use lfirst_oid instead of lfirst_xid. It works ok for now. Just in
> case we change xid to 64 bits in the future, I think we’d better use lfirst_xid
> here.

Egad.

> Here is a tiny patch to fix that.

Pushed, thanks.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
"I can't go to a restaurant and order food because I keep looking at the
fonts on the menu.  Five minutes later I realize that it's also talking
about food" (Donald Knuth)



Commits

  1. Use proper macro to access TransactionId

  2. Implement List support for TransactionId