ganeshashree commented on code in PR #58450:
URL: https://github.com/apache/spark/pull/58450#discussion_r3947010489


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala:
##########
@@ -2352,6 +2352,20 @@ class SessionCatalog(
     }
   }
 
+  /**
+   * Returns whether a temporary function is visible in the current resolution 
context, applying the
+   * same stored-view filtering as actual resolution ([[handleViewContext]]) 
but WITHOUT its
+   * side effect of recording the name as a referred temp function. Inside a 
stored view a temp
+   * function is visible only if the view captured it; outside a view this 
matches
+   * [[isTemporaryFunction]]. Ownership probes use this so they agree with the 
resolver on which
+   * routine owns a name inside a view.
+   */
+  def isTemporaryFunctionVisible(name: FunctionIdentifier): Boolean = {
+    isTemporaryFunction(name) &&

Review Comment:
   Done.



-- 
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]

Reply via email to