Skip to content

Commit fc4b029

Browse files
committed
checkstyle fixes
1 parent 7c41261 commit fc4b029

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

native/spark-expr/src/string_funcs/split.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,9 @@ pub fn spark_split(args: &[ColumnarValue]) -> DataFusionResult<ColumnarValue> {
9797
let string_array = GenericStringArray::<i32>::from(result);
9898
let list_array = create_list_array(Arc::new(string_array));
9999

100-
Ok(ColumnarValue::Scalar(ScalarValue::List(Arc::new(list_array))))
100+
Ok(ColumnarValue::Scalar(ScalarValue::List(Arc::new(
101+
list_array,
102+
))))
101103
}
102104
_ => exec_err!("split expects (array, scalar) or (scalar, scalar) arguments"),
103105
}

spark/src/test/scala/org/apache/comet/CometStringExpressionSuite.scala

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ class CometStringExpressionSuite extends CometTestBase {
159159

160160
test("split string with limit") {
161161
// Split tests with 3 arguments (with limit)
162-
withParquetTable((0 until 5).map(i => (s"a,b,c,d,e", i)), "tbl") {
162+
withParquetTable((0 until 5).map(i => ("a,b,c,d,e", i)), "tbl") {
163163
checkSparkAnswerAndOperator("SELECT split(_1, ',', 2) FROM tbl")
164164
checkSparkAnswerAndOperator("SELECT split(_1, ',', 3) FROM tbl")
165165
checkSparkAnswerAndOperator("SELECT split(_1, ',', -1) FROM tbl")
@@ -169,12 +169,12 @@ class CometStringExpressionSuite extends CometTestBase {
169169

170170
test("split string with regex patterns") {
171171
// Test with various regex patterns
172-
withParquetTable((0 until 5).map(i => (s"word1 word2 word3", i)), "tbl") {
172+
withParquetTable((0 until 5).map(i => ("word1 word2 word3", i)), "tbl") {
173173
checkSparkAnswerAndOperator("SELECT split(_1, ' ') FROM tbl")
174174
checkSparkAnswerAndOperator("SELECT split(_1, '\\\\s+') FROM tbl")
175175
}
176176

177-
withParquetTable((0 until 5).map(i => (s"foo123bar456baz", i)), "tbl2") {
177+
withParquetTable((0 until 5).map(i => ("foo123bar456baz", i)), "tbl2") {
178178
checkSparkAnswerAndOperator("SELECT split(_1, '\\\\d+') FROM tbl2")
179179
}
180180
}

0 commit comments

Comments
 (0)