Skip to content

perf(server): reuse HTTP session across requests - #203

Merged
mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:perf/server
Jun 26, 2026
Merged

mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:perf/server

Conversation

@mdevolde

Copy link
Copy Markdown
Collaborator

perf(server): reuse HTTP session across requests

Why the pull request was made

To improve perfs if a user needs to perform multiple check requests in a row.

Summary of changes

  • Init an HTTP session at server init, and reuse it for check requests.

Screenshots (if appropriate):

Not applicable.

How has this been tested?

Applied local tests.

Resources

Not applicable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (changes to documentation only)
  • Refactor / code style update (non-breaking change that improves code structure or readability)
  • Tests / CI improvement (adding or updating tests or CI configuration only)
  • Chore / maintenance (non-breaking change that does not affect functionality, such as updating dependencies or fixing typos)
  • Other (please describe): performance

Checklist

  • Followed the project's contributing guidelines.
  • Updated any relevant tests.
  • Updated any relevant documentation.
  • Added comments to your code where necessary.
  • Formatted your code, run the linters, checked types and tests.
  • Added your changes to the CHANGELOG file, if applicable.

@mdevolde mdevolde self-assigned this Jun 26, 2026
@codecov-commenter

Copy link
Copy Markdown

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

Codecov Report

❌ Patch coverage is 66.66667% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.70%. Comparing base (cf0cf4c) to head (8190c5c).

Files with missing lines Patch % Lines
src/language_tool_python/server.py 66.66% 4 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   80.83%   80.70%   -0.13%     
==========================================
  Files          13       13              
  Lines        1758     1767       +9     
==========================================
+ Hits         1421     1426       +5     
- Misses        337      341       +4     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mdevolde
mdevolde merged commit 2e9f9c8 into jxmorris12:master Jun 26, 2026
13 checks passed
@mdevolde
mdevolde deleted the perf/server branch June 26, 2026 12:19
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.

2 participants