Skip to content

chore: update dependencies and implement multi-threaded TPCC support #278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

loloxwg
Copy link
Member

@loloxwg loloxwg commented Jul 24, 2025

  • Updated librocksdb-sys to version 0.17.1 and rocksdb to version 0.23.0 in Cargo.toml and Cargo.lock.
  • Refactored RocksStorage to use TransactionDB instead of OptimisticTransactionDB.
  • Enhanced TPCC implementation to support multi-threading, allowing concurrent transaction execution with shared statistics collection.
  • Added detailed comments on the threading strategy and architecture for clarity.

What problem does this PR solve?

Add corresponding issue link with summary if exists -->

Issue link:

What is changed and how it works?

Code changes

  • Has Rust code change
  • Has CI related scripts change

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Note for reviewer

@loloxwg loloxwg force-pushed the tpcc branch 2 times, most recently from aea59fe to 3dd028f Compare July 24, 2025 02:33
- Updated `librocksdb-sys` to version 0.17.1 and `rocksdb` to version 0.23.0 in `Cargo.toml` and `Cargo.lock`.
- Refactored `RocksStorage` to use `TransactionDB` instead of `OptimisticTransactionDB`.
- Enhanced TPCC implementation to support multi-threading, allowing concurrent transaction execution with shared statistics collection.
- Added detailed comments on the threading strategy and architecture for clarity.
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.

1 participant