Reranko05 commented on PR #50642:
URL: https://github.com/apache/arrow/pull/50642#issuecomment-5085624807
I re-ran it locally and realized the reported formatting issue isn't related
to the changes in this PR. It looks like it's a pre-existing issue elsewhere in
the file. Sorry for the confusion.
```diff
diff --git a/cpp/src/arrow/compute/kernels/scalar_round_arithmetic_test.cc
b/cpp/src/arrow/compute/kernels/scalar_round_arithmetic_test.cc
index da0026d011..1b22d40e2c 100644
--- a/cpp/src/arrow/compute/kernels/scalar_round_arithmetic_test.cc
+++ b/cpp/src/arrow/compute/kernels/scalar_round_arithmetic_test.cc
@@ -930,18 +930,20 @@ TEST_F(TestUnaryRoundArithmeticDecimal,
RoundToMultipleHalfToOdd) {
// There is no exact halfway point, check what happens
set_multiple(ty, 25);
CheckScalar(func, {ArrayFromJSON(ty, R"([])")}, ArrayFromJSON(ty,
R"([])"), &options);
- CheckScalar(func, {values},
- ArrayFromJSON(ty, R"(["-0.50", "-0.25", "-0.25", "-0.25",
"-0.00", "0.00",
+ CheckScalar(
+ func, {values},
+ ArrayFromJSON(ty, R"(["-0.50", "-0.25", "-0.25", "-0.25"
, "-0.00", "0.00",
"0.00", "0.25", "0.25", "0.25", "0
.50", null])"),
- &options);
+ &options);
set_multiple(ty, 1);
CheckScalar(func, {values}, values, &options);
set_multiple(ty, 24);
CheckScalar(func, {ArrayFromJSON(ty, R"([])")}, ArrayFromJSO
N(ty, R"([])"), &options);
- CheckScalar(func, {values},
- ArrayFromJSON(ty, R"(["-0.48", "-0.48", "-0.24",
"-0.24", "-0.24", "0.00",
+ CheckScalar(
+ func, {values},
+ ArrayFromJSON(ty, R"(["-0.48", "-0.48", "-0.24", "-0.24"
, "-0.24", "0.00",
"0.24", "0.24", "0.24", "0.48", "0
.48", null])"),
- &options);
+ &options);
set_multiple(decimal128(3, 1), 1);
CheckScalar(
func, {values},
(END)
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
