Skip to content

Commit

Permalink
fix CR
Browse files Browse the repository at this point in the history
  • Loading branch information
jiacai2050 committed Mar 7, 2023
1 parent d9cfc2c commit 6d366f2
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
7 changes: 4 additions & 3 deletions analytic_engine/src/row_iter/chain.rs
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ impl<'a> Builder<'a> {
request_id: self.config.request_id,
schema: self.config.projected_schema.to_record_schema_with_key(),
streams,
_ssts: self.ssts,
ssts: self.ssts,
next_stream_idx: 0,
inited: false,
metrics: Metrics::new(self.memtables.len(), total_sst_streams),
Expand Down Expand Up @@ -226,7 +226,8 @@ pub struct ChainIterator {
schema: RecordSchemaWithKey,
streams: Vec<SequencedRecordBatchStream>,
/// ssts are kept here to avoid them from being purged.
_ssts: Vec<Vec<FileHandle>>,
#[allow(dead_code)]
ssts: Vec<Vec<FileHandle>>,
/// The range of the index is [0, streams.len()] and the iterator is
/// exhausted if it reaches `streams.len()`.
next_stream_idx: usize,
Expand Down Expand Up @@ -325,7 +326,7 @@ mod tests {
request_id: RequestId::next_id(),
schema: schema.to_record_schema_with_key(),
streams,
_ssts: Vec::new(),
ssts: Vec::new(),
next_stream_idx: 0,
inited: false,
metrics: Metrics::new(0, 0),
Expand Down
5 changes: 3 additions & 2 deletions analytic_engine/src/row_iter/merge.rs
Original file line number Diff line number Diff line change
Expand Up @@ -621,7 +621,8 @@ pub struct MergeIterator {
record_batch_builder: RecordBatchWithKeyBuilder,
origin_streams: Vec<SequencedRecordBatchStream>,
/// ssts are kept here to avoid them from being purged.
_ssts: Vec<Vec<FileHandle>>,
#[allow(dead_code)]
ssts: Vec<Vec<FileHandle>>,
/// Any [BufferedStream] in the hot heap is not empty.
hot: BinaryHeap<HeapBufferedStream>,
/// Any [BufferedStream] in the cold heap is not empty.
Expand Down Expand Up @@ -651,7 +652,7 @@ impl MergeIterator {
request_id,
inited: false,
schema,
_ssts: ssts,
ssts,
record_batch_builder,
origin_streams: streams,
hot: BinaryHeap::with_capacity(heap_cap),
Expand Down

0 comments on commit 6d366f2

Please sign in to comment.