HeartSaVioR commented on code in PR #37042:
URL: https://github.com/apache/spark/pull/37042#discussion_r911921309
##########
sql/core/src/test/scala/org/apache/spark/sql/SQLQueryTestHelper.scala:
##########
@@ -44,6 +44,13 @@ trait SQLQueryTestHelper {
.replaceAll("Last Access.*", s"Last Access $notIncludedMsg")
.replaceAll("Partition Statistics\t\\d+", s"Partition
Statistics\t$notIncludedMsg")
.replaceAll("\\*\\(\\d+\\) ", "*") // remove the WholeStageCodegen
codegenStageIds
+
+ // Below is needed since the catalog table in LogicalRelation can
produce serde class
+ // "optionally" if CatalogTable is presented and has a serde information
in the storage.
+ // This assumes LogicalRelation contains a catalog table, otherwise it
would not match with
+ // this pattern.
+ .replaceAll("Arguments: (.+), (\\[.+\\]), (`.+`\\.`.+`)(, .+)?,
(false|true)",
Review Comment:
I just realized the change of 3rd argument is required due to the merge of
another PR. I'll try to make sure it works with 2+ level.
I'm going to try applying stricter rule with 2rd argument as well.
--
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]
