Skip to content

Commit

Permalink
MINOR: [C++] Fix typo in partition sort comment (#45188)
Browse files Browse the repository at this point in the history
### 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]>
  • Loading branch information
zanmato1984 authored Jan 7, 2025
1 parent f41f590 commit e12bc56
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/src/arrow/acero/partition_util.h
Original file line number Diff line number Diff line change
Expand Up @@ -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.
///
Expand Down

0 comments on commit e12bc56

Please sign in to comment.