Skip to content
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

Upgrade RocksDB #319

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Upgrade RocksDB #319

merged 1 commit into from
Oct 8, 2024

Conversation

v01dstar
Copy link
Contributor

@v01dstar v01dstar commented May 31, 2024

  1. RocksDB's Cache has been moved to a different place with API changes. Now cache_->Insert() requires a kBlobFileReaderCacheItemHelper instead of a delete function callback solely.
  2. Titan nows uses kTypeTitanColumnFamilyBlobIndex and kTypeTitanBlobIndex to separate from RocksDB's native blob implementation's kTypeColumnFamilyBlobIndexand kTypeBlobIndex. Writing to RocksDB now uses PutTitanBlobIndex() instead of PutBlobIndex(). see Titan followup changes rocksdb#390
  3. IO priority now is intergrated inside ReadOptions, so Titan no longer need to propagate for_compaction to lower level readers to indicates its low priority for IO.
  4. Deprecated seqno in CompactionFilter. Since it is no longer used in TiKV

@v01dstar v01dstar changed the title Fix compatibility Upgrade RocksDB Jun 6, 2024
@v01dstar v01dstar force-pushed the upgrade-rocksdb branch 3 times, most recently from d0a89ec to 6b8c4ed Compare October 5, 2024 07:10
@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 5, 2024

/run-all-tests

@v01dstar v01dstar marked this pull request as ready for review October 6, 2024 05:24
Signed-off-by: Yang Zhang <[email protected]>
@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 8, 2024

/run-all-tests

@v01dstar v01dstar requested a review from Connor1996 October 8, 2024 07:27
@LykxSassinator
Copy link

/retest

@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 8, 2024

/run-all-tests

3 similar comments
@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 8, 2024

/run-all-tests

@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 8, 2024

/run-all-tests

@v01dstar
Copy link
Contributor Author

v01dstar commented Oct 8, 2024

/run-all-tests

@LykxSassinator LykxSassinator merged commit fe7cfbb into tikv:master Oct 8, 2024
3 checks passed
@v01dstar v01dstar deleted the upgrade-rocksdb branch October 8, 2024 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants