Skip to content

Commit ccc391d

Browse files
committed
housekeeping
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
1 parent 3a5dc76 commit ccc391d

2 files changed

Lines changed: 15 additions & 4 deletions

File tree

vortex-scan/public-api.lock

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ pub vortex_scan::api::ScanRequest::row_range: core::option::Option<core::ops::ra
1616

1717
pub vortex_scan::api::ScanRequest::selection: vortex_scan::Selection
1818

19+
pub vortex_scan::api::ScanRequest::target_output_bytes: core::option::Option<usize>
20+
21+
pub vortex_scan::api::ScanRequest::target_output_rows: core::option::Option<usize>
22+
1923
impl core::clone::Clone for vortex_scan::api::ScanRequest
2024

2125
pub fn vortex_scan::api::ScanRequest::clone(&self) -> vortex_scan::api::ScanRequest
@@ -28,6 +32,10 @@ impl core::fmt::Debug for vortex_scan::api::ScanRequest
2832

2933
pub fn vortex_scan::api::ScanRequest::fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result
3034

35+
pub const vortex_scan::api::DEFAULT_TARGET_OUTPUT_BYTES_HINT: usize
36+
37+
pub const vortex_scan::api::DEFAULT_TARGET_OUTPUT_ROWS_HINT: usize
38+
3139
pub trait vortex_scan::api::DataSource: 'static + core::marker::Send + core::marker::Sync
3240

3341
pub fn vortex_scan::api::DataSource::byte_size(&self) -> core::option::Option<vortex_array::expr::stats::precision::Precision<u64>>

vortex-scan/src/repeated_scan.rs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -354,8 +354,11 @@ struct StagedSplitStream<A: 'static + Send> {
354354
emit: EmitQueue<A>,
355355
}
356356

357+
type FilterTaskResult = (usize, usize, VortexResult<Option<FilteredSplit>>);
358+
type ProjectionTaskResult<A> = (usize, usize, usize, VortexResult<A>);
359+
357360
struct FilterQueue {
358-
in_flight: FuturesUnordered<Task<(usize, usize, VortexResult<Option<FilteredSplit>>)>>,
361+
in_flight: FuturesUnordered<Task<FilterTaskResult>>,
359362
ready: BTreeMap<usize, FilteredSplit>,
360363
ready_rows: usize,
361364
ready_projection_bytes: usize,
@@ -387,7 +390,7 @@ impl FilterQueue {
387390
}
388391

389392
struct ProjectionQueue<A: 'static + Send> {
390-
in_flight: FuturesUnordered<Task<(usize, usize, usize, VortexResult<A>)>>,
393+
in_flight: FuturesUnordered<Task<ProjectionTaskResult<A>>>,
391394
target_rows: usize,
392395
target_bytes: usize,
393396
row_cost_bytes: usize,
@@ -959,15 +962,15 @@ mod test {
959962
FilteredSplit {
960963
row_range: 0..10,
961964
mask: Mask::new_true(10),
962-
projection_ranges: vec![0..10],
965+
projection_ranges: std::iter::once(0..10).collect(),
963966
},
964967
),
965968
(
966969
1,
967970
FilteredSplit {
968971
row_range: 10..20,
969972
mask: Mask::new_true(10),
970-
projection_ranges: vec![10..20],
973+
projection_ranges: std::iter::once(10..20).collect(),
971974
},
972975
),
973976
]),

0 commit comments

Comments
 (0)