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.

Sign up for GitHub

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

Jump to bottom

Add insert_default param to Column #11374

Closed
CaselIT opened this issue May 9, 2024 · 2 comments
Closed

Add insert_default param to Column #11374

CaselIT opened this issue May 9, 2024 · 2 comments
Labels
schema things related to the DDL related objects like Table, Column, CreateIndex, etc. use case not really a feature or a bug; can be support for new DB features or user use cases not anticipated
Milestone
2.0.x

Comments

@CaselIT
Copy link
Member

CaselIT commented May 9, 2024

I think we might want to consider adding the insert_default param to Column directly and making that the official name. not sure why I didn't start doing that in 2.0

Originally posted by @zzzeek in #11372 (reply in thread)
@CaselIT CaselIT added schema things related to the DDL related objects like Table, Column, CreateIndex, etc. use case not really a feature or a bug; can be support for new DB features or user use cases not anticipated labels May 9, 2024
@CaselIT CaselIT added this to the 2.0.x milestone May 9, 2024
@sqla-tester
Copy link
Collaborator

Federico Caselli has proposed a fix for this issue in the main branch:

Add insert_default to Column. https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/5299

@sqla-tester
Copy link
Collaborator

Federico Caselli has proposed a fix for this issue in the rel_2_0 branch:

Add insert_default to Column. https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/5300

sqlalchemy-bot pushed a commit that referenced this issue May 22, 2024
@CaselIT
Add insert_default to Column.
951646f 
Added :paramref:`_schema.Column.insert_default` as an alias of
:paramref:`_schema.Column.default` for compatibility with
func:`_orm.mapped_column`.

Fixes: #11374
Change-Id: I5509b6cbac7b37ac37430a88442b1319cc9c1024
(cherry picked from commit c87572b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
schema things related to the DDL related objects like Table, Column, CreateIndex, etc. use case not really a feature or a bug; can be support for new DB features or user use cases not anticipated
Projects
None yet
Milestone
2.0.x
Development

No branches or pull requests
2 participants
@sqla-tester @CaselIT