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

Java time instances #84

Merged
merged 6 commits into from Jul 13, 2019

Conversation

@AdrianRaFo
Copy link
Contributor

commented Jul 8, 2019

Fixes #78
Add java time support on Encoding and Decoding based on its String representation with a custom or default formatter.

Also, fix the current implemented java instances for UUID, BigDecimal and so on

@AdrianRaFo AdrianRaFo requested review from fedefernandez and nomisRev Jul 8, 2019

@47degdev

This comment has been minimized.

Copy link
Contributor

commented Jul 8, 2019

Hood benchmark comparison:
✔️ Decoding (Threshold: 71.38222838181042)

Benchmark Value
master_benchmark 11005.660930868926
helios_benchmark 11176.454821677815

✔️ Decodingfromraw (Threshold: 250.0)

Benchmark Value
master_benchmark 7230.701971317787
helios_benchmark 7231.986973187108

✔️ Parsing (Threshold: 250.0)

Benchmark Value
master_benchmark 22114.930408176682
helios_benchmark 22586.734391453327

@AdrianRaFo AdrianRaFo added this to In progress in Release 0.2.0 Jul 9, 2019

@AdrianRaFo AdrianRaFo merged commit 520ed1d into master Jul 13, 2019

3 checks passed

benchmark-ci/hood Benchmarks comparison passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

Release 0.2.0 automation moved this from In progress to Done Jul 13, 2019

@AdrianRaFo AdrianRaFo deleted the arf-java-instances branch Jul 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.