Skip to content

Add postgresl.REGCLASS type for casting table names to OIDs and vice versa#435

Closed
xflr6 wants to merge 2 commits into
zzzeek:masterfrom
xflr6:regclass
Closed

Add postgresl.REGCLASS type for casting table names to OIDs and vice versa#435
xflr6 wants to merge 2 commits into
zzzeek:masterfrom
xflr6:regclass

Conversation

@xflr6
Copy link
Copy Markdown
Contributor

@xflr6 xflr6 commented Apr 2, 2018

@zzzeek
Copy link
Copy Markdown
Owner

zzzeek commented Apr 2, 2018

looks perfect. Can I stick with 1.3 or do you need a backport into 1.2 ? (1.3 is not released for many months).

@xflr6
Copy link
Copy Markdown
Contributor Author

xflr6 commented Apr 2, 2018

Thanks. I currently use the UserDefinedType snippet from the bb issue, but I think (if it's not too much effort) it would be nice to include it in the next 1.2 release.

@zzzeek
Copy link
Copy Markdown
Owner

zzzeek commented Apr 2, 2018

Dear contributor -

This pull request is being moved to Gerrit, at https://gerrit.sqlalchemy.org/#/c/zzzeek/sqlalchemy/+/715, where it may be tested and reviewed more closely. As such, the pull request itself is being marked "closed" or "declined", however your contribution is merely being moved to our central review system. Please register at https://gerrit.sqlalchemy.org#/register/ to send and receive comments regarding this item.

@zzzeek zzzeek closed this Apr 2, 2018
zzzeek pushed a commit that referenced this pull request Apr 3, 2018
…versa

Fixes: #4160
Change-Id: Id0bdbad1be3a0950dc8f35895ee13d9264244722
Pull-request: #435
zzzeek pushed a commit that referenced this pull request Apr 3, 2018
…versa

Fixes: #4160
Change-Id: Id0bdbad1be3a0950dc8f35895ee13d9264244722
Pull-request: #435
(cherry picked from commit 9f986ce)
@xflr6 xflr6 deleted the regclass branch April 3, 2018 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants