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 compact filter test case #2

Conversation

git-hulk
Copy link

@git-hulk git-hulk commented Aug 26, 2023

RocksDB compaction allows to use of trivial move mode(move file into the next level instead of using compaction filter to generate the output file) when the compaction is not triggered by the manual way.

So after this fix, NeedCompact will return the correct result and then wait for the background compaction schedule(become no manual way so it allows the trivial move), so after we call the storage->Compact, it won't manually compact the moved file.

I fixed this issue by using force bottom-level compaction when triggering the manual compaction, it's our expected behavior.

See https://github.com/facebook/rocksdb/blob/ba597514309b686d8addb59616f067d5522186b7/db/db_impl/db_impl_compaction_flush.cc#L3358

@PragmaTwice PragmaTwice merged commit bbc0bc7 into PragmaTwice:fix-table-prop-collector Aug 26, 2023
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.

2 participants