Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Dec 6, 2024

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.1)

Can you help keep this open source service alive? 💖 Please sponsor : )

bdraco and others added 24 commits December 1, 2024 23:44
…a in the stream reader twice (#10090)

Co-authored-by: J. Nick Koston <nick@koston.org>
…a in the stream reader twice (#10091)

Co-authored-by: J. Nick Koston <nick@koston.org>
…base url (#10094)

**This is a backport of PR #10074 as merged into master
(f733258).**
Co-authored-by: vivodi <103735539+vivodi@users.noreply.github.com>
…nse (#10098)

**This is a backport of PR #10095 as merged into master
(fcce1bf).**

We didn't have any benchmarks for file responses. From the benchmarks it
turns out most of the time is creating and processing the executor jobs.

If we combine the stat into a job that returns the open fileobj it will
likely be faster and solve
#8013

Co-authored-by: J. Nick Koston <nick@koston.org>
…nse (#10097)

**This is a backport of PR #10095 as merged into master
(fcce1bf).**

We didn't have any benchmarks for file responses. From the benchmarks it
turns out most of the time is creating and processing the executor jobs.

If we combine the stat into a job that returns the open fileobj it will
likely be faster and solve
#8013

Co-authored-by: J. Nick Koston <nick@koston.org>
…pe` for paths with Python 3.13 (#10103)

Co-authored-by: J. Nick Koston <nick@koston.org>
…pe` for paths with Python 3.13 (#10104)

Co-authored-by: J. Nick Koston <nick@koston.org>
…e is replaced during `prepare` (#10105)

Co-authored-by: J. Nick Koston <nick@koston.org>
fixes #8013
…e is replaced during `prepare` (#10106)

Co-authored-by: J. Nick Koston <nick@koston.org>
fixes #8013
…in `FileResponse` (#10108)

Co-authored-by: J. Nick Koston <nick@koston.org>
…in `FileResponse` (#10109)

Co-authored-by: J. Nick Koston <nick@koston.org>
…`StreamWriter`` (#10126)

Co-authored-by: J. Nick Koston <nick@koston.org>
…`StreamWriter`` (#10127)

Co-authored-by: J. Nick Koston <nick@koston.org>
@pull pull bot added the ⤵️ pull label Dec 6, 2024
@pull pull bot merged commit 7f92beb into tj-python:master Dec 6, 2024
@codecov-commenter
Copy link

codecov-commenter commented Dec 6, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (d58d2c3) to head (7f92beb).
Report is 24 commits behind head on master.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   98.74%   98.74%           
=======================================
  Files         122      122           
  Lines       36956    36956           
  Branches     4411     4411           
=======================================
  Hits        36494    36494           
  Misses        315      315           
  Partials      147      147           
Flag Coverage Δ
CI-GHA 98.63% <ø> (ø)
OS-Linux 98.32% <ø> (ø)
OS-Windows 96.17% <ø> (ø)
OS-macOS 97.43% <ø> (ø)
Py-3.10.11 97.27% <ø> (ø)
Py-3.10.15 97.81% <ø> (ø)
Py-3.11.10 97.84% <ø> (-0.01%) ⬇️
Py-3.11.9 97.32% <ø> (+<0.01%) ⬆️
Py-3.12.7 98.37% <ø> (ø)
Py-3.13.0 97.18% <ø> (-0.01%) ⬇️
Py-3.13.1 98.34% <ø> (+0.05%) ⬆️
Py-3.9.13 97.19% <ø> (ø)
Py-3.9.20 97.73% <ø> (-0.05%) ⬇️
Py-pypy7.3.16 97.34% <ø> (ø)
VM-macos 97.43% <ø> (ø)
VM-ubuntu 98.32% <ø> (ø)
VM-windows 96.17% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants