Thanks! I didn't see that line. I try to compile an example tomorrow. While working on https://github.com/sqlalchemy/alembic/pull/526 it was necessary to duplicate column object with foreign_keys Regards, Martin
On Thursday, 10 January 2019 16:05:42 UTC+1, Mike Bayer wrote: > > On Thu, Jan 10, 2019 at 8:18 AM Martin Haass <[email protected] > <javascript:>> wrote: > > > > Hi, > > is it a bug or a feature? > > > > > https://github.com/sqlalchemy/sqlalchemy/blob/master/lib/sqlalchemy/sql/schema.py:1292 > > ff. > > in the constructor of class column the foreign_keys and constraints are > explicitly initialized to set() whereas the other properties are popped > from kwargs > > not a bug, ForeignKey() objects are passed to Column as positional > arguments and they are taken in here: > > https://github.com/sqlalchemy/sqlalchemy/blob/master/lib/sqlalchemy/sql/schema.py#L1335 > > > > > > This kind of contradicts the column.,copy method which writes > constraints and foreign_keys into kwargs that are passed to the constructor > (lines 1492) > > I don't see that, the constraints are written into a list called args, > no keywords, which is then passed along where it is accepted by line > 1335 above > > > The duplicated column objects in my case lack the foreign_keys > > ah, where you say "my case", that's the important part to share. > Bugs are illustrated by showing an example of code, what you expect it > to do, and what it's not doing. Can you provide that? > > > > > > > Thx > > Martin > > > > -- > > SQLAlchemy - > > The Python SQL Toolkit and Object Relational Mapper > > > > http://www.sqlalchemy.org/ > > > > To post example code, please provide an MCVE: Minimal, Complete, and > Verifiable Example. See http://stackoverflow.com/help/mcve for a full > description. > > --- > > You received this message because you are subscribed to the Google > Groups "sqlalchemy" group. > > To unsubscribe from this group and stop receiving emails from it, send > an email to [email protected] <javascript:>. > > To post to this group, send email to [email protected] > <javascript:>. > > Visit this group at https://groups.google.com/group/sqlalchemy. > > For more options, visit https://groups.google.com/d/optout. > -- SQLAlchemy - The Python SQL Toolkit and Object Relational Mapper http://www.sqlalchemy.org/ To post example code, please provide an MCVE: Minimal, Complete, and Verifiable Example. See http://stackoverflow.com/help/mcve for a full description. --- You received this message because you are subscribed to the Google Groups "sqlalchemy" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/sqlalchemy. For more options, visit https://groups.google.com/d/optout.
