[GitHub] [spark] dtenedor commented on pull request #41549: [SPARK-43529][SQL][FOLLOWUP] Code cleanup in UnresolvedTableSpec and related plans

["dtenedor (via GitHub)" <gi...@apache.org>]

dtenedor commented on PR #41549:
URL: https://github.com/apache/spark/pull/41549#issuecomment-1587878086

   > Make UnresolvedTableSpec a UnaryExpression, so that transforming with mapExpressions will transform it and the option list expressions in its child
   
   @gengliangwang nice idea on this part, now we can keep the table options expressions in the `UnresolvedTableSpec` directly and the analyzer will recursive into them without any complex support for matching against it in `mapExpressions` and `treePatternBits`.


-- 
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: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org