This is an automated email from the ASF dual-hosted git repository.
wenchen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 169346caa9b2 [SPARK-48578][SQL][FOLLOWUP] Fix `dev/scalastyle` error
for `ExpressionImplUtilsSuite`
169346caa9b2 is described below
commit 169346caa9b2f99813a76b8d57f5c296938b90b1
Author: panbingkun <[email protected]>
AuthorDate: Wed Jun 26 10:32:19 2024 +0800
[SPARK-48578][SQL][FOLLOWUP] Fix `dev/scalastyle` error for
`ExpressionImplUtilsSuite`
### What changes were proposed in this pull request?
The pr is following up https://github.com/apache/spark/pull/46845, to fix `
dev/scalastyle` check error.
### Why are the changes needed?
Make `sh dev/scalastyle` happy.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Manually test
```
sh dev/scakastyle
```
Before:
```
(base) ➜ spark-community git:(master) ✗ sh dev/scalastyle
-e Scalastyle checks failed at following occurrences:
[error]
/Users/panbingkun/Developer/spark/spark-community/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala:394:28:
nonascii.message
[error]
/Users/panbingkun/Developer/spark/spark-community/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala:396:28:
nonascii.message
[error] Total time: 23 s, completed Jun 26, 2024, 9:49:06 AM
```
After:
```
(base) ➜ spark-community git:(fix_scalastyle) ✗ sh dev/scalastyle
-e Scalastyle checks passed.
```
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #47094 from panbingkun/fix_scalastyle.
Authored-by: panbingkun <[email protected]>
Signed-off-by: Wenchen Fan <[email protected]>
---
.../spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala | 2 ++
1 file changed, 2 insertions(+)
diff --git
a/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala
b/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala
index f521cbcf2e0e..40e6182e587b 100644
---
a/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala
+++
b/sql/catalyst/src/test/java/org/apache/spark/sql/catalyst/expressions/ExpressionImplUtilsSuite.scala
@@ -390,10 +390,12 @@ class ExpressionImplUtilsSuite extends SparkFunSuite {
UTF8String.fromString("A"), except = false)
validateUTF8(UTF8String.fromBytes(Array[Byte](0x61)),
UTF8String.fromString("a"), except = false)
+ // scalastyle:off nonascii
validateUTF8(UTF8String.fromBytes(Array[Byte](0x80.toByte)),
UTF8String.fromString("\uFFFD"), except = true)
validateUTF8(UTF8String.fromBytes(Array[Byte](0xFF.toByte)),
UTF8String.fromString("\uFFFD"), except = true)
+ // scalastyle:on nonascii
}
test("TryValidate UTF8 string") {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
