You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/04/24 15:12:34 UTC

[GitHub] [activemq-artemis] CAIWan1998 opened a new pull request, #4042: How about add more Forked VM when run all tests in build

CAIWan1998 opened a new pull request, #4042:
URL: https://github.com/apache/activemq-artemis/pull/4042

   
   Maven will run all tests in a single forked VM by default. This can be problematic if there are a lot of tests or some very memory-hungry ones. We can fork more test VM by setting `<fork>1.5C</fork>`.
   
   =====================
   If there are any inappropriate modifications in this PR, please give me a reply and I will change them.
   


-- 
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: gitbox-unsubscribe@activemq.apache.org

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

[GitHub] [activemq-artemis] jbertram closed pull request #4042: How about add more Forked VM when run all tests in build

Posted by GitBox <gi...@apache.org>.

jbertram closed pull request #4042: How about add more Forked VM when run all tests in build
URL: https://github.com/apache/activemq-artemis/pull/4042


-- 
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: gitbox-unsubscribe@activemq.apache.org

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

[GitHub] [activemq-artemis] gemmellr commented on pull request #4042: How about add more Forked VM when run all tests in build

Posted by GitBox <gi...@apache.org>.

gemmellr commented on PR #4042:
URL: https://github.com/apache/activemq-artemis/pull/4042#issuecomment-1108365309

   This wont work for many of the Artemis tests without significant reworking of the overall test suite, as they often do things like reuse specific ports and directories for output and other things like that which simply cant work for parallel runs.
   
   The PR run on Travis did pass, though probably only as it doesnt actually run most of the tests in the build, only a small subset. The run was also a few minutes slower than earlier runs without the change, though its admittedly hard to say how much of that was just run to run variability vs e.g. the change itself adding additional overhead in the low resource env (e.g as discussed on your qpid-jms PR).


-- 
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: gitbox-unsubscribe@activemq.apache.org

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

[GitHub] [activemq-artemis] jbertram commented on pull request #4042: How about add more Forked VM when run all tests in build

Posted by GitBox <gi...@apache.org>.

jbertram commented on PR #4042:
URL: https://github.com/apache/activemq-artemis/pull/4042#issuecomment-1108832233

   From what I can tell this is some kind of automated PR from @CAIWan1998. Looking at their account details you can see that almost 70 similar PRs were sent yesterday to various projects.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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

[GitHub] [activemq-artemis] clebertsuconic commented on pull request #4042: How about add more Forked VM when run all tests in build

Posted by GitBox <gi...@apache.org>.

clebertsuconic commented on PR #4042:
URL: https://github.com/apache/activemq-artemis/pull/4042#issuecomment-1108507120

   many tests will use a specific port while accepting. We would need some significant work as Robbie said.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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