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: remove _pandas_importer(), dedup mock/_options.py numpy handling #4317
• SNOW-2912540: use IS_V5_DRIVER for cursor request_id access #4314 👈 (View in Graphite)
• SNOW-2912540: add IS_V5_DRIVER constant; use it for SecretDetector/TelemetryData imports and pandas/pyarrow/numpy resolution #4313
• SNOW-2912540: add IS_V5_DRIVER constant for version-conditioned connector imports #4310
• SNOW-2912540: decouple LocalTestOOBTelemetryService from connector.telemetry_oob #4309
• 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.
✅ Project coverage is 80.97%. Comparing base (69382c5) to head (9a8a97c).

Additional details and impacted files

@@                           Coverage Diff                            @@
##           SNOW-2912540-secret-detector-import    #4314       +/-   ##
========================================================================
+ Coverage                                67.33%   80.97%   +13.64%     
========================================================================
  Files                                      170      170               
  Lines                                    44666    44666               
  Branches                                  7680     7680               
========================================================================
+ Hits                                     30075    36169     +6094     
+ Misses                                   11874     6975     -4899     
+ Partials                                  2717     1522     -1195     

☔ 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.