Skip to content

refactor(query-engine): centralize range top-k maintenance - #636

Merged
milindsrivastava1997 merged 1 commit into
mainfrom
maintenance/topk-range-cleanup-634
Aug 26, 2026
Merged

refactor(query-engine): centralize range top-k maintenance#636
milindsrivastava1997 merged 1 commit into
mainfrom
maintenance/topk-range-cleanup-634

Conversation

@milindsrivastava1997

@milindsrivastava1997 milindsrivastava1997 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes #634

Validation

  • cargo test -p query_engine_rust --lib (537 passed, 1 ignored)
  • cargo clippy -p query_engine_rust --lib --tests -- -D warnings
  • cargo fmt --check

Out of scope

@milindsrivastava1997
milindsrivastava1997 force-pushed the maintenance/topk-range-cleanup-634 branch from 97ad202 to e7423b2 Compare August 26, 2026 19:35
@milindsrivastava1997
milindsrivastava1997 force-pushed the maintenance/topk-range-cleanup-634 branch from e7423b2 to cacefdc Compare August 26, 2026 20:52
@milindsrivastava1997
milindsrivastava1997 marked this pull request as ready for review August 26, 2026 20:55
@milindsrivastava1997
milindsrivastava1997 merged commit f9bd1be into main Aug 26, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR #629 code review findings

1 participant