Skip to content

Conversation

@Shivs11
Copy link
Member

@Shivs11 Shivs11 commented Dec 11, 2025

This reverts commit b55fbfc.

What changed?

  • Just reverting the above commit.

Why?

  • This commit, I think, was causing existing unit-tests to timeout consistently. Reverting this should unblock the team from merging things in the temporal repo.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Potential risks

  • Not risky

Note

Sticky task queues now fetch user data from the workflow root partition instead of a hashed normal partition; removes hashing logic and farm dependency.

  • Matching / user data fetching:
    • Sticky task queues now query user data from workflow RootPartition() instead of selecting a normal partition via hashing.
    • Removed consistent-partition hashing logic and the go-farm dependency.

Written by Cursor Bugbot for commit 6d4ec7e. This will update automatically on new commits. Configure here.

@Shivs11 Shivs11 marked this pull request as ready for review December 11, 2025 14:16
@Shivs11 Shivs11 requested review from a team as code owners December 11, 2025 14:16
@Shivs11 Shivs11 merged commit c776d8a into main Dec 11, 2025
58 of 59 checks passed
@Shivs11 Shivs11 deleted the ss/fix-unit-tests-breaking-on-main branch December 11, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants