[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #410: [HTTPCORE-747]: add information about the minimum requirement for stable test execution

["ok2c (via GitHub)" <gi...@apache.org>]

ok2c commented on code in PR #410:
URL: https://github.com/apache/httpcomponents-core/pull/410#discussion_r1236616402


##########
BUILDING.txt:
##########
@@ -14,10 +14,17 @@ http://maven.apache.org/run-maven/index.html
 
 (2) Executing test cases
 
-Execute the following command in order to compile and test the components
+Execute the following command in order to compile and test the components:
 
 mvn test
 
+To avoid intermittent test failures (i.e., flaky tests), it is recommended to have a machine or virtual machine with the following specifications:
+
+Minimum of 2 CPUs.
+Minimum of 2GB of RAM.
+
+By meeting these requirements, you can mitigate potential flaky tests and ensure a smooth testing experience.
+

Review Comment:
   @Agorguy Can we rephrase this sentence to make it sound a bit less frivolous and avoid the term "flaky"
   
   ```
   These are the minimal requirements for stable test execution. In more resource constrained environments some tests may fail intermittently
   ```
   or some such? 
   
   Do tests _really_ need 2 CPUs?



-- 
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: dev-unsubscribe@hc.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org