GitHub user qiwei9743 deleted a comment on the discussion: CompactOnExpiredCollector的工作不符合预期
感谢大佬回复。 我上面提到的不是 优化，是疑问，我怕对代码理解错误，需要帮忙澄清一下。 重新描述一下我的问题。 问题1，compact meta cf的时候，为什么deleted_keys需要计算 subkey的数量，也就是deleted_keys_ += metadata.size + 1 ？ NeedCompact函数需要依赖metadata.size的原理是什么? 问题2， CompactOnExpiredCollector::AddUserKey函数中的deleted_keys_ += metadata.size + 1大概率是执行不到的代码；meta cf compact过程中，对于metakey1(已经过期了)，先调用MetadataFilter::Filter，把metakey1转换成entry_type rocksdb::kEntryDelete，导致调用CompactOnExpiredCollector::AddUserKey的时候，不会调用到deleted_keys_ += metadata.size + 1; 谢谢 GitHub link: https://github.com/apache/kvrocks/discussions/2898#discussioncomment-12930953 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
