Skip to content

[Data] Update read_lance doc#59673

Merged
bveeramani merged 2 commits intoray-project:masterfrom
owenowenisme:data/update-lance-doc
Dec 28, 2025
Merged

[Data] Update read_lance doc#59673
bveeramani merged 2 commits intoray-project:masterfrom
owenowenisme:data/update-lance-doc

Conversation

@owenowenisme
Copy link
Member

@owenowenisme owenowenisme commented Dec 26, 2025

Description

Related issues

Closes #59652

Additional information

Optional: Add implementation details, API changes, usage examples, screenshots, etc.

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

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 the docstrings in read_lance to use the new documentation URLs for the Lance project and improves the description of the filter parameter. The changes are good and improve the documentation's accuracy. I've suggested a couple of minor tweaks to further improve clarity and provide more direct links in the docstrings.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: You-Cheng Lin <106612301+owenowenisme@users.noreply.github.com>
@owenowenisme owenowenisme marked this pull request as ready for review December 27, 2025 08:35
@owenowenisme owenowenisme requested a review from a team as a code owner December 27, 2025 08:35
@owenowenisme owenowenisme added docs An issue or change related to documentation data Ray Data-related issues go add ONLY when ready to merge, run all tests labels Dec 27, 2025
@bveeramani bveeramani merged commit ac42d1b into ray-project:master Dec 28, 2025
7 checks passed
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
## Description

## Related issues
Closes ray-project#59652

## Additional information
> Optional: Add implementation details, API changes, usage examples,
screenshots, etc.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: You-Cheng Lin <106612301+owenowenisme@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
lee1258561 pushed a commit to pinterest/ray that referenced this pull request Feb 3, 2026
## Description


## Related issues
Closes ray-project#59652

## Additional information
> Optional: Add implementation details, API changes, usage examples,
screenshots, etc.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: You-Cheng Lin <106612301+owenowenisme@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
## Description

## Related issues
Closes ray-project#59652

## Additional information
> Optional: Add implementation details, API changes, usage examples,
screenshots, etc.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: You-Cheng Lin <106612301+owenowenisme@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: peterxcli <peterxcli@gmail.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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Data] Improve Read Lance doc

2 participants