On Mon, Aug 31, 2020 at 5:44 PM Ashutosh Bapat <ashutosh.bapat....@gmail.com> wrote: > > On Mon, Aug 31, 2020 at 4:59 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > > Now, as far as I can see there is no problem in using T_IntList in > > such usage because we are not going to fetch stored unsigned value as > > a signed value, so the comparison in get_schema_sent_in_streamed_txn > > should work well. However, still, I thought it would be better if > > there is a built-in T_UIntList. > > > > Thoughts? > > May be we should have separate list APIs for XID like OID, in case we > change underlying datatype of XID in future (unlikely but we have had > discussion about 64bit XIDs in the past). Apart from that it helps us > track code which deals with XID lists. >
This is a valid point but I think for now I will go with Tom's suggestion as the demand for this seems low at this stage. I don't want to introduce a new set of APIs just for one use case especially when we can work with existing APIs. -- With Regards, Amit Kapila.
