Printing of batchsize + single execution time in Results for Batching

[Huber Matthias]

Hello Aleksey,

attached is the patch that prints the batchSize in JSON output. As expected a trivial change. I hope that the form is correct. This is my first patch for an OpenJDK project and I am new to Mercurial.

So far, I respectively the person responsible from my company haven't got information if the OCA is already signed. It has been send to Oracle at the 1th of February.


Mit freundlichen Grüßen / Kind Regards

Matthias Huber
Senior Consultant

CA Application Performance Management

NOVATEC // Consulting GmbH
Dieselstraße 18/1  //  D-70771 Leinfelden-Echterdingen

phone:  +49 711 22040-700
fax:    +49 711 22040-899
mobile: +49 170 5682609

e-mail: matthias.huber at novatec-gmbh.de
web:    www.novatec-gmbh.de

Sitz der Gesellschaft: Waldenbuch // Handelsregister: Stuttgart – HRB 739078
Geschäftsführer: Stefan Bleicher // Hans-Dieter Brenner // Konrad Pfeilsticker // Michael Schuchart

inspectIT  //  freies Java Performance und Monitoringwerkzeug (www.inspectit.rocks)

________________________________________
From: Huber Matthias
Sent: 29 January 2016 16:22
To: Aleksey Shipilev; jmh-dev at openjdk.java.net
Subject: Re: Printing of batchsize + single execution time in Results for       Batching

Hello Aleksey,

I'm short before submitting the OCA. However, I'm not sure about the correct project name. Is it "OpenJDK", "JMH" or something else.


Kind Regards

Matthias Huber

________________________________________
From: jmh-dev <jmh-dev-bounces at openjdk.java.net> on behalf of Huber Matthias <Matthias.Huber at novatec-gmbh.de>
Sent: 11 January 2016 17:06
To: Aleksey Shipilev; jmh-dev at openjdk.java.net
Subject: Re: Printing of batchsize + single execution time in Results for       Batching

Hello Aleksey,

I see your point with breaking the XSV format and its impact. I will have a look at the Java API to deal with this corner case.

Right now I think that only the batch size is interesting for me to distinguish the results based on the used batch size from each other. Using the calculated time for one execution (total avg time per batch / batch size) could be dangerous for comparison. Comparing two values with each other that are calculated by dividing an average time by different batch sizes feels incorrect to me.

On 01/11/2016 01:47 PM, Shipilev Aleksey wrote:
< But, I think we can extend JSON output to include more benchmark
< metadata, including batch sizes. R should be able to pull in JSON dataset?

I haven't used the JSON DataSet so far. It's worth to have a look at it and its interoperability with R. I might come back with a contribution.

Thanks,
Matthias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jsonBatchSize.diff
Type: text/x-patch
Size: 2702 bytes
Desc: jsonBatchSize.diff
URL: <http://mail.openjdk.java.net/pipermail/jmh-dev/attachments/20160219/af481732/jsonBatchSize.diff>