[GitHub] [spark] yaooqinn commented on a diff in pull request #40922: [SPARK-43063][SQL][FOLLOWUP] Add ToPrettyString expression for Dataset.show

2023-04-27 Thread via GitHub
yaooqinn commented on code in PR #40922: URL: https://github.com/apache/spark/pull/40922#discussion_r1178728632 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/StringUtils.scala: ## @@ -67,6 +67,12 @@ object StringUtils extends Logging { "(?s)" + out.resul

[GitHub] [spark] yaooqinn commented on a diff in pull request #40922: [SPARK-43063][SQL][FOLLOWUP] Add ToPrettyString expression for Dataset.show

2023-04-26 Thread via GitHub
yaooqinn commented on code in PR #40922: URL: https://github.com/apache/spark/pull/40922#discussion_r1178621599 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyString.scala: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (

[GitHub] [spark] yaooqinn commented on a diff in pull request #40922: [SPARK-43063][SQL][FOLLOWUP] Add ToPrettyString expression for Dataset.show

2023-04-26 Thread via GitHub
yaooqinn commented on code in PR #40922: URL: https://github.com/apache/spark/pull/40922#discussion_r1178621599 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyString.scala: ## @@ -0,0 +1,76 @@ +/* + * Licensed to the Apache Software Foundation (