rooch-network / rooch

VApp Container with Move Language for Bitcoin ecosystem
https://rooch.network
Apache License 2.0
162 stars 85 forks source link

[smt] Rename hashtag to SMTNode and replace the HashValue with SMTNodeHash #2536

Closed jolestar closed 2 months ago

jolestar commented 2 months ago

Summary

  1. Rename the hashtag from rooch-smt to SMTNode because smt may be used in a different project than rooch.
  2. Replace the HashValue with SMTNodeHash.
  3. Improve the Merkle hash calculation.

Part of #2402

Do a simple benchmark:

  1. Before #2401
Screenshot 2024-08-29 at 14 14 59
  1. After #2401
Screenshot 2024-08-29 at 14 19 21
  1. this PR
Screenshot 2024-08-29 at 14 23 18

The db performance is regressed; maybe it is caused by disk usage.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rooch-portal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2024 8:24am
rooch-portal-v2.1 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 29, 2024 8:24am
1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **rooch** | ⬜️ Ignored ([Inspect](https://vercel.com/rooch/rooch/Db9qnjdPtSJZtaK3NvseQNmJvQ6p)) | [Visit Preview](https://rooch-git-refactorsmthash-rooch.vercel.app) | | Aug 29, 2024 8:24am |
github-actions[bot] commented 2 months ago

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files