Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

Remove verbose GC logging to file for Java/wildfly-ee7#2661

Merged
NateBrady23 merged 1 commit intoTechEmpower:masterfrom
zloster:wildfly-verbose-gc
Apr 5, 2017
Merged

Remove verbose GC logging to file for Java/wildfly-ee7#2661
NateBrady23 merged 1 commit intoTechEmpower:masterfrom
zloster:wildfly-verbose-gc

Conversation

@zloster
Copy link
Copy Markdown
Contributor

@zloster zloster commented Apr 4, 2017

This logging doesn't follow the project recommendation about the logs and IMO is affecting the benchmark results of Wildfly. But this gave me an idea about a new test type: which framework can fill a SSD disk faster. Or which framework logs are the biggest for given test run :)

@mention-bot
Copy link
Copy Markdown

Thanks @zloster for contributing to The Framework Benchmarks! @msmith-techempower, @nbrady-techempower and @codylerum, code you've worked on has been modified. If you have the chance, please review. If you wish to unsubscribe from these notices, please open a Pull Request with the commit message [ci skip] and your github name added to the userBlacklist array in the .mention-bot file.

@codylerum
Copy link
Copy Markdown
Contributor

👍

@NateBrady23 NateBrady23 merged commit 9578f68 into TechEmpower:master Apr 5, 2017
@zloster
Copy link
Copy Markdown
Contributor Author

zloster commented Apr 21, 2017

No better results: https://www.techempower.com/benchmarks/previews/round14/r14p3-vs-r14p4.html#wildfly-ee7.json Just the normal variation for the test results.
So the detailed garbage collection logging is basically free for this setup and conditions.

@codylerum
Copy link
Copy Markdown
Contributor

@zloster I'm guessing their are gains to be made by tweaking the thread settings. I believe the environment is 40 cores now and I don't know how many wildfly is configured to use by default.

@NateBrady23
Copy link
Copy Markdown
Member

@codylerum the server environment is 80 cores, and you can access that via the CPU_COUNT env variable if you need to.

@zloster zloster deleted the wildfly-verbose-gc branch June 19, 2017 19:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants