• P
    Put Cache and CacheWrapper in new public header (#11192) · 3cacd4b4
    Peter Dillinger 提交于
    Summary:
    The definition of the Cache class should not be needed by the vast majority of RocksDB users, so I think it is just distracting to include it in cache.h, which is primarily needed for configuring and creating caches. This change moves the class to a new header advanced_cache.h. It is just cut-and-paste except for modifying the class API comment.
    
    In general, operations on shared_ptr<Cache> should continue to work when only a forward declaration of Cache is available, as long as all the Cache instances provided are already shared_ptr. See https://stackoverflow.com/a/17650101/454544
    
    Also, the most common way to customize a Cache is by wrapping an existing implementation, so it makes sense to provide CacheWrapper in the public API. This was a cut-and-paste job except removing the implementation of Name() so that derived classes must provide it.
    
    Intended follow-up: consolidate Release() into one function to reduce customization bugs / confusion
    
    Pull Request resolved: https://github.com/facebook/rocksdb/pull/11192
    
    Test Plan: `make check`
    
    Reviewed By: anand1976
    
    Differential Revision: D43055487
    
    Pulled By: pdillinger
    
    fbshipit-source-id: 7b05492df35e0f30b581b4c24c579bc275b6d110
    3cacd4b4
db_test_util.h 43.7 KB