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

Stress-test to detect any memory leaks etc. #60

Closed
edsko opened this issue Jan 20, 2024 · 3 comments
Closed

Stress-test to detect any memory leaks etc. #60

edsko opened this issue Jan 20, 2024 · 3 comments
Assignees
Labels
performance Performance related issues priority: medium Should be done before the library can be considered complete testsuite Problem with the test suite but not the main library

Comments

@edsko
Copy link
Collaborator

edsko commented Jan 20, 2024

The main test suite is concerned with behaviour, not resource usage. We need to have a serious stress test, making thousands and thousands of connections, and thousands and thousands of calls on each connection, and verify that no resources get leaked. Might perhaps be useful to limit the heap and the stack size when we run these tests.

May require some changes to http2; see discussion in kazu-yamamoto/http2#78 .

@edsko edsko added testsuite Problem with the test suite but not the main library priority: high Usability of the library severely hindered labels Jan 20, 2024
@edsko edsko added priority: medium Should be done before the library can be considered complete and removed priority: high Usability of the library severely hindered labels Mar 29, 2024
@FinleyMcIlwaine FinleyMcIlwaine added the performance Performance related issues label Apr 10, 2024
@FinleyMcIlwaine
Copy link
Collaborator

Implemented part of this at #134, still need to implement the many-messages stress test. We also should stress test a situation where many reconnections happen.

@edsko
Copy link
Collaborator Author

edsko commented Jun 26, 2024

We also need to add a test to reproduce #169 here (i.e., many concurrent calls).

@edsko
Copy link
Collaborator Author

edsko commented Jun 26, 2024

When we close this issue, there are a few references to this ticket in the codebase that we should check and, if indeed handled, delete.

@edsko edsko closed this as completed Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance Performance related issues priority: medium Should be done before the library can be considered complete testsuite Problem with the test suite but not the main library
Projects
None yet
Development

No branches or pull requests

2 participants