Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple nodeUrls configuration #769

Closed
wants to merge 3 commits into from

Conversation

SteveYuOWO
Copy link

@SteveYuOWO SteveYuOWO commented Jan 25, 2022

Multiple nodeUrls configuration #733

  • Support multiple nodeUrls & compatible with nodeUrl
  • Failover node when fetchJson error

Based on the retry mechanism, the program only needs to switch the next node when fetch error. The user had configured n nodes. When the kth node fetch error, failover to ( k + 1 ) % n nodes.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2022

This PR is stale because it has been open for 7 days with no activity. Remove stale label or comment or this will be closed in another 7 days.

@github-actions github-actions bot added the Stale label Feb 2, 2022
@github-actions github-actions bot closed this Feb 9, 2022
@TrevorJTClarke
Copy link
Contributor

wow, just saw this - seems exactly what I was hoping for - sucks things get set to stale and not reviewed. Can it be re-opened?

@SteveYuOWO
Copy link
Author

wow, just saw this - seems exactly what I was hoping for - sucks things get set to stale and not reviewed. Can it be re-opened?

Glad to reopen it. 😃

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

Successfully merging this pull request may close these issues.

2 participants