Raw data output
Aleksey Shipilev
aleksey.shipilev at oracle.com
Thu Jul 21 15:49:51 UTC 2016
Hi Petr,
On 07/15/2016 03:35 PM, Petr Stefan wrote:
> I've updated my patch according to your comments. Thanks.
Any news about OCA? This is the thing that blocks us from going forward
with this patch :)
The patch looks OK. I have a few comments though:
a) Utils.loopOver can be simplified;
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.
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.
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?
Thanks,
-Aleksey
More information about the jmh-dev
mailing list