This is a comment left during a code review.
Path: src/service/search/cache/cacher.rs
Line: 204:206

Comment:
**logic:** This fallback logic inside the loop will execute on every iteration. Move outside the loop to avoid redundant execution.

```suggestion
            // For histogram queries, if ORDER BY is not on the histogram/timestamp column,
        // we should still cache based on the histogram's inherent time ordering
        // Check if any order_by field matches the result_ts_col
        let mut found_ts_order = false;
        for (field, order) in &order_by {
            if is_timestamp_field(field, &result_ts_col) {
                is_descending = order == &OrderBy::Desc;
                found_ts_order = true;
                break;
            }
        }

        // For histogram queries ordered by non-timestamp columns (e.g., ORDER BY count),
        // use ascending as default
        if !found_ts_order && is_histogram_query {
            is_descending = false;
        }
```

How can I resolve this? If you propose a fix, please make it concise.

This is a comment left during a code review.
Path: src/service/search/cache/result_utils.rs
Line: 90:91

Comment:
**logic:** Calling `unwrap()` on `hits.first()` and `hits.last()` without checking if the slice is empty could panic. Consider handling the empty case explicitly.

```suggestion
        if hits.is_empty() {
            return (0, 0);
        }
        let first_ts = get_ts_value(ts_column, hits.first().unwrap());
        let last_ts = get_ts_value(ts_column, hits.last().unwrap());
```

**Context Used:** Context from `dashboard` - Avoid using `expect` with potentially failing operations; instead, handle the `None` case to prevent... ([source](https://app.greptile.com/review/custom-context?memory=b18dc58c-d445-4b6e-9438-bbf3b82397ca))

How can I resolve this? If you propose a fix, please make it concise.