Skip to content

Cherrypicks to aio connector part9 #2460

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
Open
wants to merge 39 commits into
base: cherrypicks-to-aio-connector-part8
Choose a base branch
from
Open

Cherrypicks to aio connector part9 #2460

wants to merge 39 commits into from
+1,461 −1,400

Conversation

sfc-gh-fpawlowski
Copy link
Contributor

@sfc-gh-fpawlowski sfc-gh-fpawlowski commented Aug 9, 2025 •
edited
Loading

@sfc-gh-fpawlowski sfc-gh-fpawlowski self-assigned this Aug 9, 2025
@sfc-gh-fpawlowski sfc-gh-fpawlowski changed the base branch from dev/aio-connector to cherrypicks-to-aio-connector-part7 August 9, 2025 08:44
@sfc-gh-fpawlowski sfc-gh-fpawlowski force-pushed the cherrypicks-to-aio-connector-part9 branch from d2dba53 to d683f13 Compare August 9, 2025 09:12
@sfc-gh-fpawlowski sfc-gh-fpawlowski changed the base branch from cherrypicks-to-aio-connector-part7 to cherrypicks-to-aio-connector-part8 August 9, 2025 09:14
@sfc-gh-fpawlowski sfc-gh-fpawlowski force-pushed the cherrypicks-to-aio-connector-part9 branch 2 times, most recently from fba0ce8 to 68d64a4 Compare August 9, 2025 12:14
@sfc-gh-pczajka sfc-gh-pczajka force-pushed the cherrypicks-to-aio-connector-part8 branch from 609427f to 511d01e Compare August 11, 2025 10:50
sfc-gh-pczajka
sfc-gh-pczajka approved these changes Aug 11, 2025
View reviewed changes
Copy link
Collaborator

@sfc-gh-pczajka sfc-gh-pczajka left a comment

Choose a reason for hiding this comment

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

LGTM

@sfc-gh-pczajka sfc-gh-pczajka force-pushed the cherrypicks-to-aio-connector-part8 branch 4 times, most recently from b48434d to a513b10 Compare August 14, 2025 14:19
sfc-gh-lspiegelberg and others added 18 commits August 14, 2025 17:19
@sfc-gh-lspiegelberg @sfc-gh-pczajka
SNOW-2027116 Allow for UUID encoding in SnowflakeRestful interface (#…
6035eee 
…2254)
@sfc-gh-pczajka
[ASYNC] apply #2254 to async code
6d59d70
@sfc-gh-pbulawa @sfc-gh-pczajka
SNOW-1955965: Fix expired S3 credentials update (#2258)
912a19d
@sfc-gh-pczajka
[ASYNC] apply #2258 to async code
dbb7007
@sfc-gh-mkubik @sfc-gh-pczajka
NO-SNOW Add PAT to authenticators allowing empty username, remove han…
153ecd9 
…dling of PAT in password field (#2264)
@sfc-gh-pczajka
[ASYNC] Apply #2264 to async code
106fa06
@sfc-gh-mkubik @sfc-gh-pczajka
NO-SNOW Fix flaky query timeout test (#2266)
f6b293b
@sfc-gh-akolodziejczyk @sfc-gh-pczajka
SNOW-2040000 change tag to bptp-stable (#2268)
45f634d
@sfc-gh-mmishchenko @sfc-gh-pczajka
SNOW-2028051 introduce a new client_fetch_threads connection paramete…
9e5b250 
…r to decouple threads number limitations on fetching and pre-fetching (#2255)
@sfc-gh-pmansour @sfc-gh-pczajka
Add default entra app ID for Snowflake (#2267)
2d4b87e
@sfc-gh-pczajka
[ASYNC] update test after #2267
b7564b8
@sfc-gh-fpawlowski @sfc-gh-pczajka
SNOW-2011595 Masking filter introduced on library levels (#2253)
2c9393d
@sfc-gh-pczajka
[ASYNC] remove azure filter after #2253
3ef0650
@sfc-gh-mmishchenko @sfc-gh-pczajka
NO-SNOW acquiring a lock on local OCSP cache will use a timeout (#2280)
35199cd
@sfc-gh-pmansour @sfc-gh-pczajka
Accept both v1 and v2 Entra ID issuer formats for WIF (#2281)
bc7162b
@sfc-gh-pczajka
[ASYNC] apply #2281 to async code
7f56afa
@sfc-gh-mmishchenko @sfc-gh-pczajka
SNOW-2048239 revert zero timeout for oscp cache lock (#2283)
4d5ed38
@sfc-gh-pczajka
[ASYNC] remove flaky marker from OCSP tests after #2283
404bf8e
@sfc-gh-pczajka sfc-gh-pczajka force-pushed the cherrypicks-to-aio-connector-part8 branch from a513b10 to eb23308 Compare August 14, 2025 15:19
sfc-gh-zyao and others added 19 commits August 27, 2025 09:23
@sfc-gh-zyao @sfc-gh-fpawlowski
SNOW-1963078 Port _upload / _download / _upload_stream / _download_st… (
24bf2a0 
#2198)

(cherry picked from commit a3229c3)
@sfc-gh-fpawlowski
[Async] Apply #2198 to async code
f2bc003
@sfc-gh-mkubik @sfc-gh-fpawlowski
SNOW-1989239 - prevent silent failures on nano-arrow conversion (#2227)
5f2bdde 
Co-authored-by: Adam Ling <[email protected]>

(cherry picked from commit 5e96035)
@sfc-gh-fpawlowski
[Async] Apply #2227 to async code
3c2ef0a
@sfc-gh-mkubik @sfc-gh-fpawlowski
NO-SNOW skip out of range year test on old driver (#2243)
eb2336b 
(cherry picked from commit 2ab3745)
@sfc-gh-fpawlowski
[Async] Apply #2243 to async code
428083f
@sfc-gh-mkubik @sfc-gh-fpawlowski
SNOW-191538 remove copyright headers (#2238)
7810b63 
(cherry picked from commit 813bbd8)
@sfc-gh-fpawlowski
[Async] Apply #2238 to async code
a5cf354
@sfc-gh-pbulawa @sfc-gh-fpawlowski
SNOW-1789751: Pass GCS regional and virtual params (#2241)
5a3d3d1 
(cherry picked from commit 985ec5e)
@sfc-gh-fpawlowski
[Async] Apply 2241 to async code
0751e89
@sfc-gh-zyao @sfc-gh-fpawlowski
SNOW-2019505 fix inconsistent force_put_overwrite value for _upload a… (
fb1f692 
#2247)

(cherry picked from commit e200620)
@sfc-gh-fpawlowski
[Async] Apply #2247 to async code
ab9f182
@sfc-gh-pbulawa @sfc-gh-fpawlowski
SNOW-1896089: Lower log level (#2251)
c66bcf1 
(cherry picked from commit 44313a3)
@sfc-gh-fpawlowski
[Async] Apply #2251 to async code
552593c
@sfc-gh-jszczerbinski @sfc-gh-fpawlowski
SNOW-2026002: Invalid url became valid (#2252)
a13aca0 
(cherry picked from commit 3a798e8)
@sfc-gh-lspiegelberg @sfc-gh-fpawlowski
SNOW-2011670 Allow url parameter requestId to be set with statement p…
b4f88ab 
…arams (#2240)

As part of the Snowpark IR project, we will send AST as part of the request. To restore them faithfully on the server-side and avoid introducing yet another layer of indirection, we would like to use the HTTP Request Id. Currently, this Request Id is generated on the client as part of issuing the query. With this PR we introduce a new (client-side) statement parameter requestId that allows to overwrite the automatically generated request Id. requestId must be a valid UUID4 or UUID4 string.

(cherry picked from commit 6e214d0)
@sfc-gh-fpawlowski
[Async] Apply #2240 to async code
bd4554d
@sfc-gh-mkubik @sfc-gh-fpawlowski
NO-SNOW skip tests of custom requestId on olddriver (#2256)
53969eb 
(cherry picked from commit cc563d8)
@sfc-gh-fpawlowski
[Async] Apply #2256 to async code
02aeda8
@sfc-gh-fpawlowski sfc-gh-fpawlowski force-pushed the cherrypicks-to-aio-connector-part9 branch from 68d64a4 to 02aeda8 Compare August 27, 2025 07:29
@sfc-gh-pczajka sfc-gh-pczajka force-pushed the cherrypicks-to-aio-connector-part8 branch from eb23308 to 8c7688e Compare August 27, 2025 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sfc-gh-pczajka sfc-gh-pczajka sfc-gh-pczajka approved these changes

Assignees

@sfc-gh-fpawlowski sfc-gh-fpawlowski

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@sfc-gh-fpawlowski @sfc-gh-pczajka @sfc-gh-lspiegelberg @sfc-gh-pbulawa @sfc-gh-mkubik @sfc-gh-akolodziejczyk @sfc-gh-mmishchenko @sfc-gh-pmansour @sfc-gh-zyao @sfc-gh-jszczerbinski