facebook / facebook/rocksdb

add logging.h in include/rocksdb

Open
#4,316 5 comments 0 reactions 0 assignees View on GitHub
up-for-grabs
Dominant language
C++
Stars
32.1k
Forks
6.9k
Avg merge
32m
Merged PRs (30d)
1

Description

> Note: Please use Issues only for bug reports. For questions, discussions, feature requests, etc. post to dev group: https://www.facebook.com/groups/rocksdb.dev

### Expected behavior

Having a public api in "include/rocksdb" to use the Logger instance in merge operators would make it easier to log errors in them.

### Actual behavior

The merge operators in utilities are using util/logging.h but don't show how to use the Logger instance without the helpers provided in this file. It's actually unclear how to do it in fact.

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.