Skip to content

[SDK] Add retry for subgraph requests that return bad indexer#3661

Merged
portuu3 merged 7 commits intodevelopfrom
feat/sdk/retry
Nov 13, 2025
Merged

[SDK] Add retry for subgraph requests that return bad indexer#3661
portuu3 merged 7 commits intodevelopfrom
feat/sdk/retry

Conversation

@flopez7
Copy link
Copy Markdown
Contributor

@flopez7 flopez7 commented Nov 6, 2025

Issue tracking

#3660

Context behind the change

TBD

How has this been tested?

TBD

Release plan

Deploy new SDK version for python and typescript

Potential risks; What to monitor; Rollback plan

Check if graphql requests are made correctly

@flopez7 flopez7 requested a review from portuu3 November 6, 2025 16:08
@flopez7 flopez7 self-assigned this Nov 6, 2025
@flopez7 flopez7 added the WIP Work In Progress label Nov 6, 2025
@vercel
Copy link
Copy Markdown

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
human-app Ready Ready Preview Comment Nov 12, 2025 1:21pm
human-dashboard-frontend Ready Ready Preview Comment Nov 12, 2025 1:21pm
staking-dashboard Ready Ready Preview Comment Nov 12, 2025 1:21pm
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
faucet-frontend Ignored Ignored Preview Nov 12, 2025 1:21pm
faucet-server Ignored Ignored Preview Nov 12, 2025 1:21pm

Copy link
Copy Markdown
Collaborator

@portuu3 portuu3 left a comment

Choose a reason for hiding this comment

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

Let's rename SubgraphRetryConfig to something like SubgrahpRequestConfig or SubgraphRequestParams and gqlFetchWithRetry to customGqlFetch. Then we can use it for #3666

…thon SDKs to handle subgraph requests with retry logic for bad indexer errors.

- Updated interfaces to rename SubgraphRetryConfig to SubgraphOptions for better clarity.
- Generated changeset for new changes
@vercel vercel bot temporarily deployed to Preview – staking-dashboard November 10, 2025 10:53 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend November 10, 2025 10:53 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app November 10, 2025 10:53 Inactive
@vercel vercel bot temporarily deployed to Preview – staking-dashboard November 10, 2025 11:58 Inactive
@vercel vercel bot temporarily deployed to Preview – human-dashboard-frontend November 10, 2025 11:58 Inactive
@vercel vercel bot temporarily deployed to Preview – human-app November 10, 2025 11:58 Inactive
@flopez7 flopez7 requested a review from portuu3 November 10, 2025 12:02
Comment thread packages/sdk/typescript/human-protocol-sdk/src/utils.ts Outdated
Comment thread packages/sdk/typescript/human-protocol-sdk/src/utils.ts Outdated
@portuu3 portuu3 removed the WIP Work In Progress label Nov 13, 2025
@portuu3 portuu3 merged commit 398f13a into develop Nov 13, 2025
22 checks passed
@portuu3 portuu3 deleted the feat/sdk/retry branch November 13, 2025 11:26
@portuu3 portuu3 mentioned this pull request Nov 13, 2025
10 tasks
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.

2 participants