Skip to content

[Data] - Use plain mode instead of grid mode for OpMetrics logging#59907

Merged
alexeykudinkin merged 2 commits intoray-project:masterfrom
goutamvenkat-anyscale:goutam/use_plain
Jan 9, 2026
Merged

[Data] - Use plain mode instead of grid mode for OpMetrics logging#59907
alexeykudinkin merged 2 commits intoray-project:masterfrom
goutamvenkat-anyscale:goutam/use_plain

Conversation

@goutamvenkat-anyscale
Copy link
Contributor

Description

The grid is not being rendered correctly in some logs. Instead, this changes opts for a simpler representation from tabulate.

Related issues

Link related issues: "Fixes #1234", "Closes #1234", or "Related to #1234".

Additional information

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

Signed-off-by: Goutam <goutam@anyscale.com>
@goutamvenkat-anyscale goutamvenkat-anyscale requested a review from a team as a code owner January 7, 2026 01:13
@goutamvenkat-anyscale goutamvenkat-anyscale added data Ray Data-related issues go add ONLY when ready to merge, run all tests labels Jan 7, 2026
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 changes the table format for operator metrics from simple_outline to plain to address rendering issues in some logs. The change is straightforward, and the corresponding test case for _format_metrics_table has been correctly updated to reflect the new plain text format. The changes are correct and effectively resolve the described problem.

@goutamvenkat-anyscale goutamvenkat-anyscale changed the title [Data] - Use plain instead of grid for OpMetrics [Data] - Use plain mode instead of grid mode for OpMetrics logging Jan 7, 2026
Copy link
Contributor

@srinathk10 srinathk10 left a comment

Choose a reason for hiding this comment

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

LGTM!

@alexeykudinkin alexeykudinkin merged commit 1add36e into ray-project:master Jan 9, 2026
6 checks passed
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
…ay-project#59907)

## Description
The grid is not being rendered correctly in some logs. Instead, this
changes opts for a simpler representation from tabulate.

## Related issues
> Link related issues: "Fixes ray-project#1234", "Closes ray-project#1234", or "Related to
ray-project#1234".

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

Signed-off-by: Goutam <goutam@anyscale.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
lee1258561 pushed a commit to pinterest/ray that referenced this pull request Feb 3, 2026
…ay-project#59907)

## Description
The grid is not being rendered correctly in some logs. Instead, this
changes opts for a simpler representation from tabulate.

## Related issues
> Link related issues: "Fixes ray-project#1234", "Closes ray-project#1234", or "Related to
ray-project#1234".

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

Signed-off-by: Goutam <goutam@anyscale.com>
ryanaoleary pushed a commit to ryanaoleary/ray that referenced this pull request Feb 3, 2026
…ay-project#59907)

## Description
The grid is not being rendered correctly in some logs. Instead, this
changes opts for a simpler representation from tabulate.

## Related issues
> Link related issues: "Fixes ray-project#1234", "Closes ray-project#1234", or "Related to
ray-project#1234".

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

Signed-off-by: Goutam <goutam@anyscale.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
…ay-project#59907)

## Description
The grid is not being rendered correctly in some logs. Instead, this
changes opts for a simpler representation from tabulate.

## Related issues
> Link related issues: "Fixes ray-project#1234", "Closes ray-project#1234", or "Related to
ray-project#1234".

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

Signed-off-by: Goutam <goutam@anyscale.com>
Signed-off-by: peterxcli <peterxcli@gmail.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
…ay-project#59907)

## Description
The grid is not being rendered correctly in some logs. Instead, this
changes opts for a simpler representation from tabulate.

## Related issues
> Link related issues: "Fixes ray-project#1234", "Closes ray-project#1234", or "Related to
ray-project#1234".

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

Signed-off-by: Goutam <goutam@anyscale.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.

Ray fails to serialize self-reference objects

3 participants