Skip to content

support for SQL expressions associated with types #1534

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

Closed
sqlalchemy-bot opened this issue Sep 8, 2009 · 10 comments
Closed

support for SQL expressions associated with types #1534

sqlalchemy-bot opened this issue Sep 8, 2009 · 10 comments

Comments

@sqlalchemy-bot
Copy link
Collaborator

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

i.e. geo types, date types, etc. basically the bindparam() produced by this type should instead be the desired SQL expression.


Attachments: 1534.1.patch | 1534.2.patch

@sqlalchemy-bot
Copy link
Collaborator Author

Changes by Michael Bayer (@zzzeek):

  • changed milestone from "0.6.0" to "0.6.xx"

@sqlalchemy-bot
Copy link
Collaborator Author

Michael Bayer (@zzzeek) wrote:

this really shouldn't be that big of a deal. Postgis really needs it.

@sqlalchemy-bot
Copy link
Collaborator Author

Changes by Michael Bayer (@zzzeek):

  • added labels: high priority
  • changed milestone from "0.x.xx" to "0.8.0"

@sqlalchemy-bot
Copy link
Collaborator Author

Michael Bayer (@zzzeek) wrote:

first patch. gets the basic idea and the postgis example working. lots of new overhead needs to be reduced.

@sqlalchemy-bot
Copy link
Collaborator Author

Changes by Michael Bayer (@zzzeek):

  • attached file 1534.1.patch

@sqlalchemy-bot
Copy link
Collaborator Author

Michael Bayer (@zzzeek) wrote:

next patch, callcounts are OK, but we aren't meeting the "returning" use case. no tests yet.

@sqlalchemy-bot
Copy link
Collaborator Author

Changes by Michael Bayer (@zzzeek):

  • attached file 1534.2.patch

@sqlalchemy-bot
Copy link
Collaborator Author

Michael Bayer (@zzzeek) wrote:

the majority of the work with tests is in a2468c8. still need to improve the result map propagation though.

@sqlalchemy-bot
Copy link
Collaborator Author

Michael Bayer (@zzzeek) wrote:

last fix for now is in 515811e

@sqlalchemy-bot
Copy link
Collaborator Author

Changes by Michael Bayer (@zzzeek):

  • changed status to closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant