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

implement autogen detection for ONUPDATE/ONDELETE in fks #317

Closed
sqlalchemy-bot opened this issue Aug 19, 2015 · 4 comments
Closed

implement autogen detection for ONUPDATE/ONDELETE in fks #317

sqlalchemy-bot opened this issue Aug 19, 2015 · 4 comments

Comments

@sqlalchemy-bot
Copy link

@sqlalchemy-bot sqlalchemy-bot commented Aug 19, 2015

Migrated issue, originally created by Michael Bayer (@zzzeek)

since some backends support "options" and we support FKs.

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Aug 19, 2015

Michael Bayer (@zzzeek) wrote:

Issue #79 was marked as a duplicate of this issue.

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Aug 19, 2015

Michael Bayer (@zzzeek) wrote:

  • Implemented support for autogenerate detection of changes in the
    ondelete, onupdate, initially and deferrable
    attributes of :class:.ForeignKeyConstraint objects on
    SQLAlchemy backends that support these on reflection
    (as of SQLAlchemy 1.0.8 currently Postgresql for all four,
    MySQL for ondelete and onupdate only). A constraint object
    that modifies these values will be reported as a "diff" and come out
    as a drop/create of the constraint with the modified values.
    The fields are ignored for backends which don't reflect these
    attributes (as of SQLA 1.0.8 this includes SQLite, Oracle, SQL Server,
    others). fixes #317

df92071

@sqlalchemy-bot
Copy link
Author

@sqlalchemy-bot sqlalchemy-bot commented Aug 19, 2015

Changes by Michael Bayer (@zzzeek):

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

@sqlalchemy-bot sqlalchemy-bot commented Aug 24, 2015

jmagnusson (@jmagnusson) wrote:

Fantastic addition, thanks Mike!

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
You can’t perform that action at this time.