-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
add prefetch to PosixRandomAccessFile in buffered io #2196
Conversation
@lightmark has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
@lightmark updated the pull request - view changes - changes since last import |
OSX is still not fixed. |
@lightmark updated the pull request - view changes - changes since last import |
41c1633
to
d6c14c6
Compare
@lightmark updated the pull request - view changes - changes since last import |
@lightmark has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
@lightmark updated the pull request - view changes - changes since last import |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@lightmark has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: Every time after a compaction/flush finish, we issue user reads to put the table into block cache which includes a couple of IO that read footer, index blocks, meta block, etc. So we implement Prefetch here to reduce IO. Closes #2196 Differential Revision: D4931782 Pulled By: lightmark fbshipit-source-id: 5a13d58dcab209964352322217193bbf7ff78149
Every time after a compaction/flush finish, we issue user reads to put the table into block cache which includes a couple of IO that read footer, index blocks, meta block, etc. So we implement Prefetch here to reduce IO.