Skip to content
This repository has been archived by the owner on Apr 3, 2024. It is now read-only.

Update server #204

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update server #204

wants to merge 1 commit into from

Conversation

cretz
Copy link
Member

@cretz cretz commented Mar 10, 2023

What changed?

Update server to 1.20.0

@cretz cretz requested a review from jlegrone March 10, 2023 14:06
@cretz
Copy link
Member Author

cretz commented Mar 10, 2023

A change was made on Temporal server that no longer allows immediate search attribute use even when the SA-cache-disable is set. I am discussing internally with server team.

@cretz
Copy link
Member Author

cretz commented Mar 10, 2023

Note, this is blocked on 3 issues:

  1. Replace temporal.Services with temporal.DefaultServices as done at Upgrade server to v1.19.1@master cli#96
  2. Add namespace to SA adding
  3. Known server regression when disabling search attribute cache

The first 2 can be solved but the 3rd I believe is a blocker for those wanting to disable SA cache. See server issue temporalio/temporal#4017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants