Ayush Tiwari <[email protected]> writes:
> While looking at 1999fe227df, I noticed that the new Assert(refname)
> in get_variable() can be hit through a rule (v v edge case, fuzz testing):

Ah, I wondered if those Asserts were too aggressive.

> The attached patch just keeps the rule's alias in that case, and
> EXPLAIN then shows "Output: n.a".  Thoughts?

We could just drop the Asserts and thereby return to the previous
behavior.  But perhaps your proposal is better.  Dean, any thoughts?

                        regards, tom lane


Reply via email to