Revert "Clear trace response instead of metric response in OTelExportSinkNode::ConsumeSpans (#1910)"

This reverts commit 970a54a.

Author: ddelnano

src/carnot/exec/otel_export_sink_node.cc

@@ -340,7 +340,7 @@ Status OTelExportSinkNode::ConsumeSpans(ExecState* exec_state, const RowBatch& r
   }
   context.set_compression_algorithm(GRPC_COMPRESS_GZIP);
 
-  trace_response_.Clear();
+  metrics_response_.Clear();
   opentelemetry::proto::collector::trace::v1::ExportTraceServiceRequest request;
 
   for (int64_t row_idx = 0; row_idx < rb.ColumnAt(0)->length(); ++row_idx) {

src/carnot/planner/objects/otel.cc

@@ -338,6 +338,7 @@ Status OTelModule::Init(CompilerState* compiler_state, IR* ir) {
 
   AddMethod(kEndpointOpID, endpoint_fn);
   PX_RETURN_IF_ERROR(endpoint_fn->SetDocString(kEndpointOpDocstring));
+  return Status::OK();
 
   return Status::OK();
 }