Skip to content

Comments

Preserve snowflake SQL API HTTP 422 response payload#61052

Closed
dilshadshaik3031 wants to merge 2 commits intoapache:mainfrom
dilshadshaik3031:422-payload-snowflake
Closed

Preserve snowflake SQL API HTTP 422 response payload#61052
dilshadshaik3031 wants to merge 2 commits intoapache:mainfrom
dilshadshaik3031:422-payload-snowflake

Conversation

@dilshadshaik3031
Copy link
Contributor

Fixes #60894

Preserve error body when Snowflake SQL API returns HTTP 422, so users can see the actual failure message instead of losing the payload.

@boring-cyborg boring-cyborg bot added area:providers provider:snowflake Issues related to Snowflake provider labels Jan 25, 2026
Copy link
Contributor

@Srabasti Srabasti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR @dilshadshaik3031 however static checks are failing!

Indentation errors are present in multiple places, including here.

Have you reproduced locally by running prek in your branch? Prek will fix any formatting errors, and then you can push the commit from your branch. It is super easy to fix them. The first time is toughest:)

To run prek as part of git workflow, use prek install to set up git hooks.
image

@HyunWooZZ
Copy link
Contributor

I think this PR overlaps with the changes already implemented in my PR (#60891).
Could you please take a look at my PR as well?

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

Labels

area:providers provider:snowflake Issues related to Snowflake provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Snowflake SQL API: Add support for statement timeout

3 participants