Skip to content

fix: split funding_events and slab_deletion_queue migrations - #17

Merged
pcfreak30 merged 1 commit into
masterfrom
fix/split-mixed-migration
Jul 3, 2026
Merged

fix: split funding_events and slab_deletion_queue migrations#17
pcfreak30 merged 1 commit into
masterfrom
fix/split-mixed-migration

Conversation

@pcfreak30

@pcfreak30 pcfreak30 commented Jul 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Split the bundled migration that combined funding_events ALTER TABLE and CREATE TABLE slab_deletion_queue into two separate migration functions
  • slab_deletion_queue now matches upstream's migration index; the funding_events ALTER is a separate migration appended after

This PR splits a combined database migration into two separate migration steps. Previously, the funding_events table alterations (adding fund_type and pool_id columns with their indexes) and the slab_deletion_queue table creation were executed together in a single migration function. They are now separated into distinct migration functions, with the slab_deletion_queue table creation running first, followed by the funding_events column and index additions. This separation ensures each schema change is applied as an individual migration step, improving migration reliability and rollback granularity.

…tions

Slab_deletion_queue was bundled into the same migration function as the
funding_events ALTER TABLE. Split them so slab_deletion_queue matches
upstream's migration index, with the funding_events ALTER as a separate
migration appended after.
@kody-ai

kody-ai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@pcfreak30
pcfreak30 merged commit ca00b84 into master Jul 3, 2026
7 of 13 checks passed
@pcfreak30
pcfreak30 deleted the fix/split-mixed-migration branch July 3, 2026 02:16
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