Skip to content

fix: Live plan not showing and output data size is undefined #26264

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 9, 2025
Merged

fix: Live plan not showing and output data size is undefined #26264

merged 3 commits into from
Oct 9, 2025
+40 −43

Conversation

unidevel
Copy link
Contributor

@unidevel unidevel commented Oct 9, 2025

Description

  1. Fix live plan not showing
image

  1. Fix warnings in Query Header

  2. Fix output data size is undefined - due to this change , outputDataSize is changed to outputDataSizeInBytes

Motivation and Context

PR for master branch: #26033

Impact

UI

Test Plan

Manual test

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

@unidevel unidevel requested review from yhwang and a team as code owners October 9, 2025 16:27
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Oct 9, 2025
@prestodb-ci prestodb-ci requested review from a team, pramodsatya and nishithakbhaskaran and removed request for a team October 9, 2025 16:27
@unidevel unidevel mentioned this pull request Oct 9, 2025
Merged
6 tasks
@unidevel
Copy link
Contributor Author

unidevel commented Oct 9, 2025

Tested locally, seems fine.

@unidevel
Copy link
Contributor Author

unidevel commented Oct 9, 2025

@tdcmeehan seems I have to cherry pick your free space fix in PR #26216

@unidevel unidevel requested a review from czentgr as a code owner October 9, 2025 17:13
@unidevel unidevel merged commit 9f26e77 into prestodb:release-0.295 Oct 9, 2025
74 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tdcmeehan tdcmeehan tdcmeehan approved these changes

@rschlussel rschlussel rschlussel approved these changes

@yhwang yhwang Awaiting requested review from yhwang yhwang is a code owner

@pramodsatya pramodsatya Awaiting requested review from pramodsatya pramodsatya was automatically assigned from prestodb/ibm-reviewers

@nishithakbhaskaran nishithakbhaskaran Awaiting requested review from nishithakbhaskaran nishithakbhaskaran was automatically assigned from prestodb/ibm-reviewers

@czentgr czentgr Awaiting requested review from czentgr czentgr is a code owner

Assignees
No one assigned
Labels
from:IBM PR from IBM
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@unidevel @tdcmeehan @rschlussel @prestodb-ci