Skip to content

Support explicit schema selection for the migration table#386

Open
pnehrer wants to merge 4 commits into
rust-db:mainfrom
barndoor-ai:pnehrer/migration-table-schema
Open

Support explicit schema selection for the migration table#386
pnehrer wants to merge 4 commits into
rust-db:mainfrom
barndoor-ai:pnehrer/migration-table-schema

Conversation

@pnehrer

@pnehrer pnehrer commented Jun 6, 2025

Copy link
Copy Markdown

Add the ability to specify a table schema for the refinery history table, in case a non-default schema is required:

  • refinery cli takes an optional --table-schema argument
  • when a schema is provided, refinery attempts to create it, if it doesn't exist, when asserting the existence of the history table
  • all table operations subsequent to this use the qualified table name so that they are performed in the correct schema

@pnehrer

pnehrer commented Jun 25, 2025

Copy link
Copy Markdown
Author

@kriswuollett apologies in advance for my unfamiliarity with the process -- how can I get this PR reviewed? Thank you in advance!

@github-actions github-actions Bot added the Stale label Mar 28, 2026
@jxs

jxs commented Mar 29, 2026

Copy link
Copy Markdown
Member

Hi, thanks for the interest and apologies for the delay in reviewing!
I am curious, can you elaborate on why you need to define an explicit schema for the migration table?

@jxs jxs removed the Stale label Mar 29, 2026
@pnehrer

pnehrer commented Apr 20, 2026

Copy link
Copy Markdown
Author

@jxs certainly -- in our application we use a single database with multiple schemas, roughly one for each micro-service or domain. Each micro-service/domain manages its own tables and migrations. Thus, the migration table must be in the respective schema; otherwise conflicts arise among them. Thanks!

@github-actions github-actions Bot added the Stale label May 21, 2026
@github-actions github-actions Bot closed this May 29, 2026
@jxs jxs added keep-open and removed Stale labels Jun 5, 2026
@jxs jxs reopened this Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants