SNOW-2912540: use IS_V5_DRIVER for cursor request_id access

[sfc-gh-fpawlowski]

Summary

• The UD's SnowflakeCursor exposes the client-generated request UUID as a public request_id property; _request_id is only a backward-compat alias slated for removal.
• The legacy v4 connector is the opposite: it only ever sets self._request_id as a plain instance attribute in cursor.py, with no public request_id property at all.
• Branch on IS_V5_DRIVER at the one call site that reads this — execute_and_notify_query_listener in server_connection.py — and its unit test mock.
• Investigated (but left untouched) the other _request_id-named attributes in the repo: AstBatch._request_id and AstBuilder's request-id generation are Snowpark's own self-generated UUIDs for the AST Bind/Eval batching protocol — unrelated to the connector cursor, same name by coincidence.

Test plan

• grep -rn "\._request_id\b" src/ tests/ confirms only the AST-batch (unrelated) and the fixed call sites remain

Checklist

• I acknowledge that I have ensured my changes to be thread-safe

---

Stack (via Graphite)

• #4307
• #4282
• #4308
• #4309
• #4313
• #4314

🤖 Generated with Claude Code

[sfc-gh-fpawlowski]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• SNOW-2912540: use IS_V5_DRIVER for cursor request_id access #4314 : 2 dependent PRs (#4317 , #4321 ) 👈 (View in Graphite)
• SNOW-2912540: add IS_V5_DRIVER constant; use it for SecretDetector/TelemetryData/TelemetryClient/ReauthenticationRequest imports and pandas/pyarrow/numpy resolution #4313
• SNOW-2912540: decouple LocalTestOOBTelemetryService from connector.telemetry_oob #4309 : 1 other dependent PR (#4310 )
• SNOW-2912540: inline _create_temp_stage and _create_temp_file_format into analyzer_utils #4308
• SNOW-2912540: define ASYNC_RETRY_PATTERN and compat.OK locally #4282
• SNOW-2912540: translate create_temp_table=True to table_type in AST encoding (rebased onto main) #4307
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (SNOW-2912540-secret-detector-import@6568b8e). Learn more about missing BASE report.

Additional details and impacted files

@@                          Coverage Diff                           @@
##             SNOW-2912540-secret-detector-import    #4314   +/-   ##
======================================================================
  Coverage                                       ?   95.44%           
======================================================================
  Files                                          ?      172           
  Lines                                          ?    44771           
  Branches                                       ?     7682           
======================================================================
  Hits                                           ?    42730           
  Misses                                         ?     1264           
  Partials                                       ?      777           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.