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

Update server - #204

Open
cretz wants to merge 1 commit into
temporalio:mainfrom
cretz:update-server
Open

Update server#204
cretz wants to merge 1 commit into
temporalio:mainfrom
cretz:update-server

Conversation

@cretz

@cretz cretz commented Mar 10, 2023

Copy link
Copy Markdown
Contributor

What changed?

Update server to 1.20.0

@cretz
cretz requested a review from jlegrone March 10, 2023 14:06
@cretz

cretz commented Mar 10, 2023

Copy link
Copy Markdown
Contributor Author

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

cretz commented Mar 10, 2023

Copy link
Copy Markdown
Contributor Author

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