Skip to content

Add attribute precedence table for log pipelines/processors#38026

Open
10kaoru12 wants to merge 4 commits into
masterfrom
kaoru.ishizawa/logs-pipeline-attribute-precedence
Open

Add attribute precedence table for log pipelines/processors#38026
10kaoru12 wants to merge 4 commits into
masterfrom
kaoru.ishizawa/logs-pipeline-attribute-precedence

Conversation

@10kaoru12

Copy link
Copy Markdown

What does this PR do? What is the motivation?

Adds an Attribute precedence section to the Log Pipelines page that explains, in one place, what happens when the same attribute is set by more than one processor across matching pipelines.

Today this behavior is only documented piecemeal on individual processor pages (for example, the "only the first one is considered" note on the log status remapper, and the "last one is taken into account" note on the log date remapper), and several processors document nothing at all. Customers configuring multiple pipelines/processors have no single reference for which write wins.

The new section groups all processors into three behaviors:

  • Last write wins (Grok parser, Category, Arithmetic, String builder, Lookup, URL/User-Agent/GeoIP parsers, Decoder)
  • Depends on override_on_conflict (Remapper, Array processor; default false = do not override)
  • First write wins (Log status/Service/Log message/Trace/Span remappers; the Log date remapper is the exception where the last one wins)

Wording is reused verbatim from the existing processor pages to stay consistent.

Merge readiness

  • Ready for merge

For Datadog employees:

  • Branch follows the <name>/<description> convention.

AI assistance

Drafted with Claude Code. Wording was reused verbatim from existing processor pages (log_status_remapper, service_remapper, log_message_remapper, log_date_remapper, remapper). The precedence behavior was independently verified against the logs-backend source before writing.

Additional notes

English (content/en) only, per the contributing guidelines (translated files are managed by the translation service). Opened as a draft to review the preview build and Vale results before marking ready.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@10kaoru12 10kaoru12 marked this pull request as ready for review July 8, 2026 08:21
@10kaoru12 10kaoru12 requested a review from a team as a code owner July 8, 2026 08:21
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@estherk15 estherk15 self-assigned this Jul 8, 2026
@estherk15

Copy link
Copy Markdown
Contributor

Thanks for the update @10kaoru12, I moved the section down the page so that it follows the processor information(#38042). Let me know if you have any questions

estherk15 and others added 2 commits July 9, 2026 07:22
…38042)

- Relocate attribute precedence section after processor configuration steps for better logical flow
- Clarify log date remapper exception upfront in "First write wins" description
- Update link reference to use consistent documentation format [3]

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
@10kaoru12

Copy link
Copy Markdown
Author

@estherk15
Thanks for the update! I've approved #38042. Let me know if there's anything else needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants