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

Long running queries not returning results #15

Closed
manish-kotecha-cko opened this issue Nov 22, 2021 · 3 comments
Closed

Long running queries not returning results #15

manish-kotecha-cko opened this issue Nov 22, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@manish-kotecha-cko
Copy link

Hey, we've been having issues with long running queries not being executed properly. According to the snowflake docs if a query takes longer than 45 seconds to execute you will have a QueryStatus object returned with a different response code instead of the result but it seems as though this scenario is not being considered.
https://docs.snowflake.com/en/developer-guide/sql-api/guide.html#handling-the-response

@manish-kotecha-cko manish-kotecha-cko changed the title Long running queries timing out Long running queries not returning results Nov 22, 2021
@fixer-m
Copy link
Owner

fixer-m commented Dec 1, 2021

@manish-kotecha-cko,
Hi! Thanks for reporting this and sorry for a long-waited answer.
Unfortunately I've been too busy at work lately. I can't say when I will resume my active work on this client (but I hope for next 2-3 months). I think long running queries definitely should be supported by this client, so I'll make it my next higher priority feature. Not sure though how I can make it without API breaking changes.

@manish-kotecha-cko
Copy link
Author

manish-kotecha-cko commented Dec 2, 2021

Hey Ilya, no worries and thanks for the nice project.
In terms of avoiding breaking changes I think perhaps just checking the response code and retrying in a loop if it's gives the alternative response code and returning the result if there is the result response code?
I think there is a similar implementation in Snowflake.Data client.

@fixer-m fixer-m added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Jan 23, 2022
@fixer-m
Copy link
Owner

fixer-m commented Jan 29, 2022

@manish-kotecha-cko Hi!

Yep, it turns out that implementation for this is really really simple - thanks for this hint :) Not sure why I thought that this will require API breaking changes.

So I've just released new version 0.3.9 that can handle long-running queries.

@fixer-m fixer-m closed this as completed Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants