[user_events] Fix exporting attribute type in double (open-telemetry#534)

ThomsonTan · web-flow · commit fafd16d85a34 · 2025-03-12T11:18:39.000-07:00
diff --git a/exporters/user_events/src/recordable.cc b/exporters/user_events/src/recordable.cc
@@ -41,7 +41,7 @@ void Recordable::SetBody(const opentelemetry::common::AttributeValue &message) n
     return;
   }
 
-  auto event_name = !event_name_.empty() ? event_name_.data() : "OpenTelemetryLogs";
+  auto event_name = !event_name_.empty() ? event_name_.data() : "Logs";
 
   event_builder_.Reset(event_name);
   utils::PopulateAttribute("__csver__", static_cast<uint16_t>(0x400), event_builder_);
diff --git a/exporters/user_events/src/utils.cc b/exporters/user_events/src/utils.cc
@@ -69,7 +69,7 @@ void PopulateAttribute(nostd::string_view key,
   {
     event_builder.AddValue(key_name, nostd::get<uint64_t>(value), event_field_format_default);
   }
-  else if (nostd::holds_alternative<double>(value), event_field_format_default)
+  else if (nostd::holds_alternative<double>(value))
   {
     event_builder.AddValue(key_name, nostd::get<double>(value), event_field_format_float);
   }

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ void Recordable::SetBody(const opentelemetry::common::AttributeValue &message) n`
`41`	`41`	`return;`
`42`	`42`	`}`
`43`	`43`
`44`		`- auto event_name = !event_name_.empty() ? event_name_.data() : "OpenTelemetryLogs";`
	`44`	`+ auto event_name = !event_name_.empty() ? event_name_.data() : "Logs";`
`45`	`45`
`46`	`46`	`event_builder_.Reset(event_name);`
`47`	`47`	`utils::PopulateAttribute("__csver__", static_cast<uint16_t>(0x400), event_builder_);`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ void PopulateAttribute(nostd::string_view key,`
`69`	`69`	`{`
`70`	`70`	`event_builder.AddValue(key_name, nostd::get<uint64_t>(value), event_field_format_default);`
`71`	`71`	`}`
`72`		`- else if (nostd::holds_alternative<double>(value), event_field_format_default)`
	`72`	`+ else if (nostd::holds_alternative<double>(value))`
`73`	`73`	`{`
`74`	`74`	`event_builder.AddValue(key_name, nostd::get<double>(value), event_field_format_float);`
`75`	`75`	`}`