lisancao commented on PR #59416: URL: https://github.com/apache/spark/pull/59416#issuecomment-6210725664
Thanks for the correction. Creation now also analyzes the source (with its filter) on its own and passes the temporary functions recorded during that analysis (`AnalysisContext.referredTempFunctionNames`, the same record regular views use) to `verifyTemporaryObjectsNotExists`, through a small `Analyzer.referredTempFunctions`. A source, filter or filter subquery that calls one now fails with `INVALID_TEMP_OBJ_REFERENCE` on both catalog paths. Dimension and measure expressions aren't part of that analysis, so they can still use temporary functions; there's a test for that. The migration note now covers temporary functions, including existing metric views whose source calls one. -- 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]
