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

Parse and serialize circe AST using jsoniter-scala #285

Closed
wants to merge 1 commit into from

Conversation

plokhotnyuk
Copy link
Owner

@plokhotnyuk plokhotnyuk commented May 7, 2019

Results of benchmarks:

[info] REMEMBER: The numbers below are just data. To gain reusable insights, you need to follow up on
[info] why the numbers are the way they are. Use profilers (see -prof, -lprof), design factorial
[info] experiments, perform baseline and negative tests that provide experimental control, make sure
[info] the benchmarking environment is safe on JVM/OS/HW level, ask for reviews from the domain experts.
[info] Do not assume the numbers tell you what you want them to tell.
[info] Benchmark                                   Mode  Cnt      Score      Error  Units
[info] GeoJSONReading.circe                       thrpt    5   6434.510 ±  387.368  ops/s
[info] GeoJSONReading.circeJsoniter               thrpt    5   6503.791 ±  244.144  ops/s
[info] GeoJSONReading.jsoniterScala               thrpt    5  38970.101 ±  319.176  ops/s
[info] GeoJSONWriting.circe                       thrpt    5   4044.570 ±  356.425  ops/s
[info] GeoJSONWriting.circeJsoniter               thrpt    5   6216.887 ±  468.054  ops/s
[info] GeoJSONWriting.jsoniterScala               thrpt    5  13440.258 ±  108.046  ops/s
[info] GoogleMapsAPIPrettyPrinting.circe          thrpt    5   7556.502 ±  949.123  ops/s
[info] GoogleMapsAPIPrettyPrinting.circeJsoniter  thrpt    5   8787.719 ±  137.656  ops/s
[info] GoogleMapsAPIPrettyPrinting.jsoniterScala  thrpt    5  48592.752 ± 2897.298  ops/s
[info] GoogleMapsAPIReading.circe                 thrpt    5   8258.472 ±  416.014  ops/s
[info] GoogleMapsAPIReading.circeJsoniter         thrpt    5   9435.155 ±  371.863  ops/s
[info] GoogleMapsAPIReading.jsoniterScala         thrpt    5  31316.494 ±  439.374  ops/s
[info] GoogleMapsAPIWriting.circe                 thrpt    5   8031.358 ±  178.514  ops/s
[info] GoogleMapsAPIWriting.circeJsoniter         thrpt    5   9392.853 ± 1066.635  ops/s
[info] GoogleMapsAPIWriting.jsoniterScala         thrpt    5  91760.728 ± 3033.358  ops/s
[info] TwitterAPIReading.circe                    thrpt    5  12922.692 ±  170.774  ops/s
[info] TwitterAPIReading.circeJsoniter            thrpt    5  17641.198 ±  383.072  ops/s
[info] TwitterAPIReading.jsoniterScala            thrpt    5  41987.413 ± 2753.776  ops/s

@plokhotnyuk plokhotnyuk force-pushed the circe-jsoniter branch 3 times, most recently from 7093516 to 853046c Compare May 8, 2019 03:48
@codecov
Copy link

codecov bot commented May 8, 2019

Codecov Report

Merging #285 into master will decrease coverage by 0.05%.
The diff coverage is 95.52%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
- Coverage     100%   99.94%   -0.06%     
==========================================
  Files         176      177       +1     
  Lines        5096     5163      +67     
  Branches      492      494       +2     
==========================================
+ Hits         5096     5160      +64     
- Misses          0        3       +3
Impacted Files Coverage Δ
...nyuk/jsoniter_scala/benchmark/GeoJSONWriting.scala 100% <100%> (ø) ⬆️
...k/jsoniter_scala/benchmark/TwitterAPIReading.scala 100% <100%> (ø) ⬆️
...nyuk/jsoniter_scala/benchmark/GeoJSONReading.scala 100% <100%> (ø) ⬆️
...soniter_scala/benchmark/GoogleMapsAPIWriting.scala 100% <100%> (ø) ⬆️
...soniter_scala/benchmark/GoogleMapsAPIReading.scala 100% <100%> (ø) ⬆️
..._scala/benchmark/GoogleMapsAPIPrettyPrinting.scala 100% <100%> (ø) ⬆️
...chmark/src/main/scala/io/circe/CirceJsoniter.scala 94.82% <94.82%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f69b4c0...e80d510. Read the comment docs.

plokhotnyuk referenced this pull request in sirthias/borer May 11, 2019
@plokhotnyuk plokhotnyuk changed the title Parse and serialize using circe with jsoniter-scala Parse and serialize circe AST using jsoniter-scala Jun 27, 2019
@plokhotnyuk plokhotnyuk force-pushed the master branch 3 times, most recently from 8ba15e7 to 795401c Compare July 28, 2019 15:50
@plokhotnyuk plokhotnyuk deleted the circe-jsoniter branch March 19, 2020 11:25
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.

None yet

1 participant