facebook / facebook/rocksdb

fifo compaction ttl not working

Open
#9,336 5 comments 0 reactions 0 assignees View on GitHub
design discussion
Dominant language
C++
Stars
32.1k
Forks
6.9k
Avg merge
32m
Merged PRs (30d)
1

Description

@jamesgolick @sagar0
fifo compaction ttl not working when there is a file that is ingested by SSTFileWriter because of the following code in fifo compaction picker
rocksdb:compaction_picker_fifo cc at main · facebook:rocksdb 2021-12-27 09-27-37

because the loop breaks out when creation time is 0 and the creation time of sst files created by "ingesting external sst files" is 0. The SST Writer does not give any creation time.

rocksdb:sst_file_writer cc at main · facebook:rocksdb 2021-12-27 09-38-16

I think this is a bug here. Either remove this condition here or add creation time on SST file writer when constructing the file. But I am not very sure if there is any other specific purpose of this 'creation_time==0' condition here. Can I remove this?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.