HeartSaVioR commented on code in PR #36963:
URL: https://github.com/apache/spark/pull/36963#discussion_r905655365
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/trees/TreeNode.scala:
##########
@@ -868,6 +868,8 @@ abstract class TreeNode[BaseType <: TreeNode[BaseType]]
extends Product with Tre
case null => Nil
case None => Nil
case Some(null) => Nil
+ case Some(table: CatalogTable) =>
Review Comment:
This triggers test failures on SQLQueryTestSuite /
ThriftServerQueryTestSuite due to the change of representation of CatalogTable
wrapped with Some. While I think it "was" a bug or at least not intended
(because we care for CatalogTable not wrapped with Some), please let me know if
we want to retain the current.
--
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]
