Skip to content

[recordset_kql_processor] Logging tweaks#2074

Merged
drewrelmas merged 1 commit intoopen-telemetry:mainfrom
CodeBlanch:kql-recordset-processor-log-tweaks-2
Feb 20, 2026
Merged

[recordset_kql_processor] Logging tweaks#2074
drewrelmas merged 1 commit intoopen-telemetry:mainfrom
CodeBlanch:kql-recordset-processor-log-tweaks-2

Conversation

@CodeBlanch
Copy link
Copy Markdown
Member

Relates to #2066

Changes

  • Switch to logging each diagnostic returned from engine individually (instead of logging a single formatted version).
  • Tweaks to the other logs in the processor for consistency.

@CodeBlanch CodeBlanch requested a review from a team as a code owner February 19, 2026 23:42
@github-actions github-actions Bot added the rust Pull requests that update Rust code label Feb 19, 2026
"recordset_kql_processor.success",
input_items = input_items,
output_items = output_items,
input_items,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice simplification!

Copy link
Copy Markdown
Member

@cijothomas cijothomas left a comment

Choose a reason for hiding this comment

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

good improvements

@drewrelmas drewrelmas enabled auto-merge February 19, 2026 23:47
@drewrelmas drewrelmas added this pull request to the merge queue Feb 20, 2026
Merged via the queue into open-telemetry:main with commit 125bbb6 Feb 20, 2026
60 checks passed
@drewrelmas drewrelmas deleted the kql-recordset-processor-log-tweaks-2 branch February 20, 2026 00:24
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 20, 2026

Codecov Report

❌ Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.97%. Comparing base (97d1659) to head (a7ea7ed).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2074      +/-   ##
==========================================
- Coverage   86.97%   86.97%   -0.01%     
==========================================
  Files         536      536              
  Lines      172726   172726              
==========================================
- Hits       150236   150231       -5     
- Misses      21956    21961       +5     
  Partials      534      534              
Components Coverage Δ
otap-dataflow 89.11% <0.00%> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.33% <ø> (ø)
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 53.50% <ø> (ø)
quiver 91.73% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

cijothomas pushed a commit to cijothomas/otel-arrow that referenced this pull request Feb 23, 2026
Relates to open-telemetry#2066

# Changes

* Switch to logging each diagnostic returned from engine individually
(instead of logging a single formatted version).
* Tweaks to the other logs in the processor for consistency.
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: Done

Development

Successfully merging this pull request may close these issues.

3 participants