W-23900223 : Hyper Query Plan - Replace raw property dumps with tidy node rows. - #173
Open
rbrahmachary wants to merge 1 commit into
Open
W-23900223 : Hyper Query Plan - Replace raw property dumps with tidy node rows.#173rbrahmachary wants to merge 1 commit into
rbrahmachary wants to merge 1 commit into
Conversation
Node bodies previously dumped debug-name, statistics, and sqlpos verbatim via forceToString, producing noisy JSON blobs. Surface the useful figures as clean, curated rows instead. - Replace the raw dump with curated rows: table-or-alias (from debug-name.value), estimated-rows, output-rows, processed-rows, rows-matching, memory-bytes, cpu-cycles, execution-time - New : Add a grouped table-metadata block (identifier / partitioned-by /sort-order) for lakehouse scans, rendered as a header plus sub-items - Show memory-bytes in binary units via a new formatBytes helper (KiB/MiB/GiB), clearer than the decimal formatMetric - Display row order: identity/volume first, operator identifiers last - Echo the runtime hotspot flag onto the cpu-cycles row, reusing the node heat color so the tint intensity matches the label - Add getStatistic / getEstimatedRows / getActualRows helpers that read statistics and fall back to legacy analyze / cardinality - Add a firstNumber helper that skips non-numeric values instead of masking a real number further down the fallback chain - Reuse those helpers for the edge label, hotspot heatmap, and errored-icon detection, keeping tooltip figures and visual cues consistent (including on legacy-format plans) - Extract the hotspot and cardinality-mismatch magic numbers into constants - Add groupedProperties / highlightedProperties to TreeNode and the grouped-header / highlighted-row rendering in QueryNode
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hyper Query Plan - Replace raw property dumps with tidy node rows.
Node bodies previously dumped debug-name, statistics, and sqlpos verbatim via forceToString, producing noisy JSON blobs. Surface the useful figures as clean, curated rows instead.
Query Graph node rows:-


