temporal icon indicating copy to clipboard operation
temporal copied to clipboard

Make taskCategoryRegistry history service only

Open yycptt opened this issue 11 months ago • 1 comments

What changed?

  • Make taskCategoryRegistry history service only instead of a top level module that's used by all four services.
  • (tdbg will still depend on it for validation and converting task category name to ID)

Why?

  • After https://github.com/temporalio/temporal/pull/5442 and https://github.com/temporalio/temporal/pull/5426, frontend no longer depends on the taskCategoryRegisty and only history service is using that, so it doesn't need to be a top level module. Also ideally, only history service on the server side needs to be the category definition.

How did you test it?

  • Build server, existing functional tests.

Potential risks

  • There's should be no runtime risk.

Documentation

  • N/A

Is hotfix candidate?

  • No

yycptt avatar Mar 12 '24 00:03 yycptt

This PR was marked as stale. Please update or close it.

github-actions[bot] avatar Jul 11 '24 00:07 github-actions[bot]