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

backport #13769: EC async "fast read" patchset for hammer #6542

Closed
wants to merge 3 commits into from

Conversation

tsg-
Copy link
Contributor

@tsg- tsg- commented Nov 11, 2015

EC read latency improvement. Commits backported:

131214d ec: add support for fast read on PGBackend/ECBackend async read

5eb2a77 mon: add a new pool setting to configure fast read for EC pool

54090f1 mon: add a configuration for default fast read setting

Guang Yang and others added 3 commits November 11, 2015 11:00
Extend the PGBackend::objects_read_async interface to support *fast read*,
add the implemenation for ECBackend, in which we issue redundant reads,
and use the first returned (to decode) to serve clients.

Backport: Hammer
Signed-off-by: Guang Yang <yguang@yahoo-inc.com>
(cherry picked from commit 131214d)
Backport: Hammer
Signed-off-by: Guang Yang <yguang@yahoo-inc.com>
Signed-off-by: Tushar Gohad <tushar.gohad@intel.com>
(cherry-picked from commit 5eb2a77)
Backport: Hammer
Signed-off-by: Guang Yang <yguang@yahoo-inc.com>
Signed-off-by: Tushar Gohad <tushar.gohad@intel.com>
(cherry-picked from commit 54090f1)
@tsg- tsg- changed the title Hammer backport #13769: EC async "fast read" patchset for hammer backport #13769: EC async "fast read" patchset for hammer Nov 11, 2015
@ghost
Copy link

ghost commented Nov 11, 2015

@tsg backports should be against the hammer branch, not the hammer-backports branch. You will find more information about backporting at http://tracker.ceph.com/projects/ceph-releases/wiki/HOWTO_backport_commits

@tsg-
Copy link
Contributor Author

tsg- commented Nov 11, 2015

Thanks @dachary! Sorry newbie error. Let me resubmit. :^)

@ghost
Copy link

ghost commented Nov 11, 2015

@tsg thanks :-)

@ghost ghost closed this Nov 11, 2015
@tsg-
Copy link
Contributor Author

tsg- commented Nov 16, 2015

@dachary I am back. :-) - resubmitted this pull request as #6611 with the required commit format etc. Thanks.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants