Zhihong Yu <zyu@yugabyte.com> writes:
> I wonder if the (failed) assertion should be converted to an if statement:
As I said, I'm now thinking it's not the Assert that's faulty.
If I'm right about that, it's likely that the mistaken labeling
of these paths has other consequences beyond triggering this
assertion. (If it has none, I think we'd be better off to remove
these Path fields altogether, and re-identify the parent rels
here from the RelOptInfo data.)
regards, tom lane