Skip to content

Commit 30c3356

Browse files
authored
chore: remove redundant range_offset in StructuralPrimitiveFieldSchedulingJob (#3645)
The `range_offset` is ineffective as it's always 0. The offset within current range is handled by `global_row_offset`.
1 parent e51b4be commit 30c3356

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

rust/lance-encoding/src/encodings/logical/primitive.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2657,7 +2657,6 @@ struct StructuralPrimitiveFieldSchedulingJob<'a> {
26572657
ranges: Vec<Range<u64>>,
26582658
page_idx: usize,
26592659
range_idx: usize,
2660-
range_offset: u64,
26612660
global_row_offset: u64,
26622661
}
26632662

@@ -2668,7 +2667,6 @@ impl<'a> StructuralPrimitiveFieldSchedulingJob<'a> {
26682667
ranges,
26692668
page_idx: 0,
26702669
range_idx: 0,
2671-
range_offset: 0,
26722670
global_row_offset: 0,
26732671
}
26742672
}
@@ -2684,7 +2682,6 @@ impl StructuralSchedulingJob for StructuralPrimitiveFieldSchedulingJob<'_> {
26842682
}
26852683
// Get our current range
26862684
let mut range = self.ranges[self.range_idx].clone();
2687-
range.start += self.range_offset;
26882685
let priority = range.start;
26892686

26902687
let mut cur_page = &self.scheduler.page_schedulers[self.page_idx];

0 commit comments

Comments
 (0)