public inbox for [email protected]  

help / color / mirror / Atom feed
From: David G. Johnston <[email protected]>
To: Tom Lane <[email protected]>
Cc: Maxwell Dreytser <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: RowDescription for a function does not include table OID
Date: Fri, 21 Jun 2024 08:57:49 -0700
Message-ID: <CAKFQuwZ+_Hmn2oq=DmKyi=h2RJdONLtqH_j_eUdH_oYML3idhA@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <LV3PR16MB6001AF4D05893E9FF4B3C746E6CE2@LV3PR16MB6001.namprd16.prod.outlook.com>
	<[email protected]>
	<LV3PR16MB6001CD31C5982777BA9BA25EE6CE2@LV3PR16MB6001.namprd16.prod.outlook.com>
	<LV3PR16MB6001216CE02B2D908F9D87ECE6C82@LV3PR16MB6001.namprd16.prod.outlook.com>
	<LV3PR16MB60014D610A726368182ACC7EE6C92@LV3PR16MB6001.namprd16.prod.outlook.com>
	<[email protected]>

On Fri, Jun 21, 2024 at 8:51 AM Tom Lane <[email protected]> wrote:

>
> The PG wire protocol specification [1] defines these fields thus:
>
>         If the field can be identified as a column of a specific
>         table, the object ID of the table; otherwise zero.
>
>         If the field can be identified as a column of a specific
>         table, the attribute number of the column; otherwise zero.
>
> My reading of that is that we should populate these fields only for
> the case of direct selection from a table.
>

s/can be identified as/is/g  ?

Experience shows people are inferring a lot from "can be identified" so we
should remove it.  "is" maybe over-simplifies a bit but in the correct
direction.

David J.


view thread (14+ messages)  latest in thread

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: RowDescription for a function does not include table OID
  In-Reply-To: <CAKFQuwZ+_Hmn2oq=DmKyi=h2RJdONLtqH_j_eUdH_oYML3idhA@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox