bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

Correct the comment of caching index & filter block with additional options to improve performance

Open Myasuka opened this issue 4 years ago • 1 comments

Descriptions of the changes in this PR:

Motivation

The previous comment of tableOptions.setCacheIndexAndFilterBlocks(true) is Options best suited for HDDs. If the comment thought that we could cache index&filter blocks when using HDD for better performance, however, this is not correct as this option actually helps on limit the memory usage of index & filter blocks. More internal details could refer to https://github.com/facebook/rocksdb/wiki/Block-Cache#caching-index-filter-and-compression-dictionary-blocks

Apart from correcting the comment, this PR also add additional options to help improve performance.

Changes

Correct the comment of caching index & filter block with additional options to improve performance.

Myasuka avatar Oct 13 '21 09:10 Myasuka

fix old workflow,please see #3455 for detail

StevenLuMT avatar Aug 24 '22 08:08 StevenLuMT