Skip to content

Conversation

@captbaritone
Copy link
Owner

https://github.com/graphql/graphql-js/releases/tag/v16.11.0

graphql/graphql-js#4312

Validated by running our benchmark with 10,000 types:

Before

Grats completed in: 9.090s
Grats completed in: 8.969s
Grats completed in: 9.007s
Grats completed in: 9.035s

After

Grats completed in: 7.957s
Grats completed in: 7.880s
Grats completed in: 7.946s
Grats completed in: 7.934s

Thanks for the perf improvement @JoviDeCroock!

@netlify
Copy link

netlify bot commented Sep 19, 2025

Deploy Preview for grats ready!

Name Link
🔨 Latest commit 609a620
🔍 Latest deploy log https://app.netlify.com/projects/grats/deploys/68cdd8caeab8b6000832c50a
😎 Deploy Preview https://deploy-preview-194--grats.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 project configuration.

@captbaritone captbaritone merged commit 6642f69 into main Sep 19, 2025
19 checks passed
@JoviDeCroock
Copy link

If you want, you can use 0no-co/graphql.web which has even better parse/print/visit perf. Happy that the perf stuff is helping people

@captbaritone
Copy link
Owner Author

I gave 0no-co/graphql.web's visit a try as a drop in replacement for the instances that remain after #196 and it seems to not make much of a difference.

Not a problem, but I figured I'd report back what I found. Thanks again for all your work on GraphQL tooling!

import {visit} from "graphql";

Grats completed in: 7.594s
Grats completed in: 7.336s
Grats completed in: 7.405s
Grats completed in: 7.344s
Grats completed in: 7.384s

import {visit} from "0no-co/graphql.web";

Grats completed in: 7.553s
Grats completed in: 7.739s
Grats completed in: 7.520s
Grats completed in: 7.470s
Grats completed in: 7.501s

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.

3 participants