ashdnazg opened a new pull request, #15702: URL: https://github.com/apache/datafusion/pull/15702
## Rationale for this change Currently the memory limit tests hang due to lock contention and the overhead from frequently calling `sysinfo::System::refresh_all()` in the memory monitoring task. In fact, we don't really need to refresh all info, it's enough to just get the memory data about the current process, which significantly reduces the refresh time and the tests finish quickly. ## What changes are included in this PR? In the memory limit utils, the memory monitoring spawned task now refreshes system info only for what's actually necessary. ## Are these changes tested? It's part of the tests, and they seem to not hang anymore :) ## Are there any user-facing changes? No -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
