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.

Sign up for GitHub

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

Jump to bottom

Add metrics for the graphql complexity #2274

Closed
xgreenx opened this issue Oct 1, 2024 · 2 comments
Closed

Add metrics for the graphql complexity #2274

xgreenx opened this issue Oct 1, 2024 · 2 comments
Assignees
@AurelienFT
Labels
good first issue Good for newcomers

Comments

@xgreenx
Copy link
Collaborator

xgreenx commented Oct 1, 2024

Track the complexity of each query in the histogram divided by 1k, 5k, 10k, 20k, 50k, 100k, 250k, 500k, 1M, 5M, 10M
@xgreenx xgreenx added the good first issue Good for newcomers label Oct 1, 2024
@tusharnagar17
Copy link

Hi @xgreenx,
I would like to work on this.
Would appreciate any clarification or guidance you can provide ?

@netrome netrome assigned AurelienFT and unassigned AurelienFT Oct 10, 2024
@AurelienFT AurelienFT mentioned this issue Oct 14, 2024
Merged
5 tasks
xgreenx added a commit that referenced this issue Oct 14, 2024
@AurelienFT @xgreenx
Add graphql query complexity histogram metric (#2349)
aba4d36 
## Linked Issues/PRs
Closes #2274

## Checklist
- [x] Breaking changes are clearly marked as such in the PR description
and changelog
- [x] New behavior is reflected in tests
- [x] [The specification](https://github.com/FuelLabs/fuel-specs/)
matches the implemented behavior (link update PR if changes are needed)

### Before requesting review
- [x] I have reviewed the code myself
- [x] I have created follow-up issues caused by this PR and linked them
here

Co-authored-by: Green Baneling <[email protected]>
@AurelienFT
Copy link
Contributor

Done in #2349

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AurelienFT AurelienFT

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@xgreenx @AurelienFT @tusharnagar17