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

osdc/Filer: truncate large file party by party #14769

Merged
merged 1 commit into from May 16, 2017

Conversation

Projects
None yet
3 participants
@ukernel
Member

ukernel commented Apr 25, 2017

Fixes: http://tracker.ceph.com/issues/19755
Signed-off-by: "Yan, Zheng" zyan@redhat.com

osdc/Filer: truncate large file party by party
Fixes: http://tracker.ceph.com/issues/19755
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
@batrick

This comment has been minimized.

trl.unlock();
// Issue objecter ops outside tr->lock to avoid lock dependency loop

This comment has been minimized.

@jcsp

jcsp May 11, 2017

Contributor

Since we have dropped the lock here, why is it still necessary to use a C_OnFinisher below?

This comment has been minimized.

@jcsp

This comment has been minimized.

Contributor

jcsp commented May 15, 2017

This passes tests, just that one comment about whether the use of finisher is really needed.

@jcsp jcsp merged commit ea4a165 into ceph:master May 16, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@ukernel ukernel deleted the ukernel:wip-19755 branch May 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment