On Tue, Sep 16, 2025 at 10:05 AM Richard Guo <guofenglinux@gmail.com> wrote:
> There may be other functions that should be treated as non-strict but
> currently aren't. For now, I suggest we address JsonConstructorExpr
> first. If we encounter more such cases later, they should be
> straightforward to fix as well.
I don't like the test cases in the proposed patch. First, the test
output is excessively large, resulting in a patch with over 30,000
insertions. I don't think it's a good idea to output the entire tenk1
table. Second, I don't think it's useful to test json_arrayagg or
json_objectagg. As I mentioned earlier, these are aggregates, so the
subquery wouldn't be pulled up in the first place.
I've rewritten the test cases, added a commit message, and pushed the
patch. It has also been back-patched to v16.
- Richard