Skip to content

Add Microsoft Common Schema log processor#2788

Open
lalitb wants to merge 6 commits intoopen-telemetry:mainfrom
lalitb:split/microsoft-common-schema-processor
Open

Add Microsoft Common Schema log processor#2788
lalitb wants to merge 6 commits intoopen-telemetry:mainfrom
lalitb:split/microsoft-common-schema-processor

Conversation

@lalitb
Copy link
Copy Markdown
Member

@lalitb lalitb commented Apr 30, 2026

Change Summary

Split out from the original combined PR #2717.

Adds a contrib processor that promotes decoded Microsoft Common Schema log attributes into typed OTLP log fields.

This processor is independent of the user_events receiver. It expects Common Schema fields to already be decoded into log attributes, for example from EventHeader payloads.

Also adds unit tests for supported promotion behavior and pass-through cases.

What issue does this PR close?

How are these changes tested?

Unit tests

Are there any user-facing changes?

Yes, adds a processor.

@lalitb lalitb marked this pull request as ready for review April 30, 2026 06:43
@lalitb lalitb requested a review from a team as a code owner April 30, 2026 06:43
@github-actions github-actions Bot added the rust Pull requests that update Rust code label Apr 30, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

❌ Patch coverage is 87.37864% with 91 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.04%. Comparing base (f018901) to head (510e3b7).

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #2788    +/-   ##
========================================
  Coverage   86.03%   86.04%            
========================================
  Files         704      705     +1     
  Lines      264591   265312   +721     
========================================
+ Hits       227654   228280   +626     
- Misses      36413    36508    +95     
  Partials      524      524            
Components Coverage Δ
otap-dataflow 86.97% <87.37%> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.76% <ø> (ø)
otel-arrow-go 52.45% <ø> (ø)
quiver 92.25% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@albertlockett albertlockett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM @lalitb . One minor comment for improving perf in arrow_log_attrs_have_key simply for your consideration

@albertlockett albertlockett enabled auto-merge May 1, 2026 20:50
@albertlockett albertlockett disabled auto-merge May 1, 2026 20:53
@albertlockett albertlockett enabled auto-merge May 1, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rust Pull requests that update Rust code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants