Skip to content

Conversation

@chongshenng
Copy link
Member

@chongshenng chongshenng commented Jan 28, 2026

On startup, enable databases to automigrate.

Additionally, cache SQL instances in SqlLinkState and SqlObjectStore to prevent race conditions, which is caused by multiple concurrent Alembic migration processes trying to run on the same database file.

As a cleanup, removed some redundant tests.

Merge after

@chongshenng chongshenng changed the base branch from main to feat/add-alembic-migration-utils January 28, 2026 22:54
@chongshenng chongshenng marked this pull request as ready for review January 28, 2026 22:59
@chongshenng chongshenng self-assigned this Jan 28, 2026
@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Jan 28, 2026
Base automatically changed from feat/add-alembic-migration-utils to main January 29, 2026 11:57
panh99
panh99 previously approved these changes Jan 29, 2026
Copy link
Member

@panh99 panh99 left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants