Re: BUG #13404: Docs do not mention "access/htup_details.h" for C functions using heap_form_tuple

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: christianr(at)fastmail(dot)com, pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #13404: Docs do not mention "access/htup_details.h" for C functions using heap_form_tuple
Date: 2015-06-05 22:36:05
Message-ID: 25573.1433543765@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> christianr(at)fastmail(dot)com wrote:
>> Is this a bug, or did I miss something?

> Hmm. So there are two ways to fix this bug. One is to #include
> "access/htup_details.h" in funcapi.h as you propose; the other is to
> change the docs to state that heap_form_tuple is in htup_details and you
> need to include that separately if you want it.

> I could go both ways, but seeing as we changed this in 9.3 and many
> people since then first complained and then tweaked their code to cope
> by #including htup_details.h, I think changing the docs is the reasonable
> option. You can write functions without using heap_form_tuple, after
> all.

Yeah. I don't think anyone's ever felt that that part of the
documentation was more authoritative than the code (especially not where
it's merely reporting an old and rather messy situation). And as you say,
if we were going to change the code, the time for that was 2 releases
back.

regards, tom lane

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Michael Paquier 2015-06-05 23:36:24 Re: BUG #13368: standby cluster immediately promotes after pg_basebackup from previously promoted master
Previous Message Alvaro Herrera 2015-06-05 21:50:22 Re: BUG #13404: Docs do not mention "access/htup_details.h" for C functions using heap_form_tuple