Re: FW: Query execution failure

Bruce Momjian <bruce@momjian.us>

View thread

From: Bruce Momjian <bruce@momjian.us>

To: Tom Lane <tgl@sss.pgh.pa.us>

Cc: Pete Storer <Pete.Storer@sas.com>, "pgsql-bugs@lists.postgresql.org" <pgsql-bugs@lists.postgresql.org>

Date: 2023-09-28T13:48:02Z

Lists: pgsql-bugs

Attachments

fdw.diff (text/x-diff) patch

On Wed, Sep 27, 2023 at 10:27:34PM -0400, Bruce Momjian wrote:
> On Wed, Sep 27, 2023 at 09:06:13PM -0400, Tom Lane wrote:
> > Bruce Momjian <bruce@momjian.us> writes:
> > > The attached patch documents this remote sort order requirement.
> > 
> > That seems like quite a random place to insert the documentation.
> > I doubt that the requirement applies only to upper paths -- FDWs
> > are allowed to return sorted paths for base relations too,
> > comparable to indexscan paths.
> 
> That is the only section of the docs that mentions sort pushdown.
> I don't see another reasonable location.

I now think the fdw docs are the wrong place for this since it is a user
requirement, not a developer one.  I put it in CREATE SERVER, patch
attached.

-- 
  Bruce Momjian  <bruce@momjian.us>        https://momjian.us
  EDB                                      https://enterprisedb.com

  Only you can decide what is important to you.