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

[PR #10304/b6ffb1d1 backport][3.11] Add aiohttp-openmetrics to list of third party aiohttp-related python… #10305

Open
wants to merge 1 commit into
base: 3.11
Choose a base branch
from

Conversation

patchback[bot]
Copy link
Contributor

@patchback patchback bot commented Jan 7, 2025

This is a backport of PR #10304 as merged into master (b6ffb1d).

… modules

What do these changes do?

Add aiohttp-openmetrics to the list of third party aiohttp-related python modules.

Are there changes in behavior for the user?

N/A

Is it a substantial burden for the maintainers to support this?

No

Related issue number

N/A

Checklist

  • I think the code is well written (N/A)
  • Unit tests for the changes exist (N/A)
  • Documentation reflects the changes
  • If you provide code modification, please add yourself to CONTRIBUTORS.txt (N/A)
  • Add a new news fragment into the CHANGES/ folder

Copy link

codspeed-hq bot commented Jan 7, 2025

CodSpeed Performance Report

Merging #10305 will degrade performances by 9.51%

Comparing patchback/backports/3.11/b6ffb1d1a3b710c600f06c9d21cf62f75d324767/pr-10304 (780bd92) with 3.11 (0c32bc8)

Summary

⚡ 1 improvements
❌ 1 regressions
✅ 45 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark 3.11 patchback/backports/3.11/b6ffb1d1a3b710c600f06c9d21cf62f75d324767/pr-10304 Change
test_get_request_with_251308_compressed_chunked_payload[pyloop] 464.9 ms 426 ms +9.14%
test_one_hundred_get_requests_with_512kib_chunked_payload[pyloop] 211.7 ms 234 ms -9.51%

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.

1 participant