Skip to content

Conversation

@ajbeamon
Copy link
Contributor

This parameter controls the size of the (4K) page cache. Also changed the default slighty from 2000 MiB to 2GiB.

…K) page cache. Change the default slighty from 2000 MiB to 2GiB.
ajbeamon added 2 commits July 23, 2019 15:07
# Conflicts:
#	documentation/sphinx/source/release-notes.rst
" Maximum amount of memory used for storage. The default\n"
" value is 1GiB. When specified without a unit, MB is\n"
" assumed.\n");
printf(" --cache_memory SIZE\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

So this help will only be displayed if you call fdbserver --dev-help - but not if you only pass --help. I assume the reason you want this to be a parameter is because users shouldn't be expected to touch knobs...

So I guess my confusion is: what exactly is the rule for putting this in --dev-help? Or should this be moved above this if-block? I have the same question for the --memory and --storage_memory options...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I honestly don't really know the intended distinction between --dev-help and --help, but I decided not to take any stand on that here. I added this option where it is because it seemed conceptually similar to the other two memory options you listed.

Copy link
Contributor

Choose a reason for hiding this comment

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

I guess this makes sense - although I think none of them should be under --dev-help - though I think this is a bit out of scope for this PR. I will make an issue.

@etschannen etschannen merged commit 1a4ca05 into apple:master Jul 26, 2019
@ajbeamon ajbeamon deleted the add-cache-memory-parameter branch July 30, 2019 22:12
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