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

kraken: rgw: first write also tries to read object #14515

Merged
merged 4 commits into from Jul 7, 2017

Conversation

smithfarm
Copy link
Contributor

@smithfarm smithfarm self-assigned this Apr 13, 2017
@smithfarm smithfarm added this to the kraken milestone Apr 13, 2017
Fixes: http://tracker.ceph.com/issues/18621

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 54ef04b)
Fixes: http://tracker.ceph.com/issues/18623

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 66a82b4)
if fails and object already exists then retry. This improves first obj
write performance on the expense of overwrites.

Fixes: http://tracker.ceph.com/issues/18622

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 6f27f60)
Instead of this for overwrites:
 prepare (index),
   write (head) [-EEXIST]
 cancel (index)
 read (head)
 prepare (index)
   write (head)
 complete (index)

We now do:
 prepare (index),
   write (head) [-EEXIST]
   read (head)
   write (head)
 complete (index)

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit 7f4818f)
@smithfarm
Copy link
Contributor Author

Jenkins re-test this please

@smithfarm smithfarm requested a review from yehudasa July 6, 2017 06:45
@smithfarm
Copy link
Contributor Author

@yehudasa @mattbenjamin @oritwas @cbodley This PR passed an rgw suite at http://tracker.ceph.com/issues/19009#note-60 and, provided there are no objections, I'll merge it as soon as it gets a positive review from one of you.

@smithfarm smithfarm merged commit eb397c1 into ceph:kraken Jul 7, 2017
@smithfarm smithfarm deleted the wip-18904-kraken branch July 7, 2017 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants