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.

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

SNOW-878116: CopyInto does not support the PARTITION BY clause #430

Closed
azban opened this issue Jul 28, 2023 · 4 comments · Fixed by #542
Closed

SNOW-878116: CopyInto does not support the PARTITION BY clause #430

azban opened this issue Jul 28, 2023 · 4 comments · Fixed by #542
Assignees
Labels
feature status-fixed_awaiting_release The issue has been fixed, its PR merged, and now awaiting the next release cycle of the connector. status-triage_done Initial triage done, will be further handled by the driver team

Comments

@azban
Copy link
Contributor

azban commented Jul 28, 2023

What is the current behavior?

CopyInto does not support the PARTITION BY clause

What is the desired behavior?

CopyInto supports the PARTITION BY clause

How would this improve snowflake-connector-python?

This would allow users to use the PARTITION BY clause which is a documented feature of the COPY INTO statement

References, Other Background

https://docs.snowflake.com/en/sql-reference/sql/copy-into-location

@azban azban added the feature label Jul 28, 2023
@github-actions github-actions bot changed the title CopyInto does not support the PARTITION BY clause SNOW-878116: CopyInto does not support the PARTITION BY clause Jul 28, 2023
@sfc-gh-dszmolka sfc-gh-dszmolka added status-pr_pending_merge A PR is made and is under review status-triage_done Initial triage done, will be further handled by the driver team labels Apr 9, 2024
@sfc-gh-dszmolka
Copy link
Contributor

hi and thank you for raising this with us, especially the contribution in the PR to address the need ! asked review from the team, hopefully they can review soon.

@afavaro
Copy link

afavaro commented Nov 4, 2024

@sfc-gh-dszmolka

Any chance we could get a review on the PR that @azban provided? We're still maintaining our own fork of this library so that we can use this feature.

@sfc-gh-dszmolka
Copy link
Contributor

(reopening so we can keep track when the merged change will be released)

@sfc-gh-dszmolka sfc-gh-dszmolka added status-fixed_awaiting_release The issue has been fixed, its PR merged, and now awaiting the next release cycle of the connector. and removed status-pr_pending_merge A PR is made and is under review labels Nov 22, 2024
@sfc-gh-dszmolka
Copy link
Contributor

released with snowflake-sqlalchemy v1.7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature status-fixed_awaiting_release The issue has been fixed, its PR merged, and now awaiting the next release cycle of the connector. status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
4 participants