Re: Use T_IntList for uint32

On Mon, Aug 31, 2020 at 5:44 PM Ashutosh Bapat
<ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
>
> On Mon, Aug 31, 2020 at 4:59 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)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.