On Fri, Sep 23, 2016 at 12:04 AM, Tom Lane <t...@sss.pgh.pa.us> wrote:
> I thought removing the comment altogether was not appropriate, because
> it remains true that you want to work really hard to ensure that
> sizeof(ItemPointerData) is 6. We're just giving up on pretense of support
> for compilers that don't believe that
> . I'm half tempted to introduce a
> StaticAssert about it, but refrained for the moment.
I also thought about that and it probably makes sense, at least to see how
buildfarm behaves. One reason to do so is that I did not find any
discussion or evidence of why SizeOfIptrData magic is no longer necessary
and to see if it was an unintentional change at some point.
> > While htup.h refactoring happened in 9.5, I don't see any point in back
> > patching this.
> Agreed. Pushed to HEAD only.
Pavan Deolasee http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services