Merge pull request #46923 from rzarzynski/wip-crimson-opexec-reformat-switch

crimson/osd: clean up the op dispatch on OpsExecuter

Reviewed-by: Kefu Chai <tchaikov@gmail.com>
Reviewed-by: Samuel Just <sjust@redhat.com>

diff --cc src/crimson/osd/ops_executer.cc
index 54dffa4349366be2799901db1e26190fd1df0209,50c68b25566f6ed077f1b6d7bd19587ee584c5b6..4a5b5b89d0ad67f0b852c9375c95ce635f13560e
--- 1/src/crimson/osd/ops_executer.cc
--- 2/src/crimson/osd/ops_executer.cc
+++ b/src/crimson/osd/ops_executer.cc
@@@ -560,23 -559,21 +559,23 @@@ OpsExecuter::do_execute_op(OSDOp& osd_o
        // FIXME: rework needed. Move this out to do_write_op(), introduce
        // do_write_op_no_user_modify()...
        return backend.truncate(os, osd_op, txn, *osd_op_params, delta_stats);
-     }, true);
+     });
    case CEPH_OSD_OP_ZERO:
-     return do_write_op([this, &osd_op] (auto& backend, auto& os, auto& txn) {
+     return do_write_op([this, &osd_op](auto& backend, auto& os, auto& txn) {
        return backend.zero(os, osd_op, txn, *osd_op_params, delta_stats);
-     }, true);
+     });
    case CEPH_OSD_OP_SETALLOCHINT:
 -    return osd_op_errorator::now();
 +    return do_write_op([this, &osd_op](auto& backend, auto& os, auto& txn) {
 +      return backend.set_allochint(os, osd_op, txn, delta_stats);
 +    }, true);
    case CEPH_OSD_OP_SETXATTR:
-     return do_write_op([this, &osd_op] (auto& backend, auto& os, auto& txn) {
+     return do_write_op([this, &osd_op](auto& backend, auto& os, auto& txn) {
        return backend.setxattr(os, osd_op, txn, delta_stats);
-     }, true);
+     });
    case CEPH_OSD_OP_DELETE:
-     return do_write_op([this] (auto& backend, auto& os, auto& txn) {
+     return do_write_op([this](auto& backend, auto& os, auto& txn) {
        return backend.remove(os, txn, delta_stats);
-     }, true);
+     });
    case CEPH_OSD_OP_CALL:
      return this->do_op_call(osd_op);
    case CEPH_OSD_OP_STAT: