patch psutil virtual memory for test suite in CI #424
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related to #405 / #406
I have a theory that in the Windows CI
psutil.virtual_memory().percent > 80
is being hit often inprocess_udf_outputs
(which is executed under parallel mode). Code is below:datachain/src/datachain/query/dataset.py
Lines 383 to 410 in 424b05b
This would mean that we are concurrently writing to the warehouse more often, giving the UDF far more chance to fail due to SQLite locks.
This PR patches
psutil.virtual_memory
to be automatically pinned at 50% utilised for the entirety of the test suite.