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

abort promise and sync for error #6009

Merged
merged 3 commits into from
Mar 22, 2024
Merged

abort promise and sync for error #6009

merged 3 commits into from
Mar 22, 2024

Conversation

n1ru4l
Copy link
Collaborator

@n1ru4l n1ru4l commented Mar 21, 2024

No description provided.

Copy link

changeset-bot bot commented Mar 21, 2024

🦋 Changeset detected

Latest commit: 49719ee

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@graphql-tools/executor Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@n1ru4l n1ru4l requested a review from ardatan March 21, 2024 16:05
Copy link
Contributor

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphql-tools/executor 1.2.4-alpha-20240321161036-49719ee14851856e28b870619771577956190332 npm ↗︎ unpkg ↗︎

Copy link
Contributor

✅ Benchmark Results

     ✓ no_errors
     ✓ expected_result

     checks.........................: 100.00% ✓ 338       ✗ 0  
     data_received..................: 39 MB   3.9 MB/s
     data_sent......................: 145 kB  14 kB/s
     http_req_blocked...............: avg=32µs     min=2.2µs    med=2.89µs   max=4.85ms   p(90)=4.43µs   p(95)=4.91µs  
     http_req_connecting............: avg=28.32µs  min=0s       med=0s       max=4.78ms   p(90)=0s       p(95)=0s      
     http_req_duration..............: avg=55.23ms  min=47.3ms   med=51.78ms  max=154.99ms p(90)=58.72ms  p(95)=87.94ms 
       { expected_response:true }...: avg=55.23ms  min=47.3ms   med=51.78ms  max=154.99ms p(90)=58.72ms  p(95)=87.94ms 
     http_req_failed................: 0.00%   ✓ 0         ✗ 169
     http_req_receiving.............: avg=137.51µs min=103.37µs med=132.88µs max=347.74µs p(90)=155.67µs p(95)=170.79µs
     http_req_sending...............: avg=26.73µs  min=20.01µs  med=26.11µs  max=94.25µs  p(90)=31.57µs  p(95)=35.89µs 
     http_req_tls_handshaking.......: avg=0s       min=0s       med=0s       max=0s       p(90)=0s       p(95)=0s      
     http_req_waiting...............: avg=55.06ms  min=47.16ms  med=51.62ms  max=154.55ms p(90)=58.58ms  p(95)=87.76ms 
     http_reqs......................: 169     16.812229/s
     iteration_duration.............: avg=59.45ms  min=51.02ms  med=55.91ms  max=163.78ms p(90)=65.04ms  p(95)=92.49ms 
     iterations.....................: 169     16.812229/s
     vus............................: 1       min=1       max=1
     vus_max........................: 1       min=1       max=1

Copy link
Contributor

💻 Website Preview

The latest changes are available as preview in: https://ed4d2910.graphql-tools.pages.dev

@n1ru4l n1ru4l merged commit 14a001e into master Mar 22, 2024
30 checks passed
@n1ru4l n1ru4l deleted the fix-abort-non-stream branch March 22, 2024 08:18
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