Skip to content

Batching based on sharding target - #738

Merged
rboustany-crypto merged 2 commits into
masterfrom
COMMIT-26150-batching-on-cloud-id
Jul 22, 2026
Merged

Batching based on sharding target#738
rboustany-crypto merged 2 commits into
masterfrom
COMMIT-26150-batching-on-cloud-id

Conversation

@rboustany-crypto

Copy link
Copy Markdown
Contributor

Please make sure you consider the following:

  • Add tests that use __typename in queries
  • Does this change work with all nadel transformations (rename, type rename, hydration, etc)? Add tests for this.
  • Is it worth using hints for this change in order to be able to enable a percentage rollout?
  • Do we need to add integration tests for this change in the graphql gateway?
  • Do we need a pollinator check for this?

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown

Test Results

  659 files  +  659    659 suites  +659   1m 11s ⏱️ + 1m 11s
2 212 tests +2 212  1 652 ✅ +1 652  560 💤 +560  0 ❌ ±0 
2 220 runs  +2 220  1 660 ✅ +1 660  560 💤 +560  0 ❌ ±0 

Results for commit b1914cb. ± Comparison against base commit fe33085.

♻️ This comment has been updated with latest results.

@rboustany-crypto
rboustany-crypto marked this pull request as ready for review July 21, 2026 07:26
@rboustany-crypto
rboustany-crypto merged commit 2a4f8eb into master Jul 22, 2026
6 checks passed
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