rgw: get attrs in AbortMultipart only if tracing is enabled

Signed-off-by: Omri Zeneva <ozeneva@redhat.com>

diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
index ec80c7f34d63368e0b6e03bf9ce5ab7c32a3a7ed..e7287107e9d39c5c412bde584891e75750c0754e 100644 (file)
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@ -6507,16 +6507,14 @@ void RGWAbortMultipart::execute(optional_yield y)
   upload = s->bucket->get_multipart_upload(s->object->get_name(), upload_id);
   RGWObjectCtx *obj_ctx = static_cast<RGWObjectCtx *>(s->obj_ctx);
 
-  meta_obj = upload->get_meta_obj();
-  meta_obj->set_in_extra_data(true);
-  op_ret = meta_obj->get_obj_attrs(obj_ctx, s->yield, this);
-  if (op_ret < 0) {
-    ldpp_dout(this, 0) << "ERROR: failed to get obj attrs, obj=" << meta_obj
-                    << " ret=" << op_ret << dendl;
-    return;
-  }
   jspan_context trace_ctx(false, false);
-  extract_span_context(meta_obj->get_attrs(), trace_ctx);
+  if (tracing::rgw::tracer.is_enabled()) {
+    // read meta object attributes for trace info
+    meta_obj = upload->get_meta_obj();
+    meta_obj->set_in_extra_data(true);
+    meta_obj->get_obj_attrs(obj_ctx, s->yield, this);
+    extract_span_context(meta_obj->get_attrs(), trace_ctx);
+  }
   multipart_trace = tracing::rgw::tracer.add_span(name(), trace_ctx);
 
   op_ret = upload->abort(this, s->cct, obj_ctx);