Skip to content

Conversation

mhintz
Copy link
Contributor

@mhintz mhintz commented Aug 26, 2025

What does this PR do? What is the motivation?

I'd like to add an example of what happens in DDSQL when a resource has multiple values for the same tag key. This is a potential stumbling-block for customers and I'd like to be sure that it's well-documented in case customers run into this

Merge instructions

Ready to go

Merge readiness:

  • Ready for merge

I'd like to add an example of what happens in DDSQL when a resource has multiple values for the same tag key. This is a potential stumbling-block for customers and I'd like to be sure that it's well-documented in case customers run into this.
Copy link
Contributor

github-actions bot commented Aug 26, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

@mhintz mhintz requested review from a team as code owners August 26, 2025 20:17
@mhintz mhintz requested a review from a team as a code owner August 26, 2025 20:19
@github-actions github-actions bot added the Github Related to Github configurations label Aug 26, 2025
@@ -165,7 +165,7 @@ content/en/code_analysis/ @Datadog/static-analysis @Data
content/en/quality_gates/ @Datadog/ci-app-backend @Datadog/documentation

# DDSQL Editor References
content/en/ddsql_reference/*.md @Datadog/xpq @Datadog/documentation @Datadog/advanced-query-guild
content/en/ddsql_reference/*.md @Datadog/documentation @Datadog/advanced-query-guild
Copy link
Contributor Author

Choose a reason for hiding this comment

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

xpq reps are part of the advanced-query-guild group, so it's redundant to include them here twice

Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Contributor

@brett0000FF brett0000FF left a comment

Choose a reason for hiding this comment

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

Looks good! Just left a minor suggestion to switch future tense to present tense.

@mhintz
Copy link
Contributor Author

mhintz commented Aug 27, 2025

/merge

@dd-devflow-routing-codex
Copy link

dd-devflow-routing-codex bot commented Aug 27, 2025

View all feedbacks in Devflow UI.

2025-08-27 12:38:00 UTC ℹ️ Start processing command /merge


2025-08-27 12:38:06 UTC ℹ️ MergeQueue: queue is disabled

Added to the queue but the mergequeue is not enabled for now.


2025-08-27 21:02:05 UTC ℹ️ MergeQueue: This merge request was already merged

This pull request was merged directly.

@mhintz mhintz merged commit 228e912 into master Aug 27, 2025
18 checks passed
@mhintz mhintz deleted the mhintz/ddsql-add-multi-tags-example branch August 27, 2025 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Github Related to Github configurations mergequeue-status: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants