jayzhan211 commented on PR #12864:
URL: https://github.com/apache/datafusion/pull/12864#issuecomment-2437492085
> > It seems the query is accidentally correct in before this change,
because we don't know the result of the function when we build up Values plan.
>
> Values plan is build for `VALUES ('abcd')` part of the query. The type is
known to be Utf8.
> The the surrounding query is planned, the `length(Utf8)` function is known
to return the length as a number.
>
> What was accidental about this?
>
>
I think mistakenly output the values on the wrong branch.
Let's find out such a valid query in postgres to make sure we need to
support this kind of query in datafusion. And add the test to ensure the
coverage.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
