Skip to content

Fix window_functions docs formatting#17005

Merged
comphead merged 2 commits into
apache:mainfrom
mattmatravers:patch-1
Aug 1, 2025
Merged

Fix window_functions docs formatting#17005
comphead merged 2 commits into
apache:mainfrom
mattmatravers:patch-1

Conversation

@mattmatravers

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

This is a quick fix while reading the docs page at https://datafusion.apache.org/user-guide/sql/window_functions.html.

Rationale for this change

The Analytical Functions section is malformed.

What changes are included in this PR?

A markdown format fix.

Are these changes tested?

Tested via the github preview page on the PR.

Are there any user-facing changes?

This only affects documentation.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 1, 2025
FROM employees;
```

````sql

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is weird is that this documentation is supposed to be auto generated

https://github.com/apache/datafusion/blob/00132da33d51f272ddb19116121bed9a1420a0a5/datafusion/functions-window/src/row_number.rs#L56-L55

This looks like the problem might be that there are two ```sql` blocks there -- let me try and fix it another way...

@github-actions github-actions Bot added the functions Changes to functions implementation label Aug 1, 2025
@alamb

alamb commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

I took the liberty of updating the source and rerunning the generation scripts -- I think it still fixes the problem and this way it won't be undone on the next PR

Thank you @mattmatravers 🙏

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @mattmatravers 🙏

@comphead
comphead merged commit a078cd2 into apache:main Aug 1, 2025
28 checks passed
@mattmatravers
mattmatravers deleted the patch-1 branch August 1, 2025 22:17
Standing-Man pushed a commit to Standing-Man/datafusion that referenced this pull request Aug 4, 2025
* Fix window_functions docs formatting

* Fix the source and make the docs consistent

---------

Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation functions Changes to functions implementation

3 participants