Skip to content

Dynamic partitioning: refactor client#11132

Open
dnr wants to merge 4 commits into
temporalio:mainfrom
dnr:dp15
Open

Dynamic partitioning: refactor client#11132
dnr wants to merge 4 commits into
temporalio:mainfrom
dnr:dp15

Conversation

@dnr

@dnr dnr commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

What changed?

Refactor matching client to simplify code:

  • parse task queue rpc name early and pass tqid.Partition
  • move cache key construction into invokeWithPartitionCounts
  • make resolvePartition return bool "loadBalance"

The only behavior difference is that matching->matching forwarded calls now
don't involve partition counts/the partition cache (there was no reason for them
to, they were directed to a specific partition).

Why?

Cleaner, more efficient code, easier to add methods.

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)

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.

1 participant