J. Eric Ivancich
J. Eric Ivancich
Please note -- this issue on the dmclock library, which is written to be generic, so that it could be used in other projects. At the moment, the ceph use...
Is this change related to the delayed tag calculation? It seems to be working to address the same performance issue.
Thank you for that detailed explanation. I'm very interested in your understanding of the situation and want to make sure I'm fully understanding the points you are making. So I...
Thank you for clarifying your understanding. Your argument that the rho values are too small makes sense. I'll hold off on merging this for the time being since it does...
@bspark8 Where do we stand on this PR now that we've merged the PR with the borrowing tracker (https://github.com/ceph/dmclock/pull/39)? Is this PR still of value? Will it need re-analysis?
This PR primarily has functionality addtions for the librados integration plus a few other related changes.
@cbodley Thanks for all your input. I reworked it and I think the code is much better now.
Teuthology results: * http://pulpito.ceph.com/ivancich-2023-02-17_21:47:17-rgw-wip-eric-testing-1-distro-default-smithi/ * http://pulpito.ceph.com/ivancich-2023-02-17_21:46:42-rgw-wip-eric-testing-1-distro-default-smithi/
@dang , @cbodley , @theanalyst : I failed to notice discussion was still underway when I ran QA on this. I leave the decision to merge to y'all. I'm adding...
> @dang , @cbodley , @theanalyst : I failed to notice discussion was still underway when I ran QA on this. I leave the decision to merge to y'all. I'm...