DNM: RGW: Make RADOS handles in RGW to be a configurable option#5612
DNM: RGW: Make RADOS handles in RGW to be a configurable option#5612prallabh wants to merge 2 commits intoceph:hammerfrom
Conversation
|
@prallabh could you please cherry-pick -x the commit to be backported and repush ? git reset --hard ceph/hammer git cherry-pick -x 77e0084 git push --force origin hammer assuming origin is the remote of your github clone and ceph is the remote of the official repository. |
|
@yehudasa could you please confirm that this feature should be backported to hammer ? |
|
@dachary This PR is with your suggestion of 'cheery-pick -x', please let me know if am missing anything else. |
|
@dachary confirmed |
|
@prallabh could you please rebase and repush to trigger the bot again ? It looks like this is a not a real error. |
Having a configurable number of RADOS handles is resulting in significant performance boost for all types of workloads. Each RGW worker thread would now get to pick a RADOS handle for its lifetime, from the available bunch. Signed-off-by: Pavan Rallabhandi <pavan.rallabhandi@sandisk.com> (cherry picked from commit b3c3a24)
Signed-off-by: Pavan Rallabhandi <pavan.rallabhandi@sandisk.com> (cherry picked from commit 7476e94)
|
@prallabh looks good, thanks |
…igurable option Reviewed-by: Loic Dachary <ldachary@redhat.com>
|
@prallabh is there an issue matching this change in http://tracker.ceph.com/ ? |
|
@dachary if you are referring to, whether we have a tracker for this change set, there ain't any. Do you want me to create one? We never had one even for the original PR on the master. |
…igurable option Reviewed-by: Loic Dachary <ldachary@redhat.com>
|
@prallabh it's ok to create an issue even after the commits have been merged. And it helps with tracking backports. Could you create one for the original PR on the master ? I'll take care of the things related to the backport. |
|
Marked DNM because it's already included in #5717 |
|
@dachary Please find the tracker http://tracker.ceph.com/issues/12893. Please update the relevant fields in the tracker appropriately if they are out of place. |
|
Merged in #5717 onto hammer branch .. this can be closed. |
|
@theanalyst nice catch :-) |
http://tracker.ceph.com/issues/12894