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

Move metadata settings to advanced level. #833

Merged
merged 2 commits into from
Feb 19, 2014
Merged

Move metadata settings to advanced level. #833

merged 2 commits into from
Feb 19, 2014

Conversation

evanmcc
Copy link
Contributor

@evanmcc evanmcc commented Feb 11, 2014

move md cache cuttlefish tunings into advanced so people are less
likely to play with it. as written it is hard to make it improve
performance in a predictable way.

  • Eng +1
  • PM +1
  • CSE +1

likely to play with it.  as written it is hard to make it improve
performance in a predictable way.
@lordnull
Copy link
Contributor

Same generated value before and after change, only difference is the metadata_cache_size is not visible in the riak.conf file.

👍

evanmcc added a commit that referenced this pull request Feb 19, 2014
Move metadata settings to advanced level.
@evanmcc evanmcc merged commit 7d6e837 into develop Feb 19, 2014
@seancribbs
Copy link
Contributor

This did not get CSE/PM approval. 😦

@evanmcc
Copy link
Contributor Author

evanmcc commented Feb 19, 2014

shrug

It should have never been in there in the first place, since the feature
was never fully baked. I'll be sure to ping everyone next time.

@jonmeredith
Copy link
Contributor

It was discussed on the Core/KV mumble and I think GregB was there. But I could have remembered wrong.

@seancribbs did it break something?

@seancribbs
Copy link
Contributor

@jonmeredith Nope, just wanted to be sure the process was clear.

@seancribbs seancribbs deleted the metadata-yank branch April 1, 2015 23:36
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.

4 participants