dbatomic commented on PR #44316:
URL: https://github.com/apache/spark/pull/44316#issuecomment-1858208985
> I agree with the semantic changes. General comment, though: VALUES in its
current implementation is much too restrictive. Will this PR move us closer to
allowing arbitrary expressions (including non determinism and correlation)?
E.g. the following cannot be done in VALUES today: scala> spark.sql("VALUES
(rand())").show() org.apache.spark.sql.AnalysisException:
[INVALID_INLINE_TABLE.CANNOT_EVALUATE_EXPRESSION_IN_INLINE_TABLE] Invalid
inline table. Cannot evaluate the expression "rand()" in inline table
definition. SQLSTATE: 42000; line 1 pos 8
>
> or: SELECT pk, c FROM T, LATERAL(VALUES(T.c1), (T.c2)) AS unpivot(c)
Yeah, I maybe can use this PR to deal with rand() problem as well (i.e.
allow non-deterministic expressions in VALUES), because it is rather similar to
CURRENT_* issue.
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
