Skip to content

Conversation

@spkane31
Copy link
Contributor

READ BEFORE MERGING: All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted.

What changed?
Adding degraded_workflow_visibility to NamespaceInfo.Capabilities.

This will allow us to programmatically turn on the saved query for degraded workflow visibility.

Breaking changes
No

@spkane31 spkane31 requested review from a team as code owners September 30, 2025 21:17
// True if the namespace supports worker heartbeats
bool worker_heartbeats = 4;
// True if the namespace supports degraded workflow visibility
bool degraded_workflow_visibility = 5;
Copy link
Member

@cretz cretz Sep 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would suggest a rename, e.g. reported_problems_search_attribute or something heh. At first glance I read this as there was a form of workflow visibility that was degraded from the normal form if this was true. This is not blocking, though I am not sure the word "degraded" is the best product name.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed

@spkane31 spkane31 merged commit 467d73d into master Sep 30, 2025
4 checks passed
@spkane31 spkane31 deleted the spk/degraded-wf-vis-ns-info branch September 30, 2025 22:30
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.

4 participants