jayshrivastava commented on code in PR #26593:
URL: https://github.com/apache/datafusion/pull/26593#discussion_r4457135180
##########
datafusion/physical-expr/src/partitioning.rs:
##########
@@ -293,9 +318,58 @@ impl RangePartitioning {
ordering,
samples,
split_points,
+ start_bound: None,
+ end_bound: None,
})
}
+ /// Sets the inclusive start and exclusive end of the covered key space.
+ ///
+ /// `None` leaves that end unbounded. Bounds use the same lexicographic
order
+ /// as the split points, including descending and null ordering. Every
+ /// retained sample must lie between the bounds, inclusive. Equality
permits
+ /// empty first or last partitions; equal start and end bounds describe an
+ /// entirely empty range.
+ ///
+ /// This changes metadata only. The caller must ensure that every row falls
+ /// within the declared bounds and in the partition described by the split
points.
Review Comment:
Done
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
