It is unclear why this path was originally needed, but it does predate
the removal of the dual-ack removal, so I expect it was related to
that. It is no longer necesarry.
Signed-off-by: Sage Weil <sage@redhat.com>
if (m && !ctx->sent_reply) {
MOSDOpReply *reply = ctx->reply;
- if (reply)
- ctx->reply = nullptr;
- else {
- reply = new MOSDOpReply(m, 0, get_osdmap_epoch(), 0, true);
- reply->set_reply_versions(ctx->at_version,
- ctx->user_at_version);
- }
+ ctx->reply = nullptr;
reply->add_flags(CEPH_OSD_FLAG_ACK | CEPH_OSD_FLAG_ONDISK);
dout(10) << " sending reply on " << *m << " " << reply << dendl;
osd->send_message_osd_client(reply, m->get_connection());