Skip to content

Commit b203c3d

Browse files
committed
Address review comments
1 parent f49c618 commit b203c3d

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

cpp/src/arrow/compute/kernels/vector_statistics.cc

+1
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ struct Winsorize {
112112
CallFunction("quantile", {input}, &quantile_options, ctx->exec_context()));
113113
auto quantile_array = quantile.array_as<ArrayType>();
114114
DCHECK_EQ(quantile_array->length(), 2);
115+
// The quantile function outputs either all nulls or no nulls at all.
115116
if (quantile_array->null_count() == 2) {
116117
return std::nullopt;
117118
}

docs/source/cpp/compute.rst

+2-2
Original file line numberDiff line numberDiff line change
@@ -1728,8 +1728,8 @@ overflow is detected.
17281728

17291729
* \(2) :member:`CumulativeOptions::start` is ignored.
17301730

1731-
Other numeric functions
1732-
~~~~~~~~~~~~~~~~~~~~~~~
1731+
Statistical functions
1732+
~~~~~~~~~~~~~~~~~~~~~
17331733

17341734
+-------------------------+-------+-------------+-------------+--------------------------------+-----------+
17351735
| Function name | Arity | Input types | Output type | Options class | Notes |

0 commit comments

Comments
 (0)