Skip to content

Conversation

@ChuckHend
Copy link
Owner

@ChuckHend ChuckHend commented Nov 16, 2025

  • remove redundant operations in query to improve query perf
  • add standalone worker binary to build for scaling out worker count
  • fix delete job bug (wrong pgmq table name)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR improves search query performance, adds a standalone worker binary for scaling, and fixes a bug in the delete job functionality.

  • Optimized hybrid search query by removing unused window function calculations (COUNT(*) OVER ())
  • Added standalone vectorize-worker binary to enable independent worker scaling
  • Fixed PGMQ table name in job cleanup from pgmq.vectorize_jobs to pgmq.q_vectorize_jobs

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
core/src/query.rs Simplified hybrid search query by removing redundant COUNT(*) OVER () calculations and replacing CASE WHEN with COALESCE for cleaner null handling
core/src/init.rs Fixed PGMQ table name in cleanup_job to use the correct q_vectorize_jobs table
server/Cargo.toml Added binary definition for the new standalone worker
server/Dockerfile Added build and copy steps for the vectorize-worker binary to enable separate deployment
server/tests/tests.rs Refactored test to use common helper function for table creation, reducing code duplication

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ChuckHend ChuckHend merged commit 9bb04b2 into main Nov 16, 2025
11 checks passed
@ChuckHend ChuckHend deleted the query-opt branch November 16, 2025 16:08
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.

2 participants