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

udpate README #1

Merged
merged 1 commit into from
Dec 18, 2020
Merged

udpate README #1

merged 1 commit into from
Dec 18, 2020

Conversation

Evil-crow
Copy link
Collaborator

provide content table and shield.io cards

@TendisDev TendisDev merged commit 4968f7b into Tencent:master Dec 18, 2020
TendisDev added a commit that referenced this pull request Jan 19, 2021
RocksDB version update to 6.14.5
1) 默认关闭新版本, 还是使用 RocksDB 5.13.4

## RocksDB 兼容性修改

1) TableFactory::GetOptions() 的修改
  6.14 版本改为如下方式: `table_factory->GetOptions<rocksdb::BlockBasedTableOptions>()`。

2) `getProperty:rocksdb.levelstatsex`
  暂时未修改, 为我们添加的功能

3) RocksDB `getBaseDB()->Resume()`
5.13.4 不支持, 新版本才支持

## ASAN 工具集成

编译方式如下:
`cmake -DCMAKE_BUILD_TYPE=Debug -DWITH_ASAN=ON -DENABLE_JEMALLOC=OFF ..`

可以流水线自动跑, 目前有几个。
目前检测出一处内存泄漏, 已修复。`rocksdb::Checkpoint* checkpoint` 的释放。
遗留: 有几个单元测例应该没有内存泄漏, 报如下错误, 实际中应该可以忽略。

```sh
==34329==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 48 byte(s) in 2 object(s) allocated from:
    #0 0x4a1bba in operator new(unsigned long, std::nothrow_t const&) ../../../../libsanitizer/asan/asan_new_delete.cc:65
    #1 0x7fde603a7d65 in __cxa_thread_atexit ../../../../libstdc++-v3/libsupc++/atexit_thread.cc:142

SUMMARY: AddressSanitizer: 48 byte(s) leaked in 2 allocation(s).
```
@mozz85 mozz85 mentioned this pull request Nov 17, 2022
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.

None yet

2 participants