Skip to content

Conversation

@yaacovCR
Copy link
Contributor

The incremental graph can handle a stream as a stream, rather than creating a linked list where each incremental data record also includes the next record in addition to any new defers and/or streams.

Enables easily batching all available stream items within the same incremental entry.

Depends on #4094

@netlify
Copy link

netlify bot commented May 30, 2024

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit dfada7d
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/6661261460b1fb00083ba314
😎 Deploy Preview https://deploy-preview-4098--compassionate-pike-271cb3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions
Copy link

Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment:

  • @github-actions run-benchmark - Run benchmark comparing base and merge commits for this PR
  • @github-actions publish-pr-on-npm - Build package from this PR and publish it on NPM

@yaacovCR yaacovCR force-pushed the stream-as-stream branch 2 times, most recently from 4ea959d to a2f24f5 Compare May 30, 2024 19:25
@yaacovCR yaacovCR requested a review from robrichard May 30, 2024 19:27
@yaacovCR yaacovCR added the PR: polish 💅 PR doesn't change public API or any observed behaviour label May 30, 2024
@yaacovCR yaacovCR force-pushed the stream-as-stream branch 2 times, most recently from 5fc2723 to 198cb70 Compare June 4, 2024 06:55
Rather than creating a linked list where each incremental data record also includes the next record in addition to any new defers and/or streams.

Enables easily batching all available stream items within the same incremental entry.
@yaacovCR yaacovCR force-pushed the stream-as-stream branch from 198cb70 to dfada7d Compare June 6, 2024 02:59
@yaacovCR yaacovCR merged commit 89f9223 into graphql:main Jun 12, 2024
@yaacovCR yaacovCR deleted the stream-as-stream branch June 12, 2024 19:49
yaacovCR added a commit that referenced this pull request Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: polish 💅 PR doesn't change public API or any observed behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants