Split serialization and export of spans into separate functions for http trace exporter (#3826)

Co-authored-by: Diego Hurtado <[email protected]>

Author: kchoudhu

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py

@@ -126,19 +126,13 @@ def _retryable(resp: requests.Response) -> bool:
             return True
         return False
 
-    def export(self, spans) -> SpanExportResult:
-        # After the call to Shutdown subsequent calls to Export are
-        # not allowed and should return a Failure result.
-        if self._shutdown:
-            _logger.warning("Exporter already shutdown, ignoring batch")
-            return SpanExportResult.FAILURE
-
-        serialized_data = encode_spans(spans).SerializeToString()
+    def _serialize_spans(self, spans):
+        return encode_spans(spans).SerializePartialToString()
 
+    def _export_serialized_spans(self, serialized_data):
         for delay in _create_exp_backoff_generator(
             max_value=self._MAX_RETRY_TIMEOUT
         ):
-
             if delay == self._MAX_RETRY_TIMEOUT:
                 return SpanExportResult.FAILURE
 
@@ -163,6 +157,17 @@ def export(self, spans) -> SpanExportResult:
                 return SpanExportResult.FAILURE
         return SpanExportResult.FAILURE
 
+    def export(self, spans) -> SpanExportResult:
+        # After the call to Shutdown subsequent calls to Export are
+        # not allowed and should return a Failure result.
+        if self._shutdown:
+            _logger.warning("Exporter already shutdown, ignoring batch")
+            return SpanExportResult.FAILURE
+
+        serialized_data = self._serialize_spans(spans)
+
+        return self._export_serialized_spans(serialized_data)
+
     def shutdown(self):
         if self._shutdown:
             _logger.warning("Exporter already shutdown, ignoring call")