Skip to content

Fix SystemStackError when extending ActiveRecord::SchemaDumper#initialize with Module#prepend#12

Merged
krororo merged 1 commit intofeaturesfrom
fix-system-stack-error-with-strong_migrations
Sep 16, 2025
Merged

Fix SystemStackError when extending ActiveRecord::SchemaDumper#initialize with Module#prepend#12
krororo merged 1 commit intofeaturesfrom
fix-system-stack-error-with-strong_migrations

Conversation

@krororo
Copy link
Copy Markdown
Collaborator

@krororo krororo commented Sep 14, 2025

This PR imports the implementation from citusdata#271 originally created for citusdata/activerecord-multi-tenant.

Same issue as citusdata#216.

strong_migrations overrides ActiveRecord::SchemaDumper#initialize using Module#prepend, which caused a SystemStackError when used with activerecord-multi-tenant.

https://github.com/ankane/strong_migrations/blob/8e9baaa05c35fc9fdf206160c00b309c6e3a8bb6/lib/strong_migrations/schema_dumper.rb#L3-L7

Fix the issue by changing the alias in activerecord-multi-tenant to use prepend.

…lize with Module#prepend

Same issue as citusdata#216.

`strong_migrations` overrides `ActiveRecord::SchemaDumper#initialize` using `Module#prepend`, which caused a `SystemStackError` when used with `activerecord-multi-tenant`.

https://github.com/ankane/strong_migrations/blob/8e9baaa05c35fc9fdf206160c00b309c6e3a8bb6/lib/strong_migrations/schema_dumper.rb#L3-L7

Fix the issue by changing the `alias` in `activerecord-multi-tenant` to use `prepend`.
@krororo krororo changed the base branch from master to features September 14, 2025 06:39
@krororo krororo merged commit 8be008f into features Sep 16, 2025
290 of 292 checks passed
@krororo krororo deleted the fix-system-stack-error-with-strong_migrations branch September 20, 2025 08:53
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.

3 participants