public inbox for [email protected]  
help / color / mirror / Atom feed
From: Tom Lane <[email protected]>
To: Richard Guo <[email protected]>
Cc: tender wang <[email protected]>
Cc: [email protected]
Subject: Re: wrong query result due to wang plan
Date: Wed, 22 Feb 2023 12:41:52 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAMbWs48rtnkepkgyvMzj3Cmuo9BcR3xNM2mpwdGHfO7AV-Azhg@mail.gmail.com>
References: <CAHewXNk9eJ35ru5xATWioTV4+xZPHptjy9etdcNPjUfY9RQ+uQ@mail.gmail.com>
	<CAMbWs48MiT0HNm9ng_DKdDEPDTC=Bk==aenkauoOp0-s8Fhy8w@mail.gmail.com>
	<CAMbWs49YpO08dwrvygjzm=do_=tojFcWapzEungp=132UW_hmg@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<CAMbWs48rtnkepkgyvMzj3Cmuo9BcR3xNM2mpwdGHfO7AV-Azhg@mail.gmail.com>

Richard Guo <[email protected]> writes:
> I'm thinking that maybe we can do the strip-the-outer-joins work only
> when it's not the top JoinDomain.  When we are in the top JoinDomain, it
> seems to me that it's safe to push the qual to the top of the tree.

Yeah, because there's nothing to commute with.  Might as well do that
for consistency with the distribute_qual_to_rels behavior, although
I felt it was better to put it inside get_join_domain_min_rels.

Pushed, thanks for the report!

			regards, tom lane






view thread (6+ messages)

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]
  Subject: Re: wrong query result due to wang plan
  In-Reply-To: <[email protected]>

* 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