fleece-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hendrik Dev <hendrikde...@gmail.com>
Subject Re: Some suggestions for Strings.java and JsonLongImpl.java
Date Sat, 05 Jul 2014 08:11:22 GMT
The Strings.java JMH based benchmark i did can be found here
https://github.com/salyh/fleece_tmp/tree/benchmark-streamparser/fleece-core

avg_benchmark_jmh_result_f2_t16_w3_i5.txt for example shows

Benchmark Mode Samples Score Score error Units
o.a.f.c.j.b.BenchmarkStringsCompact.normalWithUnicode
avgt        10   117411,047     6748,646    us/op
o.a.f.c.j.b.BenchmarkStringsCompact.normalWithoutUnicode
avgt        10   118481,121    10540,816    us/op
o.a.f.c.j.b.BenchmarkStringsCompact.optimizedWithUnicode
avgt        10   102078,601     7623,861    us/op
o.a.f.c.j.b.BenchmarkStringsCompact.optimizedWithoutUnicode
avgt        10   125038,896     8359,245    us/op

There is no real improvement if i read the figures correctly.

@romain: BTW what do exactly mean with "slow hashcode computation"?
hashcode of what?


Implementations:
https://github.com/salyh/fleece_tmp/blob/benchmark-streamparser/fleece-core/src/main/java/org/apache/fleece/core/Strings_Optimized.java
https://github.com/salyh/fleece_tmp/blob/benchmark-streamparser/fleece-core/src/main/java/org/apache/fleece/core/Strings.java

Thanks
Hendrik

On Sat, Jun 28, 2014 at 11:58 PM, Romain Manni-Bucau
<rmannibucau@gmail.com> wrote:
> Ok,
>
> thanks a lot for the jira + patch (I know it looks too much for 1 word
> but makes obvious you contributed the patch ;)).
>
> Waiting to read the result of your tests ;)
>
>
>
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog: http://rmannibucau.wordpress.com/
> LinkedIn: http://fr.linkedin.com/in/rmannibucau
> Github: https://github.com/rmannibucau
>
>
> 2014-06-28 23:36 GMT+02:00 Hendrik Dev <hendrikdev22@gmail.com>:
>> jira issue opened
>>
>> regarding the figures: i have some measurements on my own and the code
>> shows a slight speedup, but i will look into
>> https://github.com/RichardHightower/json-parsers-benchmark
>>
>> On Sat, Jun 28, 2014 at 11:16 PM, Romain Manni-Bucau
>> <rmannibucau@gmail.com> wrote:
>>> second fix is ok (please open a jira and attach a patch)
>>>
>>> the first one need figures (I used
>>> https://github.com/RichardHightower/json-parsers-benchmark ), last
>>> time I tried such a thing computing hashcode was too slow.
>>>
>>>
>>> Romain Manni-Bucau
>>> Twitter: @rmannibucau
>>> Blog: http://rmannibucau.wordpress.com/
>>> LinkedIn: http://fr.linkedin.com/in/rmannibucau
>>> Github: https://github.com/rmannibucau
>>>
>>>
>>> 2014-06-28 23:01 GMT+02:00 Hendrik Dev <hendrikdev22@gmail.com>:
>>>> only constants in escape() method, unicode cache, clean up of possibly
>>>> unnecessary code
>>>> https://github.com/salyh/fleece_tmp/commit/a8d852bc3beeddc9607af1005a2cfdde2f252230
>>>>
>>>> long is integral
>>>> https://github.com/salyh/fleece_tmp/commit/03dc2df406d171ded8db5199d4618786d29e5298
>>>>
>>>> Thanks
>>>> Hendrik



-- 
Hendrik Saly (salyh, hendrikdev22)
@hendrikdev22
PGP: 0x22D7F6EC

Mime
View raw message