-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Expose tokenCount on the DocumentNode #4292
Conversation
Hi @JoviDeCroock, I'm @github-actions bot happy to help you with this PR 👋 Supported commandsPlease post this commands in separate comments and only one per comment:
|
2c53b79
to
780846d
Compare
This comment has been minimized.
This comment has been minimized.
Benchmark output
🍳 Preparing HEAD... �[32mHEAD�[0m�[90m x �[0m�[32m42.36�[0m ops/sec �[90m±�[0m�[32m0.83�[0m�[36m%�[0m�[90m x �[0m 2.2 MB/op�[90m (8 runs sampled)�[0m ⏱ Build Schema from Introspection �[32mHEAD�[0m�[90m x �[0m�[32m51.03�[0m ops/sec �[90m±�[0m�[32m0.83�[0m�[36m%�[0m�[90m x �[0m1.12 MB/op�[90m (11 runs sampled)�[0m ⏱ Execute Introspection Query HEAD�[90m x �[0m�[32m 48.8�[0m ops/sec �[90m±�[0m�[32m1.32�[0m�[36m%�[0m�[90m x �[0m2.49 MB/op�[90m (8 runs sampled)�[0m ⏱ Parse introspection query HEAD�[90m x �[0m�[32m13,524�[0m ops/sec �[90m±�[0m�[32m0.35�[0m�[36m%�[0m�[90m x �[0m2.62 KB/op�[90m (34 runs sampled)�[0m ⏱ Many repeated fields HEAD�[90m x �[0m�[32m97.93�[0m ops/sec �[90m±�[0m�[32m0.68�[0m�[36m%�[0m�[90m x �[0m524 KB/op�[90m (20 runs sampled)�[0m ⏱ Validate Introspection Query �[32mHEAD�[0m�[90m x �[0m�[32m559�[0m ops/sec �[90m±�[0m�[32m0.42�[0m�[36m%�[0m�[90m x �[0m270 KB/op�[90m (14 runs sampled)�[0m ⏱ Validate Invalid Query �[32mHEAD�[0m�[90m x �[0m�[32m503�[0m ops/sec �[90m±�[0m�[32m0.47�[0m�[36m%�[0m�[90m x �[0m326 KB/op�[90m (14 runs sampled)�[0m ⏱ Validate SDL Document HEAD�[90m x �[0m�[32m62.79�[0m ops/sec �[90m±�[0m�[32m1.63�[0m�[36m%�[0m�[90m x �[0m223 KB/op�[90m (10 runs sampled)�[0m ⏱ Visit all AST nodes HEAD�[90m x �[0m�[32m299�[0m ops/sec �[90m±�[0m�[33m2.48�[0m�[36m%�[0m�[90m x �[0m311 KB/op�[90m (19 runs sampled)�[0m ⏱ Visit all AST nodes in parallel HEAD�[90m x �[0m�[32m28.19�[0m ops/sec �[90m±�[0m�[32m0.28�[0m�[36m%�[0m�[90m x �[0m1.23 MB/op�[90m (7 runs sampled)�[0m |
Backports #4251 from v17
Backporting this so we can document the
maxTokens
for production use cases. The performance of this change looks to be neutral