SNOW-2912540: translate create_temp_table=True to table_type in AST encoding (rebased onto main)

[sfc-gh-fpawlowski]

Summary

Rebases the fix from #4296 onto main. #4296's head branch (worktree-create-temp-table-ast-fix) is based on the stale ud-local-test-scripts branch, which is 49 commits behind main — so it can't land there cleanly. This PR cherry-picks #4296's two commits (unchanged, same authorship) directly onto current main, where they apply without conflicts.

Same fix as #4296: stops emitting the deprecated create_temp_table parameter into the encoded AST sent to the server, without changing Snowpark's own public API — create_temp_table stays in the function signatures, still fires its deprecation warning, and still works exactly as before for callers. Only table_type (already resolved from create_temp_table when needed) is now recorded in the AST.

Changes:

• dataframe_writer.py: Move create_temp_table deprecation coercion to before the AST emission block in save_as_table, so table_type is already resolved when WriteTable is encoded. Remove expr.create_temp_table emission.
• session.py: Remove ast.create_temp_table = create_temp_table from the write_pandas AST block — the coercion already fires before AST emission there, so ast.table_type carries the correct value.
• dataframe.py: Replace create_temp_table=True with table_type="temp" in the internal cache_result mock path, matching the real code path and avoiding a spurious deprecation warning from internal code.
• ast.proto: Mark both create_temp_table fields as // Deprecated: use table_type instead. (fields retained for wire compatibility).
• tests/ast/data/session_write_pandas.test: Remove create_temp_table: true from expected encoded AST and create_temp_table=True from expected unparser output.

Test plan

• tests/ast/test_ast_driver.py::test_ast[session_write_pandas.test] passes
• tests/ast/test_ast_driver.py::test_ast[DataFrame.write.test] passes
• Manually verified on a local-testing session: save_as_table(..., create_temp_table=True) still logs the deprecation warning and creates a temp table
• Manually verified cache_result() no longer emits the spurious deprecation warning

Supersedes #4296 for the purpose of landing on main.

---

Stack (via Graphite)

• #4307
• #4282
• #4308
• #4309
• #4310

🤖 Generated with Claude Code

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.46%. Comparing base (e7d77e7) to head (7d281d5).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4307      +/-   ##
==========================================
+ Coverage   95.26%   95.46%   +0.19%     
==========================================
  Files         171      171              
  Lines       44720    44718       -2     
  Branches     7676     7676              
==========================================
+ Hits        42604    42691      +87     
+ Misses       1299     1253      -46     
+ Partials      817      774      -43     

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

[sfc-gh-fpawlowski]

• 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 👈 (View in Graphite)
• main

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