[sfc-gh-yixie]

1. Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

   Fixes SNOW-4468388

2. Fill out the following pre-review checklist:

   • I am adding a new automated test(s) to verify correctness of my new code
     • If this test skips Local Testing mode, I'm requesting review from snowflakedb/local-testing
   • I am adding new logging messages
   • I am adding a new telemetry message
   • I am adding new credentials
   • I am adding a new dependency
   • If this is a new feature/behavior, I'm adding the Local Testing parity changes.
   • I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines
   • If adding any arguments to public Snowpark APIs or creating new public Snowpark APIs, I acknowledge that I have ensured my changes include AST support. Follow the link for more information: AST Support Guidelines

3. Please describe how your code solves the related issue.

   This is a follow-up to SNOW-4467757: Add TRY_TO_* conversion functions #4496. The new try_to_* functions accept the format as a Column or a literal str, and a str is always a literal format, never a column name. Most existing to_* functions already do the same. This PR aligns the four existing functions that didn't. Snowflake accepts a non-constant format for all of them; I verified this live.

   Function	Before	Change
   to_timestamp	Worked with a str or a Column, but typed Optional[Column]	Type hint widened to ColumnOrLiteralStr or an int scale; AST built explicitly. Generated SQL is unchanged.
   to_char / to_varchar	Already handled a Column, but typed Optional[str]	Type hint and docstring only
   to_binary, try_to_binary	A Column fmt raised TypeError: Cannot convert a Column object into bool (if fmt:)	Use fmt is not None and _to_col_if_lit, build the AST explicitly, widen the type hints

   Behavior change: to_binary/try_to_binary with fmt="" used to drop the format silently. It is now passed to Snowflake. This is noted in the CHANGELOG.

   Local testing: the mock layer decides how to pass arguments from the public type hints, so after widening them a literal format reaches mock_to_binary as a column holding one repeated value. mock_to_binary now handles that:

   • It treats a format column with a single repeated value as a constant.
   • It returns NULL when the format is NULL, as Snowflake does.
   • It raises NotImplementedError when the format differs from row to row.

   The to_char and to_timestamp mocks already handled per-row formats. try_to_binary has no local-testing mock on main, so its new test is skipped in local testing.

   Testing:

   • New integration tests in tests/integ/test_function.py:

     • test_to_binary_fmt_literal_and_column and test_try_to_binary_fmt_literal_and_column: HEX, BASE64 and UTF-8 formats in different rows; invalid values give NULL for try_to_binary.
     • test_to_char_column_format (for both to_char and to_varchar): two date formats in different rows.
     • test_to_timestamp_fmt_variants: a str format, a Column format and int scales.

     All pass live and with --local_testing_mode, except the try_to_binary test, which is skipped in local testing as noted above.

   • AST test data in tests/ast/data/functions_fmt_column.test. The existing functions1 and functions2 expectations did not change.

   • The doctests, tests/mock, tests/unit and the local-testing runs of test_function.py/test_column.py pass.

🤖 Generated with Claude Code

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.47%. Comparing base (aad22f1) to head (b7133a8).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4497   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files         179      179           
  Lines       46927    46927           
  Branches     8110     8106    -4     
=======================================
+ Hits        44803    44805    +2     
+ Misses       1332     1331    -1     
+ Partials      792      791    -1     

☔ 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-yuwang]

can we add tests to test when fmt is ""?

[sfc-gh-yixie]

Added in 131152a. Checked against Snowflake first:

• to_binary(e, "") raises Invalid binary format string ''. The test asserts this, and also covers an unknown format ("FOO").
• try_to_binary(e, "") returns NULL for every row, including valid hex input. The test asserts this. It runs live only, because try_to_binary has no local-testing mock.

This also turned up two bugs in the local-testing to_binary mock, fixed in the same commit:

• It treated "" as HEX, so it silently decoded valid input where Snowflake errors.
• Its invalid-format check tested the wrong variable, so an unknown format failed with an unrelated error.

Both now raise Invalid binary format string, matching Snowflake. The fix is listed under the local-testing bug fixes in the CHANGELOG.

---

_{🤖 Posted by an AI agent (Claude Code · skill none) on behalf of @sfc-gh-yixie.}