This is an automated email from the ASF dual-hosted git repository.
zanmato pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git
The following commit(s) were added to refs/heads/main by this push:
new e12bc56c15 MINOR: [C++] Fix typo in partition sort comment (#45188)
e12bc56c15 is described below
commit e12bc56c158d97b72b18a4120e05feb2b0a79aff
Author: Rossi Sun <[email protected]>
AuthorDate: Tue Jan 7 18:43:32 2025 +0800
MINOR: [C++] Fix typo in partition sort comment (#45188)
### Rationale for this change
"prtn_ranges + 1" is apparently a typo in
https://github.com/apache/arrow/blob/f41f59066b79fbf59719e68ef0f908afd6c5218c/cpp/src/arrow/acero/partition_util.h#L40
### What changes are included in this PR?
Change to `num_prtns + 1`.
### Are these changes tested?
### Are there any user-facing changes?
Authored-by: Rossi Sun <[email protected]>
Signed-off-by: Rossi Sun <[email protected]>
---
cpp/src/arrow/acero/partition_util.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/src/arrow/acero/partition_util.h
b/cpp/src/arrow/acero/partition_util.h
index 42b21565fb..d02e9cb03f 100644
--- a/cpp/src/arrow/acero/partition_util.h
+++ b/cpp/src/arrow/acero/partition_util.h
@@ -37,7 +37,7 @@ class PartitionSort {
/// This corresponds to ranges in the sorted array containing all row ids for
/// each of the partitions.
///
- /// prtn_ranges must be initialized and have at least prtn_ranges + 1
elements
+ /// prtn_ranges must be initialized and have at least num_prtns + 1 elements
/// when this method returns prtn_ranges[i] will contains the total number of
/// elements in partitions 0 through i. prtn_ranges[0] will be 0.
///
