Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

feat: switch to borc #31

Merged
merged 1 commit into from
Dec 11, 2016
Merged

feat: switch to borc #31

merged 1 commit into from
Dec 11, 2016

Conversation

dignifiedquire
Copy link
Member

This makes cbor encoding and decoding a lot faster.

Fixes #23

cc @parkan @diasdavid

Latest benchmarks on node 7.2.1

encode - node-cbor - 74 x 312 ops/sec ±5.13% (52 runs sampled)
encode - borc - 74 x 9,572 ops/sec ±2.85% (88 runs sampled)

decode - node-cbor - 45 x 260 ops/sec ±7.55% (52 runs sampled)
decode - borc - 45 x 8,485 ops/sec ±6.81% (72 runs sampled)

@dignifiedquire
feat: switch to borc
209a8cb 
This makes cbor encoding and decoding a lot faster.

Fixes #23
@daviddias daviddias merged commit 3164a81 into master Dec 11, 2016
@daviddias daviddias deleted the borc branch December 11, 2016 16:50
@daviddias daviddias removed the status/in-progress In progress label Dec 11, 2016
@daviddias
Copy link
Member

Thank you @dignifiedquire <3

@parkan
Copy link

parkan commented Dec 14, 2016

swedish-chef-psd71677

@parkan
Copy link

parkan commented Dec 14, 2016

I can report that this has given us at least 2x overall improvement in our ingestion pipeline, nice work!

@dignifiedquire
Copy link
Member Author

awesome :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@daviddias daviddias daviddias approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dignifiedquire @daviddias @parkan