Skip to content

Remove eager duplicate matching from jobs list responses#358

Merged
DaKheera47 merged 2 commits intomainfrom
fix-duplicate-job-performance
Apr 13, 2026
Merged

Remove eager duplicate matching from jobs list responses#358
DaKheera47 merged 2 commits intomainfrom
fix-duplicate-job-performance

Conversation

@DaKheera47
Copy link
Copy Markdown
Owner

Summary

  • stop attaching appliedDuplicateMatch on GET /api/jobs list and full responses to remove the hot-path CPU bottleneck
  • keep duplicate match hydration on job detail responses and make the client/shared types tolerate omitted list metadata
  • add benchmark logging plus a dedicated benchmark:jobs script to measure route phases against a DB snapshot
  • precompute normalized duplicate-match inputs so remaining detail-path matching work is cheaper

Testing

  • ./orchestrator/node_modules/.bin/biome ci .
  • npm run check:types:shared
  • npm --workspace orchestrator run check:types
  • npm --workspace gradcracker-extractor run check:types
  • npm --workspace ukvisajobs-extractor run check:types
  • npm --workspace orchestrator run build:client
  • npm --workspace orchestrator run test:run
  • npm --workspace orchestrator run benchmark:jobs

@DaKheera47 DaKheera47 marked this pull request as ready for review April 13, 2026 21:36
@DaKheera47 DaKheera47 merged commit 89c1f2e into main Apr 13, 2026
10 checks passed
@DaKheera47 DaKheera47 deleted the fix-duplicate-job-performance branch April 13, 2026 21:37
therealsamyak pushed a commit to therealsamyak/job-ops that referenced this pull request Apr 14, 2026
)

* Remove duplicate match hydration from jobs list responses

* Format jobs route test assertion
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.

1 participant