Skip to content

Fix non-deterministic row ordering - #4411

Merged
sfc-gh-mayliu merged 1 commit into
mainfrom
fix-doctest-random-ordering
Oct 5, 2026
Merged

sfc-gh-mayliu merged 1 commit into
mainfrom
fix-doctest-random-ordering

Conversation

@sfc-gh-mayliu

@sfc-gh-mayliu sfc-gh-mayliu commented Oct 2, 2026 •

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

    Fixes SNOW-NNNNNNN

  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.

    Please write a short description of how your code change solves the related issue.

Fixing flaky doctests for non-deterministic row ordering in AI functions

@sfc-gh-mayliu
sfc-gh-mayliu requested a review from a team as a code owner October 2, 2026 23:41
@sfc-gh-mayliu sfc-gh-mayliu added the NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md label Oct 2, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4411   +/-   ##
=======================================
  Coverage   95.47%   95.47%           
=======================================
  Files         176      176           
  Lines       45271    45271           
  Branches     7759     7759           
=======================================
  Hits        43222    43222           
  Misses       1269     1269           
  Partials      780      780           

☔ 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-mayliu
sfc-gh-mayliu merged commit 4c82827 into main Oct 5, 2026
36 of 44 checks passed
@sfc-gh-mayliu
sfc-gh-mayliu deleted the fix-doctest-random-ordering branch October 5, 2026 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants