fix: [Geneva Exporter] Remove sorting from Geneva Exporter schema ID calculation for performance optimization (#413)

Copilot · lalitb · web-flow · commit 46b18fbf0672 · 2025-08-06T13:49:15.000-07:00
Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: lalitb &lt;1196320+lalitb@users.noreply.github.com&gt;
Co-authored-by: Lalit Kumar Bhasin &lt;lalit_fin@yahoo.com&gt;
diff --git a/opentelemetry-exporter-geneva/geneva-uploader/src/payload_encoder/otlp_encoder.rs b/opentelemetry-exporter-geneva/geneva-uploader/src/payload_encoder/otlp_encoder.rs
@@ -213,9 +213,7 @@ impl OtlpEncoder {
             }
         }
 
-        // Sort fields by name for consistent schema ID generation
-        fields.sort_by(|a, b| a.0.cmp(&b.0));
-
+        // No sorting - field order affects schema ID calculation
         // Hash field names and types while converting to FieldDef
         let field_defs: Vec<FieldDef> = fields
             .into_iter()

Original file line number	Diff line number	Diff line change
`@@ -213,9 +213,7 @@ impl OtlpEncoder {`
`213`	`213`	`}`
`214`	`214`	`}`
`215`	`215`
`216`		`- // Sort fields by name for consistent schema ID generation`
`217`		`- fields.sort_by(\|a, b\| a.0.cmp(&b.0));`
`218`		`-`
	`216`	`+ // No sorting - field order affects schema ID calculation`
`219`	`217`	`// Hash field names and types while converting to FieldDef`
`220`	`218`	`let field_defs: Vec<FieldDef> = fields`
`221`	`219`	`.into_iter()`