Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Parse simulation logs using parallel stream #24

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

luvarqpp
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jun 18, 2020

CLA assistant check
All committers have signed the CLA.

@luvarqpp
Copy link
Contributor Author

Sample difference for small simulation can be seen only in order of logs. Note that other two pull request made by me was included in this build.

➜  wrk java -jar gatling-report-4.1-SNAPSHOT-capsule-fat.jar -f results/complexscenario-20200618125705159/simulation.log results/complexscenario-20200618163115911/simulation.log -o comparison/jedenVersusDvajaSync
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by Capsule (file:gatling-report-4.1-SNAPSHOT-capsule-fat.jar) to field com.sun.jmx.mbeanserver.JmxMBeanServer.mbsInterceptor
WARNING: Please consider reporting this to the maintainers of Capsule
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
22:29:37,326 INFO  [App] Parsing /someDir/results/complexscenario-20200618125705159/simulation.log
22:29:37,326 INFO  [App] Parsing /someDir/results/complexscenario-20200618163115911/simulation.log
22:29:37,974 INFO  [App] Parsing finished in 641 ms. File /someDir/results/complexscenario-20200618163115911/simulation.log
22:29:38,012 INFO  [App] Parsing finished in 682 ms. File /someDir/results/complexscenario-20200618125705159/simulation.log
22:29:38,026 WARN  [App] Overriding existing report directorycomparison/jedenVersusDvajaSync
22:29:38,426 INFO  [App] Report generated: /someDir/comparison/jedenVersusDvajaSync/index.html


➜  wrk java -jar gatling-report-4.1-SNAPSHOT-capsule-fat.jar -f results/complexscenario-20200618125705159/simulation.log results/complexscenario-20200618163115911/simulation.log -o comparison/jedenVersusDvajaSync
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by Capsule (file:gatling-report-4.1-SNAPSHOT-capsule-fat.jar) to field com.sun.jmx.mbeanserver.JmxMBeanServer.mbsInterceptor
WARNING: Please consider reporting this to the maintainers of Capsule
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
22:30:03,014 INFO  [App] Parsing /someDir/results/complexscenario-20200618125705159/simulation.log
22:30:03,563 INFO  [App] Parsing finished in 546 ms. File /someDir/results/complexscenario-20200618125705159/simulation.log
22:30:03,563 INFO  [App] Parsing /someDir/results/complexscenario-20200618163115911/simulation.log
22:30:03,869 INFO  [App] Parsing finished in 304 ms. File /someDir/results/complexscenario-20200618163115911/simulation.log
22:30:03,869 WARN  [App] Overriding existing report directorycomparison/jedenVersusDvajaSync
22:30:04,158 INFO  [App] Report generated: /someDir/comparison/jedenVersusDvajaSync/index.html

@luvarqpp luvarqpp changed the title Parse simulation logs using parallel stream WIP: Parse simulation logs using parallel stream Jun 23, 2020
@luvarqpp
Copy link
Contributor Author

DO NOT MERGE yet. There is bug in current PR. Parsed data from options.simulations is being saved during processing into App.stats in order as they are parsed and not in order as they appear on commandline. I will fix this problem in near time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants