Yavdhesh commented on PR #6855: URL: https://github.com/apache/jmeter/pull/6855#issuecomment-6856657389
@orimarko Thank you for testing this on Windows 11 and sharing the detailed results! Your numbers line up with what I measured: throughput is essentially unchanged (619.8 vs 619.2 req/s in your run), while JMeter's OS thread count dropped from 540 to 62. That thread and memory saving is the point of the change, since user threads spend most of their time blocked on I/O. A short recap of the design for reviewers: - **Opt-in only** — `jmeterthread.use_virtual_threads=true`, off by default, so existing behaviour is unchanged - **Java 17 safe** — falls back to platform threads with a one-time warning when virtual threads aren't available - **One dedicated thread per user**, never pooled, so ThreadLocal-based context handling is unchanged Happy to make adjustments based on feedback from you and the other maintainers. Please let me know if anything else would help with the review. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
