ab4238a0e2
crypto/merkle: optimize merkle tree hashing ( #6513 ) ( #9446 )
...
* crypto/merkle: optimize merkle tree hashing (#6513 )
Upstream https://github.com/lazyledger/lazyledger-core/pull/351 to optimize merkle tree hashing
```
benchmark old ns/op new ns/op delta
BenchmarkHashAlternatives/recursive-8 22914 21949 -4.21%
BenchmarkHashAlternatives/iterative-8 21634 21939 +1.41%
benchmark old allocs new allocs delta
BenchmarkHashAlternatives/recursive-8 398 200 -49.75%
BenchmarkHashAlternatives/iterative-8 399 301 -24.56%
benchmark old bytes new bytes delta
BenchmarkHashAlternatives/recursive-8 19088 6496 -65.97%
BenchmarkHashAlternatives/iterative-8 21776 13984 -35.78%
```
cc @odeke-em @cuonglm
* update pending log
Co-authored-by: Marko <marbar3778@yahoo.com >
2022-09-21 10:18:05 +02:00
Thane Thomson and GitHub
cceea4de22
chore: Format and fix lints ( #9336 )
...
* make format
Signed-off-by: Thane Thomson <connect@thanethomson.com >
* Fix linting directives
Signed-off-by: Thane Thomson <connect@thanethomson.com >
* make mockery
Signed-off-by: Thane Thomson <connect@thanethomson.com >
* Appease CI linter
Signed-off-by: Thane Thomson <connect@thanethomson.com >
* Appease CI linter
Signed-off-by: Thane Thomson <connect@thanethomson.com >
Signed-off-by: Thane Thomson <connect@thanethomson.com >
2022-08-30 12:28:46 -04:00
Erik Grinaker and GitHub
f66b7a8e32
merkle: return hashes for empty merkle trees ( #5193 )
...
Fixes #5192 .
@liamsi Can you verify that the test vectors match the Rust implementation? I updated `ProofsFromByteSlices()` as well, anything else that should be updated?
2020-08-11 10:31:05 +00:00
Marko and GitHub
46f6d17601
crypto/merkle: remove simple prefix ( #4989 )
...
## Description
This PR removes simple prefix from all types in the crypto/merkle directory.
The two proto types `Proof` & `ProofOp` have been moved to the `proto/crypto/merkle` directory.
proto messge `Proof` was renamed to `ProofOps` and `SimpleProof` message to `Proof`.
Closes : #2755
2020-06-10 14:57:38 +00:00