Skip to content

chore(sql): Update SQL reference with additional functions#6359

Merged
sanderson merged 13 commits intomasterfrom
sql-updates
Sep 5, 2025
Merged

chore(sql): Update SQL reference with additional functions#6359
sanderson merged 13 commits intomasterfrom
sql-updates

Conversation

@sanderson
Copy link
Collaborator

Closes https://github.com/influxdata/DAR/issues/476

Updates SQL function reference docs with map, array, and struct functions as well as a few other additions.


  • Rebased/mergeable

This comment was marked as outdated.

@sanderson sanderson requested a review from Copilot September 4, 2025 18:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the SQL function reference documentation with additional functions and improvements to existing content. The updates include adding new array, map, and struct functions, introducing previously missing conditional functions, and standardizing heading levels throughout the documentation.

  • Adds comprehensive documentation for array functions including manipulation, searching, and conversion operations
  • Introduces map functions for creating and operating on Arrow maps
  • Adds struct functions for working with Arrow structs
  • Updates conditional functions to include greatest and least functions

Reviewed Changes

Copilot reviewed 92 out of 92 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
content/shared/sql-reference/functions/array.md New comprehensive array function documentation with 96 functions
content/shared/sql-reference/functions/map.md New map function documentation covering creation and operations
content/shared/sql-reference/functions/struct.md New struct function documentation for creating Arrow structs
content/shared/sql-reference/functions/conditional.md Updated to include previously commented greatest and least functions
content/shared/sql-reference/functions/misc.md Added get_field function documentation
Various function pages Standardized heading levels from ##### to ### for Arguments sections
Various product pages Created new navigation entries for array, map, and struct functions

Copy link
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

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

  • Fixes some confusing/imprecise grammar
  • Fixes some grammar typos
  • Fixes what seems to be struct() example using aliases mismatched to results that don't include aliases.

Copy link
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

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

Hit the wrong button. See my previous comment.

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>
@sanderson sanderson requested a review from jstirnaman September 4, 2025 21:46
@sanderson sanderson merged commit 8f729f9 into master Sep 5, 2025
2 checks passed
@sanderson sanderson deleted the sql-updates branch September 5, 2025 17:51
jstirnaman added a commit that referenced this pull request Sep 11, 2025
* feat(sql): WIP added struct and map functions

* chore(sql): update sql reference with new functions

* chore(sql): migrate sql reference function updates to other projects

* chore(sql): readd deleted cache functions pages

* Update content/shared/sql-reference/functions/array.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix(sql): fixed broken anchor links

* fix(sql): fixed typos

* Apply suggestions from code review

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>
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.

3 participants