Skip to content

refactor(metadata-db): extract job_status projection module#1895

Merged
shiyasmohd merged 1 commit intomainfrom
shiyasmohd/job-status-table
Mar 4, 2026
Merged

refactor(metadata-db): extract job_status projection module#1895
shiyasmohd merged 1 commit intomainfrom
shiyasmohd/job-status-table

Conversation

@shiyasmohd
Copy link
Contributor

@shiyasmohd shiyasmohd commented Mar 3, 2026

Separate job status tracking from the jobs module into a dedicated job_status module.

  • Add jobs_status table via migration with backfill and indexes
  • Create job_status module with state transition functions
  • Move JobStatus enum and JobStatusUpdateError from jobs module
  • Update jobs/sql.rs queries to JOIN jobs_status for status fields
  • Add integration tests

@shiyasmohd shiyasmohd force-pushed the shiyasmohd/job-status-table branch 2 times, most recently from eb5ec33 to 9453d2a Compare March 3, 2026 19:25
@shiyasmohd shiyasmohd requested review from LNSD, Theodus, leoyvens and mitchhs12 and removed request for LNSD March 3, 2026 19:34
@shiyasmohd shiyasmohd self-assigned this Mar 3, 2026
@shiyasmohd shiyasmohd force-pushed the shiyasmohd/job-status-table branch from 9453d2a to 2f89ce3 Compare March 4, 2026 05:38
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

Please, check my comments 🙂

@shiyasmohd shiyasmohd force-pushed the shiyasmohd/job-status-table branch from 2f89ce3 to 01d7b40 Compare March 4, 2026 08:35
@shiyasmohd shiyasmohd requested a review from LNSD March 4, 2026 08:46
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@shiyasmohd shiyasmohd merged commit da4d5cb into main Mar 4, 2026
8 of 9 checks passed
@shiyasmohd shiyasmohd deleted the shiyasmohd/job-status-table branch March 4, 2026 11:01
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