Skip to content

Add documentation for new SQL functions available for Log Workspaces #28643

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Apr 11, 2025

Conversation

vishalshah-dd
Copy link
Contributor

@vishalshah-dd vishalshah-dd commented Apr 8, 2025

What does this PR do? What is the motivation?

A follow-up of #28591 where we're adding documentation for new functions that are now supported in Log Workspaces. This is in preparation for our GA launch.

Adding documentation for the following functions:

  • cardinality
  • array_position
  • strpos
  • string_to_array
  • split_part
  • array_agg
  • unnest

Jira: https://datadoghq.atlassian.net/browse/LOGSQ-830

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:
Merge queue is enabled in this repo. Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass in CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

To have your PR automatically merged after it receives the required reviews, add the following PR comment:

/merge

Additional notes

Sorry, something went wrong.

Copy link
Contributor

github-actions bot commented Apr 8, 2025

Preview links (active after the build_preview check completes)

Modified Files

@vishalshah-dd vishalshah-dd marked this pull request as ready for review April 8, 2025 18:59
@vishalshah-dd vishalshah-dd requested a review from a team as a code owner April 8, 2025 18:59
Base automatically changed from vishalshah/update-sql-reference to master April 10, 2025 21:35
@vishalshah-dd vishalshah-dd force-pushed the vishalshah/add-missing-functions-sql-reference branch from 691f9e7 to 46f21bc Compare April 10, 2025 21:55
@vishalshah-dd
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Apr 11, 2025

View all feedbacks in Devflow UI.

2025-04-11 15:57:12 UTC ℹ️ Start processing command /merge


2025-04-11 15:57:18 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 18m (p90).


2025-04-11 16:13:14 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 80f9976 into master Apr 11, 2025
15 of 16 checks passed
@dd-mergequeue dd-mergequeue bot deleted the vishalshah/add-missing-functions-sql-reference branch April 11, 2025 16:13
estherk15 pushed a commit that referenced this pull request Apr 17, 2025
…28643)

* Add documentation for CARDINALITY function

* Add documentation for ARRAY_POSITION function

* Add documentation for STRPOS function

* Add documentation for STRING_TO_ARRAY function

* Add documentation for SPLIT_PART function

* Add documentation for ARRAY_AGG function

* Add documentation for UNNEST function

* Edits based on review comments

* Change return type for unnest and add missing period
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants