-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change structs in otlpgrpc to follow standard go encoding interfaces #5062
Conversation
6e7d9b6
to
2a74e47
Compare
Codecov Report
@@ Coverage Diff @@
## main #5062 +/- ##
=======================================
Coverage 91.01% 91.02%
=======================================
Files 180 180
Lines 10814 10826 +12
=======================================
+ Hits 9842 9854 +12
- Misses 752 753 +1
+ Partials 220 219 -1
Continue to review full report at Codecov.
|
* Change `UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]` to be a func `UnmarshalJSON` on the struct. * Rename `[Traces|Metrics|Logs][Reques|Response].Marshal` to `MarshalProto` consistent with JSON/Text standard interfaces. * Change `UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]` to be a func `UnmarshalProto` on the struct. Signed-off-by: Bogdan Drutu <[email protected]>
a3473a8
to
7afe523
Compare
…pen-telemetry#5062) * Change `UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]` to be a func `UnmarshalJSON` on the struct. * Rename `[Traces|Metrics|Logs][Reques|Response].Marshal` to `MarshalProto` consistent with JSON/Text standard interfaces. * Change `UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]` to be a func `UnmarshalProto` on the struct. Signed-off-by: Bogdan Drutu <[email protected]>
…ated by prometheus, rather than making our own (open-telemetry#3989) * Revert "receiver/prometheus: glue and complete staleness marking for disappearing metrics (open-telemetry#3423)" This reverts commit 8b79380. * Revert "receiver/prometheus: add store to track stale metrics (open-telemetry#3414)" This reverts commit cdc1634. * stop dropping staleness markers from prometheus, and fix tests * add staleness end to end test from open-telemetry#3423 * fix import grouping
UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]
to be a funcUnmarshalJSON
on the struct.[Traces|Metrics|Logs][Reques|Response].Marshal
toMarshalProto
consistent with JSON/Text standard interfaces.UnmarshalJSON[Traces|Metrics|Logs][Reques|Response]
to be a funcUnmarshalProto
on the struct.Signed-off-by: Bogdan Drutu [email protected]