Revert "Connect broken traces in consensus for preexecuted transactions (#2616)" (#2645)

This reverts commit 0edb6be.

Author: yuliasherman

bftengine/src/bftengine/ReplicaImp.cpp

@@ -2097,8 +2097,7 @@ void ReplicaImp::onCommitCombinedSigSucceeded(SeqNum seqNumber,
       (seqNumber > lastExecutedSeqNum + config_.getconcurrencyLevel() + activeExecutions_);
 
   auto span = concordUtils::startChildSpanFromContext(
-      commitFull->spanContext<std::remove_pointer<decltype(commitFull)>::type>(),
-      "bft_handle_commit_combined_sig_succeeded_message");
+      commitFull->spanContext<std::remove_pointer<decltype(commitFull)>::type>(), "bft_execute_committed_reqs");
   updateCommitMetrics(CommitPath::SLOW);
   startExecution(seqNumber, span, askForMissingInfoAboutCommittedItems);
 }
@@ -2145,8 +2144,7 @@ void ReplicaImp::onCommitVerifyCombinedSigResult(SeqNum seqNumber, ViewNum view,
   LOG_INFO(CNSUS, "Request committed, proceeding to try to execute" << KVLOG(view));
 
   auto span = concordUtils::startChildSpanFromContext(
-      commitFull->spanContext<std::remove_pointer<decltype(commitFull)>::type>(),
-      "bft_handle_commit_verify_combined_sig_result");
+      commitFull->spanContext<std::remove_pointer<decltype(commitFull)>::type>(), "bft_execute_committed_reqs");
   bool askForMissingInfoAboutCommittedItems =
       (seqNumber > lastExecutedSeqNum + config_.getconcurrencyLevel() + activeExecutions_);
   updateCommitMetrics(CommitPath::SLOW);
@@ -2205,7 +2203,7 @@ void ReplicaImp::onFastPathCommitCombinedSigSucceeded(SeqNum seqNumber,
        lastExecutedSeqNum + config_.getconcurrencyLevel() + activeExecutions_);  // TODO(GG): check/improve this logic
 
   auto span = concordUtils::startChildSpanFromContext(fcp->spanContext<std::remove_pointer<decltype(fcp)>::type>(),
-                                                      "bft_handle_fast_path_commit_combined_sig_succeeded");
+                                                      "bft_execute_committed_reqs");
 
   updateCommitMetrics(cPath);
 
@@ -2270,7 +2268,7 @@ void ReplicaImp::onFastPathCommitVerifyCombinedSigResult(SeqNum seqNumber,
        lastExecutedSeqNum + config_.getconcurrencyLevel() + activeExecutions_);  // TODO(GG): check/improve this logic
 
   auto span = concordUtils::startChildSpanFromContext(fcp->spanContext<std::remove_pointer<decltype(fcp)>::type>(),
-                                                      "bft_handle_fast_path_commit_verify_combined_sig_result");
+                                                      "bft_execute_committed_reqs");
 
   updateCommitMetrics(cPath);
 
@@ -4970,14 +4968,14 @@ void ReplicaImp::executeSpecialRequests(PrePrepareMsg *ppMsg,
     reqIdx++;
   }
 
-  auto span_context = ppMsg->spanContext<std::remove_pointer<decltype(ppMsg)>::type>();
   // TODO(GG): the following code is cumbersome. We can call to execute directly in the above loop
   IRequestsHandler::ExecutionRequestsQueue singleRequest;
   for (IRequestsHandler::ExecutionRequest &req : accumulatedRequests) {
     ConcordAssert(singleRequest.empty());
     singleRequest.push_back(req);
     {
-      auto span = concordUtils::startChildSpanFromContext(span_context, "bft_client_special_request");
+      const concordUtils::SpanContext &span_context{""};
+      auto span = concordUtils::startChildSpanFromContext(span_context, "bft_client_request");
       span.setTag("rid", config_.getreplicaId());
       span.setTag("cid", req.cid);
       span.setTag("seq_num", req.requestSequenceNum);
@@ -5039,12 +5037,12 @@ void ReplicaImp::executeRequests(PrePrepareMsg *ppMsg, Bitmap &requestSet, Times
       setConflictDetectionBlockId(req, pAccumulatedRequests->back());
     }
   }
-  auto span_context = ppMsg->spanContext<std::remove_pointer<decltype(ppMsg)>::type>();
   if (ReplicaConfig::instance().blockAccumulation) {
     LOG_DEBUG(GL,
               "Executing all the requests of preprepare message with cid: " << ppMsg->getCid() << " with accumulation");
     {
       //      TimeRecorder scoped_timer1(*histograms_.executeWriteRequest);
+      const concordUtils::SpanContext &span_context{""};
       auto span = concordUtils::startChildSpanFromContext(span_context, "bft_client_request");
       span.setTag("rid", config_.getreplicaId());
       span.setTag("cid", ppMsg->getCid());
@@ -5067,6 +5065,7 @@ void ReplicaImp::executeRequests(PrePrepareMsg *ppMsg, Bitmap &requestSet, Times
       singleRequest.push_back(req);
       {
         //        TimeRecorder scoped_timer1(*histograms_.executeWriteRequest);
+        const concordUtils::SpanContext &span_context{""};
         auto span = concordUtils::startChildSpanFromContext(span_context, "bft_client_request");
         span.setTag("rid", config_.getreplicaId());
         span.setTag("cid", ppMsg->getCid());