Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce an option in context.configure() to specify a particular schema name for the alembic_version table #76

Closed
sqlalchemy-bot opened this issue Oct 4, 2012 · 9 comments

Comments

@sqlalchemy-bot
Copy link

@sqlalchemy-bot sqlalchemy-bot commented Oct 4, 2012

Migrated issue, originally created by Christian Blume (@cblume)

In init of MigrationContext I would suggest the following:

version_table = opts.get('version_table', 'alembic_version')
version_table_schema = opts.get('version_table_schema', None)
self._version = Table(
    version_table, MetaData(),
    Column('version_num', String(32), nullable=False),
    schema=version_table_schema)

This would solve this issue quite easily. What do you think?

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Oct 4, 2012

Changes by Christian Blume (@cblume):

  • edited description
1 similar comment
@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Oct 4, 2012

Changes by Christian Blume (@cblume):

  • edited description
@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Oct 4, 2012

Michael Bayer (@zzzeek) wrote:

seems fine. can you create a pull request?

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Oct 4, 2012

Christian Blume (@cblume) wrote:

never done that before but I'll look into it tomorrow. Pretty late over here, time to go home..

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Oct 8, 2012

Michael Bayer (@zzzeek) wrote:

yeah. sorry, takes a while to get through backlog for things, also need to put in #77

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Apr 4, 2013

Michael Bayer (@zzzeek) wrote:

thanks

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Apr 4, 2013

Changes by Michael Bayer (@zzzeek):

  • changed status to closed
@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented May 16, 2013

Changes by Michael Bayer (@zzzeek):

  • edited description
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant