[otap-dataflow] Debug Exporter#644
Conversation
…, takes in otlp data and outputs information to console
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #644 +/- ##
==========================================
+ Coverage 72.46% 73.43% +0.97%
==========================================
Files 304 310 +6
Lines 49258 51457 +2199
==========================================
+ Hits 35695 37789 +2094
- Misses 13029 13134 +105
Partials 534 534
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new debug exporter to the otlp crate that outputs OTLP objects to stdout with configurable verbosity levels. Key changes include:
- Integration of the new debug_exporter module into lib.rs.
- Addition of a Verbosity enum and a marshaler implementation for logging OTLP data in various formats.
- Implementation of a DebugExporter that handles different OTLP data types and configuration-related control messages.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| rust/otap-dataflow/crates/otlp/src/lib.rs | Added a module reference for the new debug exporter. |
| rust/otap-dataflow/crates/otlp/src/debug_exporter/verbosity.rs | Introduces the Verbosity enum to control output detail. |
| rust/otap-dataflow/crates/otlp/src/debug_exporter/mod.rs | Exposes exporter, marshaler, and verbosity modules. |
| rust/otap-dataflow/crates/otlp/src/debug_exporter/marshaler.rs | Implements a normal marshaler to convert OTLP messages into string reports. |
| rust/otap-dataflow/crates/otlp/src/debug_exporter/exporter.rs | Implements the DebugExporter and its associated push functions for different OTLP data types. |
Comments suppressed due to low confidence (1)
rust/otap-dataflow/crates/otlp/src/debug_exporter/exporter.rs:151
- The variable 'resouce_metrics' appears to be misspelled. Consider renaming it to 'resource_metrics' for better clarity.
let resouce_metrics = metric_request.resource_metrics.len();
Example Output => Basic VerbosityTimer tick received Example Output => Normal VerbosityTimer tick received Example Output => Detailed VerbosityTimer tick received |
lquerel
left a comment
There was a problem hiding this comment.
Suggested few modifications. Once updated, we can merge this PR.
…utput it at the end when the exporter shuts down
Add a debug exporter to the otlp crate The exporter will output to stdout various levels of verbosity (basic -> normal -> detailed) on the otlp objects going through the pipeline
Add a debug exporter to the otlp crate
The exporter will output to stdout various levels of verbosity (basic -> normal -> detailed) on the otlp objects going through the pipeline