wiedld commented on code in PR #7379:
URL: https://github.com/apache/arrow-datafusion/pull/7379#discussion_r1302264850
##########
datafusion/core/src/physical_plan/sorts/cursor.rs:
##########
@@ -99,6 +100,16 @@ pub trait Cursor: Ord {
/// Advance the cursor, returning the previous row index
fn advance(&mut self) -> usize;
+
+ /// Go to reference row, returning the previous row index
+ fn seek(&mut self, row: usize) -> usize;
Review Comment:
Seek was added in anticipation of the TODO/planned optimizations.
a.k.a. cycles of slicing and concating of normalized key rows, also matching
the cascaded sort recommendations in Graefe 2006.
--
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]
