szehon-ho commented on PR #13106: URL: https://github.com/apache/iceberg/pull/13106#issuecomment-3046284271
I guess Spark just needs all Expression to mark whether they are deterministic , for things like knowing whether you can use them in various places like filter aggregate, merge condition, default value, etc. not sure they all apply here :) but it makes sense for now that procedure output for iceberg is not deterinistic as it doesnt make sense to use the output like that. As there's a lot of approvals here, I can wait a bit today if any more comment and merge if not to unblock 1.10 release -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
