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

Projects
None yet
3 participants
@tchaikov
Copy link
Contributor

commented Aug 28, 2015

mon: add a cache layer over MonitorDBStore
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: #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

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5697: mon: add a cache layer over MonitorDBStore
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

Copy link

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

This comment has been minimized.

Copy link
Member

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

Loic Dachary
Merge pull request #5697 from tchaikov/wip-12638-hammer
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 tchaikov:wip-12638-hammer branch Sep 11, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.