Skip to content

Issue635 - #650

Open
pachadotdev wants to merge 4 commits into
rstudio:mainfrom
pachadotdev:issue635
Open

Issue635#650
pachadotdev wants to merge 4 commits into
rstudio:mainfrom
pachadotdev:issue635

Conversation

@pachadotdev

Copy link
Copy Markdown
Collaborator

the solution is to generalize is_tbl_mssql() to DuckDB and other backends and detect by class

@rich-iannone

Copy link
Copy Markdown
Member

@pachadotdev Thank you for this PR (and all the other ones you submitted). In getting that last urgent release out I wasn't able to get this one in for that. But I'm hoping to resolve this one soon. Along with some light modernization work, this PR will be part of the next release. Thanks again!

@pachadotdev

Copy link
Copy Markdown
Collaborator Author

@pachadotdev Thank you for this PR (and all the other ones you submitted). In getting that last urgent release out I wasn't able to get this one in for that. But I'm hoping to resolve this one soon. Along with some light modernization work, this PR will be part of the next release. Thanks again!

Thanks @rich-iannone !

No rush, I have been using PostgreSQL 100% of the time in the last months. This was just a tiny back-contribution as I saw another user had an issue.

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