Skip to content

[Do Not Merge] chore: Changed to RocksDB#833

Draft
Klapeyron wants to merge 5 commits intomainfrom
test/rocksdb
Draft

[Do Not Merge] chore: Changed to RocksDB#833
Klapeyron wants to merge 5 commits intomainfrom
test/rocksdb

Conversation

@Klapeyron
Copy link
Contributor

Overview

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

kics-logo

KICS version: v2.1.19

Category Results
CRITICAL CRITICAL 0
HIGH HIGH 0
MEDIUM MEDIUM 47
LOW LOW 3
INFO INFO 59
TRACE TRACE 0
TOTAL TOTAL 109
Metric Values
Files scanned placeholder 26
Files parsed placeholder 26
Files failed to scan placeholder 0
Total executed queries placeholder 73
Queries failed to execute placeholder 0
Execution time placeholder 11

@gilescope
Copy link
Contributor

Could do with some reasoning behind why to select a less performant database with native dependencies that is harder to install for users.

@Klapeyron
Copy link
Contributor Author

draft testing changes, I want to decouple db used by blockchain with db used by ledger, easier to track memory allocations

@gilescope gilescope changed the title chore: Changed to RocksDB [Do Not Merge] chore: Changed to RocksDB Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants