Skip to content

Minor: fix link errors in docs - #19088

Merged
comphead merged 1 commit into
apache:mainfrom
alamb:alamb/fix_links2
Dec 4, 2025
Merged

Minor: fix link errors in docs#19088
comphead merged 1 commit into
apache:mainfrom
alamb:alamb/fix_links2

Conversation

@alamb

@alambalamb commented Dec 4, 2025

Copy link
Copy Markdown
Contributor

I had an AI tool (codex) look for broken links in the docs and it found two. I tested both changes locally

@alambalamb added the documentation Improvements or additions to documentation label Dec 4, 2025

@compheadcomphead 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.

Thanks @alamb

@comphead
comphead added this pull request to the merge queueDec 4, 2025
Merged via the queue into apache:main with commit 8dac8f1Dec 4, 2025
6 checks passed
@alamb
alamb deleted the alamb/fix_links2 branch December 4, 2025 17:26
@alamb

alamb commented Dec 4, 2025

Copy link
Copy Markdown
ContributorAuthor

I believe this broke CI - https://github.com/apache/datafusion/actions/runs/19938148766/job/57168950868

I will fix it

@alambalamb mentioned this pull request Dec 4, 2025
@alamb

alamb commented Dec 4, 2025

Copy link
Copy Markdown
ContributorAuthor

@alamb

alamb commented Dec 4, 2025

Copy link
Copy Markdown
ContributorAuthor

show benchmark queue

@alamb-ghbot

Copy link
Copy Markdown

🤖 Hi @alamb, you asked to view the benchmark queue (#19088 (comment)).

JobUserBenchmarksComment
18183_3614381681.shalambcase_whenhttps://github.com/apache/datafusion/pull/18183#issuecomment-3614381681

github-merge-queueBot pushed a commit that referenced this pull request Dec 4, 2025
## Which issue does this PR close?
## Rationale for this change
CI is failing on main. For example see :
https://github.com/apache/datafusion/actions/runs/19938148766/job/57168950868
```diff
diff --git a/docs/source/user-guide/sql/scalar_functions.md b/docs/source/user-guide/sql/scalar_functions.md
index fab6221..c5380b2 100644
--- a/docs/source/user-guide/sql/scalar_functions.md
+++ b/docs/source/user-guide/sql/scalar_functions.md
@@ -2125,7 +2125,7 @@ upper(str)
### `uuid`
-Returns [`UUID v4`](https://en.wikipedia.org/wiki/Universally_unique_identifier#Version_4_%28random%29) string value which is unique per row.
+Returns [`UUID v4`](<https://en.wikipedia.org/wiki/Universally_unique_identifier#Version_4_(random)>) string value which is unique per row.
```sql
uuid()
```
The problem is that I updated the markdown in
#19088 but not the original
code in the function
## What changes are included in this PR?
Fix the issue
## Are these changes tested?
Yes by CI
## Are there any user-facing changes?
I will also file a ticket to track this testing hole:
- #19095
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentationImprovements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@alamb@alamb-ghbot@comphead