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

SNOW-657330: WithFileStream support for GET queries #654

Closed
mihaitodor opened this issue Sep 7, 2022 · 6 comments
Closed

SNOW-657330: WithFileStream support for GET queries #654

mihaitodor opened this issue Sep 7, 2022 · 6 comments
Assignees
@sfc-gh-anugupta
Labels
enhancement The issue is a request for improvement or a new 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

@mihaitodor
Copy link

Issue description

I'd like to write some code which downloads a file from a Snowflake stage into an in-memory stream. However, it looks like this client library only supports specifying an actual file as the destination. Note that for PUT operations, I can use WithFileStream to do the reverse operation.

Example code

See the documentation here and here and the code here which only applies to PUT operations.

Error log

N/A

Configuration

N/A
@github-actions github-actions bot changed the title WithFileStream support for GET queries SNOW-657330: WithFileStream support for GET queries Sep 7, 2022
@sfc-gh-dszmolka
Copy link
Contributor

thank you for raising this request with us, we'll see how this capability could be added to the driver. cannot commit to any expected timeline - thank you for bearing with us !

@sfc-gh-dszmolka sfc-gh-dszmolka added the enhancement The issue is a request for improvement or a new feature label Mar 28, 2023
@mihaitodor
Copy link
Author

Are there any updates on your roadmap regarding this enhancement? It would be great to be able to add a GET-based input to Benthos, since right now it only provides a PUT-based output.

@sfc-gh-pfus
Copy link
Collaborator

Hi @mihaitodor ! We are in the process of planning upcoming three months. I added this one as a candidate. Fingers crossed it will be accepted to do!

@sfc-gh-dszmolka sfc-gh-dszmolka added the status-triage_done Initial triage done, will be further handled by the driver team label Mar 12, 2024
@sfc-gh-dszmolka
Copy link
Contributor

we now have a draft PR at #1192 for the implementation. will keep this thread posted.

@sfc-gh-dszmolka sfc-gh-dszmolka added status-pr_pending_merge A PR is made and is under review 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 Aug 23, 2024
@sfc-gh-dszmolka
Copy link
Contributor

change has been merged and will be part of the next release, expected coming in the near future

@sfc-gh-dszmolka
Copy link
Contributor

released with gosnowflake v.1.11.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sfc-gh-anugupta sfc-gh-anugupta

Labels
enhancement The issue is a request for improvement or a new 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
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mihaitodor @sfc-gh-dszmolka @sfc-gh-anugupta @sfc-gh-pfus