Skip to content

[Data] Fix broken link in read_unity_catalog docstring#59745

Merged
richardliaw merged 2 commits into
ray-project:masterfrom
kongjy:fix-uc-link
Jan 7, 2026
Merged

[Data] Fix broken link in read_unity_catalog docstring#59745
richardliaw merged 2 commits into
ray-project:masterfrom
kongjy:fix-uc-link

Conversation

@kongjy

@kongjy kongjy commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

Description

Fixed a broken link in the read_unity_catalog doc string. Previous URL was outdated.

Related issues

None

Additional information

N/A

Signed-off-by: Jess <jessica.jy.kong@gmail.com>
@kongjy kongjy requested a review from a team as a code owner December 29, 2025 20:35

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates a broken link in the read_unity_catalog docstring. The fix is correct in pointing to a more relevant page about credential vending. I've suggested a small improvement to use a more general documentation link that covers all supported cloud providers (AWS, Azure, and GCP) instead of just AWS.

Comment thread python/ray/data/read_api.py Outdated
@richardliaw richardliaw added the go add ONLY when ready to merge, run all tests label Dec 29, 2025
@ray-gardener ray-gardener Bot added docs An issue or change related to documentation data Ray Data-related issues labels Dec 30, 2025
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Jessica Kong <jessica.jy.kong@gmail.com>
@richardliaw richardliaw merged commit a6fcca4 into ray-project:master Jan 7, 2026
6 checks passed
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
…9745)

## Description
Fixed a broken link in the read_unity_catalog doc string. Previous URL
was outdated.

## Related issues
None

## Additional information
N/A

---------

Signed-off-by: Jess <jessica.jy.kong@gmail.com>
Signed-off-by: Jessica Kong <jessica.jy.kong@gmail.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Ray Data-related issues docs An issue or change related to documentation go add ONLY when ready to merge, run all tests

3 participants