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

Mv tuning6 #96

Merged
merged 4 commits into from
Feb 15, 2014
Merged

Mv tuning6 #96

merged 4 commits into from
Feb 15, 2014

Conversation

matthewvon
Copy link
Contributor

Detail description here: https://github.com/basho/leveldb/wiki/mv-tuning6

This branch requires manual pull of leveldb's mv-tuning6 branch to compile.

Adds block_cache_threshold parameter to options interfaces. Corrects an incorrect type on m_TotalMem member of EleveldbOptions structure.

{mapping, "leveldb.block_cache_threshold", "eleveldb.block_cache_threshold", [
{default, "16MB"},
{datatype, bytesize},
{level, advanced}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

{level, advanced} is deprecated, use hidden instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected. And pushed to origin.

On Feb 13, 2014, at 2:26 PM, Sean Cribbs notifications@github.com wrote:

In priv/eleveldb.schema:

@@ -134,6 +134,23 @@
{level, advanced}
]}.

+%% @doc Defines the limit where block cache memory can no longer be
+%% released in favor of the page cache. This has no impact with
+%% regard to release in favor of file cache. The value is per
+%% vnode.
+{mapping, "leveldb.block_cache_threshold", "eleveldb.block_cache_threshold", [

  • {default, "16MB"},
  • {datatype, bytesize},
  • {level, advanced}
    {level, advanced} is deprecated, use hidden instead.


Reply to this email directly or view it on GitHub.

@andrewjstone
Copy link
Contributor

code is good. tests pass. 👍

matthewvon pushed a commit that referenced this pull request Feb 15, 2014
@matthewvon matthewvon merged commit 63733a9 into develop Feb 15, 2014
@seancribbs seancribbs deleted the mv-tuning6 branch April 1, 2015 22:45
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.

3 participants