Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Failover support#98

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
linglingye001 merged 24 commits intopreviewfromlinglingye/failover
Nov 18, 2024
Merged

Failover support#98

linglingye001 merged 24 commits intopreviewfromlinglingye/failover
Nov 18, 2024

Conversation

@linglingye001
Copy link
Member

@linglingye001linglingye001 commentedSep 14, 2024•
edited by zhiyuanliang-ms
Loading

Replica Auto-Discovery: For App Configuration stores with geo-replication enabled, the provider will now automatically discover any additional replicas and attempt to connect to them when it fails to connect to user-provided endpoints in code. This capability allows applications to leverage geo-replication for enhanced resiliency without code change and redeployment. Replica discovery is enabled by default and can be disabled by setting the new AzureAppConfigurationOptions.replicaDiscoveryEnabled property equal to false.

Usage:

constappConfig=awaitload("YOUR-CONNECTION-STRING",{replicaDiscoveryEnabled:true})

Note:

Replica discovery can only work in node.js environment.

@zhiyuanliang-mszhiyuanliang-ms changed the base branch frommain topreviewSeptember 23, 2024 09:53
@linglingye001linglingye001 marked this pull request as ready for reviewSeptember 30, 2024 08:32
@zhiyuanliang-ms
Copy link
Member

zhiyuanliang-ms commentedOct 21, 2024•
edited
Loading

How about seperating this PR into two smaller PR to make it easier to review. For example, the change of adding client manager can be a seperated PR and I can approve it quickly. And with client manager change merged, it won't block the following new feature like snapshot support.

Eskibear reacted with thumbs up emoji

@linglingye001linglingye001 merged commit7db665f intopreviewNov 18, 2024
4 checks passed
@linglingye001linglingye001 deleted the linglingye/failover branchNovember 18, 2024 08:07

functionisFailoverableError(error:any):boolean{
// ENOTFOUND: DNS lookup failed, ENOENT: no such file or directory
returnisRestError(error)&&(error.code==="ENOTFOUND"||error.code==="ENOENT"||

Choose a reason for hiding this comment

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

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@zhiyuanliang-mszhiyuanliang-mszhiyuanliang-ms approved these changes

@RichardChen820RichardChen820Awaiting requested review from RichardChen820

@juniwangjuniwangAwaiting requested review from juniwang

+1 more reviewer

@EskibearEskibearEskibear left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@linglingye001@zhiyuanliang-ms@Eskibear@RichardChen820
[8]ページ先頭
©2009-2025 Movatter.jp