CosmWasm / CosmWasm/cw-storage-plus

Benchmark integer `Key` performance

Open
#7 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
51
Forks
34
PR merge metrics
No merged PRs in 30d

Description

In https://github.com/CosmWasm/cw-plus/pull/694 support for 128 bits integer keys was added.

There are some concerns with maximum integer key length, as [it impacts all the int types](https://github.com/CosmWasm/cw-plus/pull/547#discussion_r749359057).

Confirm this doesn't impact performance of reading / writing int keys to storage. It may be tricky to test / benchmark this, as code changes will be needed, to have a `Key` implementation with and without `u128` support. This can be a one time benchmark, in a branch that we end up not merging to main, by example.

Contributor guide

No contributing guide indexed for this repository

Research direction

Review PR 694 and the PR 547 discussion first, then locate the storage read and write paths for integer keys. Compare benchmarks for Key implementations with and without u128 support; done means reporting whether maximum key length affects integer-key storage performance.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
databases, performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.