cloud-fan commented on pull request #28840: URL: https://github.com/apache/spark/pull/28840#issuecomment-648134509
I spent more time thinking about this and how to move forward with the new DDL/DML command resolution framework. Some facts: 1. there is no function API in the v2 catalog, and there is no one working on it yet. For function related commands, we have to use the old framework. 2. we have to follow the general name resolution rule, which considers the catalog name. I think it makes sense to have `RefreshFunction` as the entry plan, and `RefreshFunctionCommand` as the v1 fallback. For `UnresolvedFunc` and `ResolvedFunc`, there are 2 options: 1. follow `ResolvedView` and just put an `Identifier`. 2. put all the necessary function info in the `ResolvedFunc`. Since we have other function-related v1 commands, maybe the first option is better? cc @imback82 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
