Skip to content

[Data] Fix iceberg warning msg#60044

Merged
bveeramani merged 2 commits intoray-project:masterfrom
owenowenisme:data/add-trailing-space-to-read-iceberg-warning-msg
Jan 12, 2026
Merged

[Data] Fix iceberg warning msg#60044
bveeramani merged 2 commits intoray-project:masterfrom
owenowenisme:data/add-trailing-space-to-read-iceberg-warning-msg

Conversation

@owenowenisme
Copy link
Member

@owenowenisme owenowenisme commented Jan 12, 2026

Fixes missing space in the warning message.

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@owenowenisme owenowenisme requested a review from a team as a code owner January 12, 2026 08:13
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 correctly fixes a formatting issue in a warning message by adding a necessary trailing space. I've suggested a minor improvement to combine the string literals into a single f-string for better readability and to prevent similar issues in the future.

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@owenowenisme owenowenisme changed the title [Data] Add trailing space to to read iceberg warning msg [Data] Fix iceberg warning msg Jan 12, 2026
@owenowenisme owenowenisme added data Ray Data-related issues go add ONLY when ready to merge, run all tests labels Jan 12, 2026
@bveeramani bveeramani merged commit 39bc94b into ray-project:master Jan 12, 2026
7 checks passed
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
Fixes missing space in the warning message.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
ryanaoleary pushed a commit to ryanaoleary/ray that referenced this pull request Feb 3, 2026
Fixes missing space in the warning message.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
Fixes missing space in the warning message.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: peterxcli <peterxcli@gmail.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
Fixes missing space in the warning message.

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.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 go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants