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

Use pwritev() #238

Closed
benbjohnson opened this issue Aug 6, 2014 · 6 comments
Closed

Use pwritev() #238

benbjohnson opened this issue Aug 6, 2014 · 6 comments
Assignees
Milestone

Comments

@benbjohnson
Copy link
Member

Use pwritev() when flushing data to disk so that we can batch multiple pages into a single syscall.

/cc @siddontang @pkieltyka

@benbjohnson benbjohnson added this to the v0.1.0 milestone Aug 6, 2014
@benbjohnson benbjohnson self-assigned this Aug 6, 2014
@pkieltyka
Copy link

👍

@pkieltyka
Copy link

aren't you on vacation? haha

@benbjohnson
Copy link
Member Author

That's what my wife keeps trying to tell me. :)

@siddontang
Copy link
Contributor

er...... vacation is unimaginable in China.

@pkieltyka
Copy link

hey guys, just wondering if this is on the radar, since it sounds like it can help with write performance?

@benbjohnson
Copy link
Member Author

I haven't made any progress on this yet. It hasn't been much of a priority because I'm not sure how much it'll actually help performance. The main performance issue with random writes is that they amplify writes a ton. You likely end up getting throttled by disk write speed and not syscall invocation time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants