Michael Bayer wrote:
> im inclined to just get something in there that does the job for
> now...the patch thats on ticket #155 is very close to whats needed.   if
> you can try it out and maybe link to a "quote=True" keyword argument on
> Table and Column, make some test cases, we're in business.

I noticed the patch quotes Table.fullname. I'm not sure that will work
because you will get "schema.table" rather than schema."table". And the
patch doesn't account for quoted schema names. I wonder if schemas
shouldn't be upgraded from just a string to an object.

myschema = Schema("MySchema", quoted=True)
mytable = Table("MyTable",
        Column("MyColumn", quoted=True),
        ...
        quoted=True,
        schema=myschema
        )


I'd be happy to rewrite the patch and run some tests, but I think I'll
need a little bit more direction. Perhaps I can catch you on IRC
sometime to talk it over.

Aaron Spike

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Sqlalchemy-users mailing list
Sqlalchemy-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlalchemy-users

Reply via email to