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

mon: add a cache layer over MonitorDBStore #5697

Merged
1 commit merged into from Sep 9, 2015

Conversation

tchaikov
Copy link
Contributor

the cache of of leveldb does not perform well under some condition,
so we need a cache in our own stack.

* add an option "mon_osd_cache_size" to control the size of cache size
  of MonitorDBStore.

Fixes: ceph#12638
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 5e99a57)

Conflicts:
	src/mon/OSDMonitor.h
minor differences in OSDMonitor ctor
@tchaikov tchaikov added this to the hammer milestone Aug 28, 2015
@ghost ghost self-assigned this Aug 29, 2015
@ghost ghost added bug-fix core labels Aug 29, 2015
ghost pushed a commit that referenced this pull request Aug 30, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 1, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 2, 2015
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Sep 6, 2015

@jecluis this hammer backport passed a rados suite (see http://tracker.ceph.com/issues/12701#note-62 for the details). Do you think it is ready to be merged ?

@ghost ghost assigned jecluis and unassigned ghost Sep 6, 2015
@jecluis
Copy link
Member

jecluis commented Sep 9, 2015

LGTM

@jecluis jecluis assigned ghost and unassigned jecluis Sep 9, 2015
ghost pushed a commit that referenced this pull request Sep 9, 2015
mon: add a cache layer over MonitorDBStore

Reviewed-by: Joao Eduardo Luis <joao@suse.de>
@ghost ghost merged commit 69a320e into ceph:hammer Sep 9, 2015
@tchaikov tchaikov deleted the wip-12638-hammer branch September 11, 2015 02:48
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants