panbingkun commented on code in PR #43603: URL: https://github.com/apache/spark/pull/43603#discussion_r1382737960
########## sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/PlanParserSuite.scala: ########## @@ -1677,25 +1677,23 @@ class PlanParserSuite extends AnalysisTest { ScriptInputOutputSchema(List.empty, List.empty, None, None, List.empty, List.empty, None, None, false))) - // verify with ROW FORMAT DELIMETED + // verify with ROW FORMAT DELIMITED assertEqual( - """ - |SELECT TRANSFORM(a, b, c) - | ROW FORMAT DELIMITED - | FIELDS TERMINATED BY '\t' - | COLLECTION ITEMS TERMINATED BY '\u0002' - | MAP KEYS TERMINATED BY '\u0003' - | LINES TERMINATED BY '\n' - | NULL DEFINED AS 'null' - | USING 'cat' AS (a, b, c) - | ROW FORMAT DELIMITED - | FIELDS TERMINATED BY '\t' - | COLLECTION ITEMS TERMINATED BY '\u0004' - | MAP KEYS TERMINATED BY '\u0005' - | LINES TERMINATED BY '\n' - | NULL DEFINED AS 'NULL' - |FROM testData - """.stripMargin, + "SELECT TRANSFORM(a, b, c)\n" + Review Comment: Although this writing may seem ugly, it seems better than the previous version, haha -- 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-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org