Skip to content

Drop surplus_capturing_jit_order_owners table#4633

Merged
MartinquaXD merged 3 commits into
mainfrom
drop-jit-owners-table
Jul 16, 2026
Merged

Drop surplus_capturing_jit_order_owners table#4633
MartinquaXD merged 3 commits into
mainfrom
drop-jit-owners-table

Conversation

@MartinquaXD

Copy link
Copy Markdown
Contributor

Description

Follow up to #4613.
We had to drop the table 1 release after we stop using the table to not cause any issues during the rolling release when 2 autopilots are running.

Changes

drop surplus_capturing_jit_order_owners table and its documentation

@MartinquaXD
MartinquaXD requested a review from a team as a code owner July 16, 2026 12:33
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

Reminder: Please update the DB Readme and comment whether migrations are reversible (include rollback scripts if applicable).

  • If creating new tables, update the tables list.
  • When adding a new index, consider using CREATE INDEX CONCURRENTLY for tables involved in the critical execution path.
  • For breaking changes, remember that during rollout k8s starts the new autopilot, runs the Flyway migration, and only then shuts down the old pod. That overlap means the previous version can still be processing requests on the migrated schema, so make it compatible first and ship the breaking DB change in the following release.

Caused by:

@claude

claude Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Claude finished @MartinquaXD's task in 1m 19s —— View job


Review: Drop surplus_capturing_jit_order_owners table

  • Read PR changes
  • Verify no remaining code references to the table
  • Verify migration numbering & conventions
  • Post review

I posted 1 inline finding.

Summary: The intent is fine — the table's data was moved to competition_auctions, and the remaining surplus_capturing_jit_order_owners references in crates/database all point at the surviving competition_auctions column, not the dropped table. The version number 114 correctly follows V113.

The one blocking issue is that the migration file is missing the V prefix (114__… instead of V114__…), so Flyway would silently skip it and the table would never be dropped. Details in the inline comment.

Comment thread database/sql/V114__drop_surplus_capturing_jit_order_owners.sql

@jmg-duarte jmg-duarte left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹

@MartinquaXD
MartinquaXD added this pull request to the merge queue Jul 16, 2026
Merged via the queue into main with commit 5ff9e95 Jul 16, 2026
22 checks passed
@MartinquaXD
MartinquaXD deleted the drop-jit-owners-table branch July 16, 2026 15:03
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants