google / google/leveldb

Allow clients to set maximum number of memory mapped files

Open
#238 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
39.4k
Forks
8.2k
PR merge metrics
No merged PRs in 30d

Description

Original [issue 232](https://code.google.com/p/leveldb/issues/detail?id=232) created by mattjrichards on 2014-04-16T18:12:59.000Z:

Currently LevelDB sets a maximum of 1000 mmap()ed files on 64-bit systems and none on 32-bit systems via the MmapLimiter class. It would be beneficial for clients to be able to control this limit. For example, a client concerned about reliability might set this limit to zero on all systems to avoid SIGBUS when reading mmaped() files that have unrecoverable read errors.

Admittedly the notion of memory mapped files is hidden in the POSIX environment utility class, but I think they are a common enough feature that it wouldn't be to bad of an abstraction leakage to expose a setter for this limit.

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.