Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Add benchmarks #15

Merged
merged 2 commits into from
Oct 6, 2019
Merged

Add benchmarks #15

merged 2 commits into from
Oct 6, 2019

Conversation

Kubuxu
Copy link
Member

@Kubuxu Kubuxu commented Oct 5, 2019

No description provided.

Jakub Sztandera added 2 commits October 5, 2019 21:50
Add benchmarks
cd78345 
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
Fix pool usage in benchmark
52ca04e 
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protocol.ai>
@Kubuxu Kubuxu mentioned this pull request Oct 5, 2019
Closed
@Kubuxu
Copy link
Member Author

Kubuxu commented Oct 5, 2019 

BenchmarkRabin-4     	      12	  95914912 ns/op	 174.92 MB/s	19145327 B/op	     186 allocs/op
BenchmarkDefault-4   	     672	   1721040 ns/op	9748.30 MB/s	     487 B/op	       2 allocs/op

Stebalien
Stebalien approved these changes Oct 6, 2019
View reviewed changes
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note: rabbin is slow because the implementation is slow, not because the algorithm is slow.

@Kubuxu
Copy link
Member Author

Kubuxu commented Oct 6, 2019

I know.

@Kubuxu Kubuxu merged commit 9a794d0 into master Oct 6, 2019
@ribasushi ribasushi deleted the feat/benchmarks branch March 26, 2020 09:21
Jorropo pushed a commit to ipfs/go-libipfs-rapide that referenced this pull request Mar 23, 2023
@Kubuxu
Merge pull request ipfs/go-ipfs-chunker#15 from ipfs/feat/benchmarks
0e9f3c9 
Add benchmarks

This commit was moved from ipfs/go-ipfs-chunker@9a794d0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Stebalien Stebalien Stebalien 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.
2 participants
@Kubuxu @Stebalien