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-1005946: Make SQL text optional for the input to StmtUtil.cancel #1602

Closed
sfc-gh-pliu opened this issue Jan 11, 2024 · 0 comments
Closed
Labels

Comments

@sfc-gh-pliu
Copy link

sfc-gh-pliu commented Jan 11, 2024

What is the current behavior?

Currently, this low-level query-cancellation util will throw if the SQL text is not provided in the argument stmtInput, as shown below.

AssertUtil.assertTrue(stmtInput.sql != null, "Missing sql for statement execution");

Sometimes when we are cancelling query, the SQL text itself is not available. Can we make this field optional?

What is the desired behavior?

The util won't throw when no SQL text is provided.

How would this improve snowflake-jdbc?

Allow stmtUtil.cancel to be used in cases where the SQL text is not available.

References, Other Background

What is your Snowflake account identifier, if any?

@github-actions github-actions bot changed the title Make SQL text optional for the input to StmtUtil.cancel SNOW-1005946: Make SQL text optional for the input to StmtUtil.cancel Jan 11, 2024
@sfc-gh-pliu sfc-gh-pliu closed this as not planned Won't fix, can't repro, duplicate, stale Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant