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

L1 has overlapping ranges in VersionSet::DumpManifest #11604

Closed
huangmengbin opened this issue Jul 12, 2023 · 0 comments
Closed

L1 has overlapping ranges in VersionSet::DumpManifest #11604

huangmengbin opened this issue Jul 12, 2023 · 0 comments

Comments

@huangmengbin
Copy link
Contributor

huangmengbin commented Jul 12, 2023

RocksDB Tag: 8.2.3
Exec: ROCKSDB_NAMESPACE::ManifestDumpCommand

Expected behavior

next_file_number XXXX last_sequence XXXXX prev_log_number X max_column_family XX min_log_number_to_keep XXX

Actual behavior

Corruption: force_consistency_checks(DEBUG): VersionBuilder: L1 has overlapping ranges: file #139009 largest key: 'AB' seq:2154465125, type:2 vs. file #139013 smallest key: 'AAA' seq:2152627326, type:2

Steps to reproduce the behavior

std::vector<ROCKSDB_NAMESPACE::ColumnFamilyDescriptor> column_families = XXX;
column_families[0].options.comparator = MY_CF_COMPARATOR_0;
column_families[1].options.comparator = MY_CF_COMPARATOR_1;
// ......
ROCKSDB_NAMESPACE::ManifestDumpCommand cmd(
        {}, {{ROCKSDB_NAMESPACE::LDBCommand::ARG_PATH, FLAGS_path}},
        {"verbose", ROCKSDB_NAMESPACE::LDBCommand::ARG_HEX});
cmd.SetDBOptions(options);
cmd.SetColumnFamilies(&column_families);
cmd.DoCommand();
auto s = cmd.GetExecuteState();

But we can open, read and write the DB normally.

rockeet pushed a commit to topling/toplingdb that referenced this issue Dec 18, 2023
Summary:
Fixes facebook/rocksdb#11604

Pull Request resolved: facebook/rocksdb#11605

Reviewed By: jowlyzhang

Differential Revision: D47459254

Pulled By: ajkr

fbshipit-source-id: 4420e443fbf4bd01ddaa2b47285fc4445bf36246
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 a pull request may close this issue.

1 participant