srielau commented on code in PR #53047: URL: https://github.com/apache/spark/pull/53047#discussion_r2525507025
########## docs/sql-ref-identifier-clause.md: ########## @@ -117,18 +145,18 @@ DECLARE mytab = 'tab1'; -- Creation of a table using variable. CREATE TABLE IDENTIFIER(mytab)(c1 INT); -DESCRIBE IDENTIFIER(mytab); +EXECUTE IMMEDIATE 'DESCRIBE IDENTIFIER(:mytab)' USING mytab; Review Comment: The result is identical. We are exchanging breath of scope for IDENTIFIER() against capability. with mytab being a session variable we need to fall back to the "old" way of using IDENTIFIER(). Since I labeled that way as beeing not recommended I wanted to flip the examples to the new more general way. I'm having regrets with the original implementation. It is way to complex and narrow. -- 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]
