Skip to content

Ensure readability of plan printer and add test#24710

Merged
shangm2 merged 1 commit intoprestodb:masterfrom
shangm2:planPrinterFormatting
Mar 12, 2025
Merged

Ensure readability of plan printer and add test#24710
shangm2 merged 1 commit intoprestodb:masterfrom
shangm2:planPrinterFormatting

Conversation

@shangm2
Copy link
Contributor

@shangm2 shangm2 commented Mar 11, 2025

Description

  1. Make sure the printout from plan printer is easy to read
  2. Add test

Motivation and Context

  1. Make sure the printout from plan printer is easy to read

Impact

Test Plan

  1. passed the verifier test https://our.internmc.facebook.com/intern/presto/verifier/results/?test_id=213849
Screenshot 2025-03-11 at 15 34 14

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@shangm2 shangm2 requested a review from presto-oss March 11, 2025 22:35
@prestodb-ci prestodb-ci added the from:Meta PR from Meta label Mar 11, 2025
@shangm2 shangm2 merged commit 22acbde into prestodb:master Mar 12, 2025
92 checks passed
unidevel pushed a commit to unix280/presto that referenced this pull request Mar 12, 2025
@prestodb-ci prestodb-ci mentioned this pull request Mar 28, 2025
30 tasks
pradeepvaka pushed a commit to pradeepvaka/presto that referenced this pull request Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:Meta PR from Meta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants