Skip to content

Commit

Permalink
address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
anekkanti committed Nov 7, 2023
1 parent 173aa90 commit cecb4be
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions temporal/api/cloud/namespace/v1/message.proto
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,11 @@ message NamespaceSpec {
// This will create a namespace that's available at '<name>.<account>.tmprl.cloud:7233'.
// The name is immutable. Once set, it cannot be changed.
string name = 1;
// The regions where the namespace should be hosted.
// Specifying more than one region makes the namespace a Global Namespace, which is currently a preview only feature with restricted access.
// Please reach out to temporal support for more information on Global Namespaces.
// The regions of availability for the namespace.
// Specifying more than one region makes the namespace "global", which is currently a preview only feature with restricted access.
// Please reach out to Temporal support for more information on global namespaces.
// When provisioned the Global Namespace will be active on the first region in the list and passive on the rest.
// Number of supported regions today is 2. Could be more in the future.

This comment has been minimized.

Copy link
@atihkin

atihkin Nov 7, 2023

Ah sorry - this was my mistake. Didn't intend for this to be the text verbatim. Let's just say "Number of supported regions is 2."

This comment has been minimized.

Copy link
@anekkanti

anekkanti Nov 7, 2023

Author Member

fixed.

// The regions is immutable. Once set, it cannot be changed.
repeated string regions = 2;
// The number of days the workflows data will be retained for.
Expand Down

0 comments on commit cecb4be

Please sign in to comment.