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-558109: QueryStatus Enum indicates "blocked" status is both running and an error #717

Closed
taylor-dv opened this issue Mar 10, 2022 · 4 comments
Assignees
Labels
feature status-triage_done Initial triage done, will be further handled by the driver team

Comments

@taylor-dv
Copy link

When a statement is waiting for a lock, the query status may be "BLOCKED".

The two QueryStatus enum helper methods to determine if a query is still running or resulted in an error indicate that "BLOCKED" is both "RUNNING" and an "ERROR".

It does not make sense for "BLOCKED" to be both "RUNNING" and an "ERROR" when no other existing status is grouped liked this.

It seems like "BLOCKED" should be categorized the same as "QUEUED". Neither state are a result of some error in the query itself but rather the state of warehouse.

I'd like to propose "BLOCKED" be removed from the "ERROR" category. Alternatively query statues could be distinguished between waiting, running, and completed states.

@github-actions github-actions bot changed the title QueryStatus Enum indicates "blocked" status is both running and an error SNOW-558109: QueryStatus Enum indicates "blocked" status is both running and an error Mar 10, 2022
@sfc-gh-igarish
Copy link
Collaborator

To clean up and re-prioritize more pressing bugs and feature requests we are closing all issues older than 6 months as of March 1, 2023. If there are any issues or feature requests that you would like us to address, please create them according to the new templates we have created. For urgent issues, opening a support case with this link Snowflake Community is the fastest way to get a response.

@bpodgursky
Copy link

bruh

@sfc-gh-igarish
Copy link
Collaborator

sfc-gh-igarish commented Nov 28, 2023

@sfc-gh-ext-simba-ti Our apps team (Josh Zana) interested in this fix.

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

fixed in #1579 and released with 3.14.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants