Skip to content
This repository has been archived by the owner on Mar 9, 2019. It is now read-only.

introduce initialMmapSize #432

Closed
wants to merge 1 commit into from
Closed

introduce initialMmapSize #432

wants to merge 1 commit into from

Conversation

xiang90
Copy link
Contributor

@xiang90 xiang90 commented Oct 3, 2015

InitialMmapSize is the initial mmap size of the database in bytes.
Read transaction is guaranteed not to block write transaction
if the mmap size required by the database is smaller than
the initial mmap.

InitialMmapSize is the initial mmap size of the database in bytes.
Read transaction is guaranteed not to block write transaction
if the mmap size required by the database is smaller than
the initial mmap.
@xiang90
Copy link
Contributor Author

xiang90 commented Oct 3, 2015

/cc @ngaut Probably you are also interested in this one?

// InitialMmapSize is the initial mmap size of the database
// in bytes.
//
// Read transaction is guaranteed not to block write transaction
Copy link
Contributor

Choose a reason for hiding this comment

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

"Read transactions are guaranteed not to block write transactions ..."

@benbjohnson
Copy link
Member

@xiang90 were you able to test this out on etcd?

@xiang90
Copy link
Contributor Author

xiang90 commented Oct 28, 2015

@benbjohnson We will test it before early next week. Sorry for the delay.

@xiang90
Copy link
Contributor Author

xiang90 commented Dec 21, 2015

@benbjohnson Yes. We tested it in both etcd and a test case. It works well. See #472 for more information. @gyuho is taking over this.

@xiang90 xiang90 closed this Dec 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants