Raw data output

Petr Stefan ptr.stef at gmail.com
Thu Jul 21 16:35:07 UTC 2016


Hi Aleksey,

Dne 21.7.2016 v 17:49 Aleksey Shipilev napsal(a):
> Any news about OCA? This is the thing that blocks us from going forward
> with this patch :)

No, I haven't heard from them yet. I can send you scan of the signed 
document, but I think before they approve it's not valid.


>   b) There is no need to emit empty "rawDataHistogram". If some data is
> not available, it's best to avoid printing out special values, like
> empty array.

OK, I don't need it. My idea was to keep symetry, like empty 
secondaryMetrics when you don't use any profiler. But you have a point, 
better keep it simple :).


>   c) The grouping is incorrect: it adds up histograms from different
> iterations. This happens because the code reuses the old loop that
> prints out *scalar* per each iteration. But in histogram case, you have
> to wrap it up one more time.

Now I got it, you're right. My mistake, sorry.


> Hopefully fixed all three here:
>    http://cr.openjdk.java.net/~shade/jmh/rawjson-1.patch
>
> Can you look into it, and verify it still works as intended?

Yes, from my point it's perfect. Thank you very much.

Cheers,
Petr



More information about the jmh-dev mailing list