Skip to content

Conversation

@steveburnett
Copy link
Contributor

@steveburnett steveburnett commented Jun 27, 2025

Description

Add documentation for verbose_optimizer_info_enabled and optimizers_to_enable_verbose_runtime_stats session properties. Fixes #25452.

Motivation and Context

These two session properties are undocumented and are useful for debugging optimizer-related problems in development.

Impact

Documentation.

Test Plan

Local doc build. Screenshot:
Screenshot 2025-06-27 at 3 50 11 PM

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 ==

@steveburnett steveburnett self-assigned this Jun 27, 2025
@steveburnett steveburnett requested review from a team and elharo as code owners June 27, 2025 19:33
@github-project-automation github-project-automation bot moved this to 🆕 Unprioritized in Presto Documentation Jun 27, 2025
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Jun 27, 2025
@prestodb-ci prestodb-ci requested review from a team, Joe-Abraham and Mariamalmesfer and removed request for a team June 27, 2025 19:34
@steveburnett steveburnett marked this pull request as draft June 27, 2025 19:34
@steveburnett steveburnett force-pushed the steveburnett-optimizer-properties branch from c857c6a to eff6309 Compare June 27, 2025 19:51
@steveburnett steveburnett moved this from 🆕 Unprioritized to 👀 Review in Presto Documentation Jun 27, 2025
@steveburnett steveburnett marked this pull request as ready for review June 27, 2025 19:52
@prestodb-ci prestodb-ci requested a review from a team June 27, 2025 19:52
tdcmeehan
tdcmeehan previously approved these changes Jun 27, 2025
@github-project-automation github-project-automation bot moved this from 👀 Review to ✅ Done in Presto Documentation Jun 27, 2025
Copy link
Member

@hantangwangd hantangwangd left a comment

Choose a reason for hiding this comment

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

Thanks for adding this, just one nit.

@imjalpreet
Copy link
Member

@steveburnett, a fix has been added to trigger a dummy run instead of skipping the workflow for the required tests that are currently marked as Waiting for status to be reported above. Please try rebasing all pending docs PRs on the latest master.

@steveburnett
Copy link
Contributor Author

@steveburnett, a fix has been added to trigger a dummy run instead of skipping the workflow for the required tests that are currently marked as Waiting for status to be reported above. Please try rebasing all pending docs PRs on the latest master.

Will do, thank you!

@steveburnett steveburnett merged commit b5d956a into prestodb:master Jul 1, 2025
106 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs from:IBM PR from IBM

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[docs] Add doc for verbose_optimizer_info_enabled session property

6 participants