Skip to content

update rocksdb (rocksdb#432) - #855

Merged
ti-chi-bot[bot] merged 1 commit into
tikv:masterfrom
HunDunDM:hundundm/single-sst-pick-2
Jul 27, 2026
Merged

ti-chi-bot[bot] merged 1 commit into
tikv:masterfrom
HunDunDM:hundundm/single-sst-pick-2

Conversation

@HunDunDM

@HunDunDM HunDunDM commented Jul 27, 2026

Copy link
Copy Markdown
Member

Description

Update RocksDB to include tikv/rocksdb#432.

This restores ldb manifest_dump --sst_file_number support, which is required by tikv-ctl bad-ssts to obtain the final live metadata of a corrupted SST file.

Related issue: tikv/tikv#19873

The RocksDB submodule is updated from:

  • 0cbfd4a7866b3a9448fbc2edabc79d55d0d80077

to:

  • 430d3f947e90636b00a2bf7dcc024d693f0e1fc1

No Rust API or FFI changes are required.

Summary by CodeRabbit

  • Chores
    • Updated the underlying database engine components to a newer revision.
    • No user-facing features, configuration changes, or API changes were introduced.

@ti-chi-bot ti-chi-bot Bot added dco-signoff: yes Indicates the PR's author has signed the dco. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 27, 2026
@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 412785d2-8dc0-46a9-a169-68d324950364

📥 Commits

Reviewing files that changed from the base of the PR and between 4333548 and 700220f.

📒 Files selected for processing (1)
  • librocksdb_sys/rocksdb
🚧 Files skipped from review as they are similar to previous changes (1)
  • librocksdb_sys/rocksdb

📝 Walkthrough

Walkthrough

The librocksdb_sys/rocksdb submodule reference changes from commit 430d3f9... to 044e8b9.... No exported or public declarations change.

Changes

RocksDB revision update

Layer / File(s) Summary
Update RocksDB submodule pointer
librocksdb_sys/rocksdb
The pinned RocksDB commit changes from 430d3f947e90636b00a2bf7dcc024d693f0e1fc1 to 044e8b96e6a8422985e837a721a202bf669604b3.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: connor1996, v01dstar

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main change: updating the RocksDB submodule to a specific upstream revision.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: HunDunDM <hundundm@gmail.com>
@HunDunDM
HunDunDM force-pushed the hundundm/single-sst-pick-2 branch from 4333548 to 700220f Compare July 27, 2026 10:53
@ti-chi-bot ti-chi-bot Bot added the lgtm label Jul 27, 2026
@ti-chi-bot

ti-chi-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: LykxSassinator

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot

ti-chi-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

[LGTM Timeline notifier]

Timeline:

  • 2026-07-27 10:55:14.247601765 +0000 UTC m=+1834300.283696821: ☑️ agreed by LykxSassinator.

@ti-chi-bot ti-chi-bot Bot added the approved label Jul 27, 2026
@ti-chi-bot
ti-chi-bot Bot merged commit cdb577c into tikv:master Jul 27, 2026
9 checks passed
@HunDunDM
HunDunDM deleted the hundundm/single-sst-pick-2 branch July 27, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dco-signoff: yes Indicates the PR's author has signed the dco. lgtm size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants