SNOW-2912540: remap write_pandas missing-table errors via ProgrammingError.raw_msg

[sfc-gh-fpawlowski]

Summary

• Session.write_pandas remapped a missing table only when ProgrammingError.msg ended with "does not exist".
• Universal Driver suffixes .msg with (request_id=..., sfqid=...), so the remap never fired and raw 001757 leaked.
• Check getattr(pe, "raw_msg", pe.msg) instead — both v4 and v5 store the unformatted body there.

Context

Stacked on #4326. Recovers test_write_pandas / test_write_pandas_with_overwrite on UD (CI run 32475873257, 9 unique tests / 18 grouped failures).

Test plan

• tests/integ/test_pandas_to_df.py::test_write_pandas
• tests/integ/test_pandas_to_df.py::test_write_pandas_with_overwrite
• Confirm legacy connector still remaps (.raw_msg / .msg both end with "does not exist").

Made with Cursor

[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: remap write_pandas missing-table errors via ProgrammingError.raw_msg #4328 👈 (View in Graphite)
• SNOW-2912540: mock _connect on v5 so local testing skips UD connection_init #4326
• SNOW-2912540: mock-patch try_add_log_to_batch instead of reading _log_batch #4321 : 1 other dependent PR (#4322 )
• SNOW-2912540: use IS_V5_DRIVER for cursor request_id access #4314 : 1 other dependent PR (#4317 )
• 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.

[graphite-app]

Potential AttributeError if raw_msg attribute exists but is None. The getattr function only returns the default value when the attribute doesn't exist, not when it exists with a None value. If pe.raw_msg is None, line 3632 will call None.endswith() and crash.

Fix:

msg = getattr(pe, "raw_msg", None) or pe.msg
if msg.endswith("does not exist"):

Or add a null check:

msg = getattr(pe, "raw_msg", pe.msg)
if msg and msg.endswith("does not exist"):

Suggested change

	msg = getattr(pe, "raw_msg", pe.msg)
	if msg.endswith("does not exist"):
	msg = getattr(pe, "raw_msg", None) or pe.msg
	if msg.endswith("does not exist"):

Spotted by Graphite



Is this helpful? React 👍 or 👎 to let us know.

[snowflake-security-bot]

Snowflake Security Review

Security grade: A — Passed ✅

This PR was classified as LOW risk by the automated pre-screen.

[snowflake-security-bot]

Snowflake Security Review

Security grade: A — Passed ✅

This PR was classified as LOW risk by the automated pre-screen.