Skip to content

Commit 1ebc0ba

Browse files
committed
more zombie code
1 parent 5aea315 commit 1ebc0ba

File tree

1 file changed

+6
-14
lines changed

1 file changed

+6
-14
lines changed

crates/store/re_dataframe/src/query.rs

Lines changed: 6 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -175,19 +175,13 @@ impl QueryHandle<'_> {
175175
metadata: Default::default(),
176176
};
177177

178-
let using_index_values_stack = self
179-
.query
180-
.using_index_values
181-
.as_ref()
182-
.map(|values| values.iter().rev().copied().collect_vec());
183-
184178
// 4. Perform the query and keep track of all the relevant chunks.
185179
let query = {
186180
let index_range =
187-
if let Some(using_index_values_stack) = using_index_values_stack.as_ref() {
188-
using_index_values_stack
189-
.last() // reminder: it's reversed (stack)
190-
.and_then(|start| using_index_values_stack.first().map(|end| (start, end)))
181+
if let Some(using_index_values) = self.query.using_index_values.as_ref() {
182+
using_index_values
183+
.first()
184+
.and_then(|start| using_index_values.last().map(|end| (start, end)))
191185
.map_or(ResolvedTimeRange::EMPTY, |(start, end)| {
192186
ResolvedTimeRange::new(*start, *end)
193187
})
@@ -266,10 +260,8 @@ impl QueryHandle<'_> {
266260
//
267261
// Used to achieve ~O(log(n)) pagination.
268262
let unique_index_values =
269-
if let Some(using_index_values_stack) = using_index_values_stack.as_ref() {
270-
let mut all_unique_index_values = using_index_values_stack.clone();
271-
all_unique_index_values.sort();
272-
all_unique_index_values
263+
if let Some(using_index_values) = self.query.using_index_values.as_ref() {
264+
using_index_values.iter().copied().collect_vec()
273265
} else {
274266
re_tracing::profile_scope!("index_values");
275267

0 commit comments

Comments
 (0)