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

Fix write global seqno #1563

Merged
merged 3 commits into from
Jul 11, 2023
Merged

Conversation

ruojieranyishen
Copy link
Collaborator

@ruojieranyishen ruojieranyishen commented Jul 7, 2023

#1562

Add this option to turn off rocksdb modifying rocksdb.external_sst_file.global_seqno
field of external sst file during ingest process. This is used to address a coredump
caused by incorrect field value in RocksDB. This pull request fixs the bug above.
Rocksdb 6.6.4 no longer recommends using this field.

New configuration is added:

[pegasus.server]
+ rocksdb_write_global_seqno = false

…o false by default. apache#1560

apache#1560

Add rocksdb_write_global_seqno, and set its default value to false.
…efault. apache#1562

apache#1562

Add rocksdb_write_global_seqno, and set its default value to false.
@github-actions github-actions bot added the cpp label Jul 7, 2023
@hycdong hycdong added the type/config-change Added or modified configuration that should be noted on release note of new version. label Jul 10, 2023
@hycdong
Copy link
Contributor

hycdong commented Jul 10, 2023

This pull request adds a new configuration, please add it into your pr description like the old one: XiaoMi/rdsn#851

@empiredan empiredan merged commit 99456f4 into apache:master Jul 11, 2023
@empiredan empiredan mentioned this pull request Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cpp type/config-change Added or modified configuration that should be noted on release note of new version.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants