Closed ValarDragon closed 2 months ago
The recent update involves modifications to the telemetry measurement handling within the Store
struct's methods in the store/iavl/store.go
file. Specifically, the Get
, Has
, and Delete
methods no longer defer telemetry measurements, which might impact the timing and performance data collection.
File Path | Change Summary |
---|---|
store/iavl/store.go |
Removed telemetry measurement defers in Get , Has , Delete methods |
🐇💻✨ Code hops through fields of bits, Changes sown, a new path it knits. In the grove of IAVL trees, Telemetry leaves flutter in the breeze. Each commit, a carrot sweet, Celebrate, for our code is neat!
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
its already removed upstream, nice
Remove timing info from every IAVL get/delete call.
Varies between .5 to 1% speedups to blocksync, and I have never once used this telemetry info.
Summary by CodeRabbit
Get
,Has
, andDelete
operations to enhance performance tracking.