Skip to content

move ai session cleanup after migration#9541

Merged
pjain1 merged 1 commit into
mainfrom
ai_session_migrate_race
Jun 11, 2026
Merged

move ai session cleanup after migration#9541
pjain1 merged 1 commit into
mainfrom
ai_session_migrate_race

Conversation

@pjain1

@pjain1 pjain1 commented Jun 10, 2026

Copy link
Copy Markdown
Member

After #9500 rill start is throwing error like because the cleanup starts in open method before migration has run which creates these tables.

ERROR    sqlite: failed to delete expired AI sessions    {"error": "failed to query expired AI sessions: SQL logic error: no such table: ai_sessions (1)"}

Although its harmless but still move clean up after catalog migration has run.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@pjain1 pjain1 requested a review from begelundmuller June 10, 2026 21:17
Comment thread runtime/drivers/sqlite/migrate.go
@pjain1 pjain1 merged commit 3def2c9 into main Jun 11, 2026
23 checks passed
@pjain1 pjain1 deleted the ai_session_migrate_race branch June 11, 2026 10:16
pjain1 added a commit that referenced this pull request Jun 11, 2026
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