-
- Downloads
[SPARK-14225][SQL] Cap the length of toCommentSafeString at 128 chars
## What changes were proposed in this pull request? Builds on https://github.com/apache/spark/pull/12022 and (a) appends "..." to truncated comment strings and (b) fixes indentation in lines after the commented strings if they happen to have a `(`, `{`, `)` or `}` ## How was this patch tested? Manually examined the generated code. Author: Sameer Agarwal <sameer@databricks.com> Closes #12044 from sameeragarwal/comment.
Showing
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeFormatter.scala 33 additions, 4 deletions...park/sql/catalyst/expressions/codegen/CodeFormatter.scala
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/package.scala 6 additions, 3 deletions...in/scala/org/apache/spark/sql/catalyst/util/package.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeFormatterSuite.scala 40 additions, 1 deletion...sql/catalyst/expressions/codegen/CodeFormatterSuite.scala
Loading
Please register or sign in to comment