Skip to content
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

Added "time: ###" and "###% of ActualRows vs EstimateRows" #116

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added "time: ###" and "###% of ActualRows vs EstimateRows" #116

wants to merge 2 commits into from

Conversation

goranschwarz
Copy link

  • in "qp-node" Added: "Time: #### s" when the element: "s:RunTimeInformation/s:RunTimeCountersPerThread/@ActualElapsedms" exists
  • in "qp-node" Added: "A={$ActualRows} of E={$EstimateRows} (###%)" when the element "s:RunTimeInformation/s:RunTimeCountersPerThread/@ActualRows" exists
  • in "qp-node" When it's a "Eager Index Spool" -- Added class "qp-node-label-2-warning", which for the moment is in color red (to "warn" about this operator, so it stands out a bit... Possibly we want to create an index here...)
  • Added: Normal HTML Tooltip (<div title="tooltipContent") on missing indexes (in some cases the text is truncated... But the tooltip will show them, so you do not have to copy past the rows)

@goranschwarz
in "qp-node" Added: "Time: #### s" when the element: "s:RunTimeInform…
077d312 
…ation/s:RunTimeCountersPerThread/@ActualElapsedms" exists

in "qp-node" Added: "A={$ActualRows} of E={$EstimateRows} (###%)" when the element "s:RunTimeInformation/s:RunTimeCountersPerThread/@ActualRows" exists
in "qp-node" When it's a "Eager Index Spool" -- Added class "qp-node-label-2-warning", which for the moment is in color red (to "warn" about this operator, so it stands out a bit... Possibly we want to create an index here...)
Added: Normal HTML Tooltip (<div title="tooltipContent">) on missing indexes (in some cases the text is truncated... But the tooltip will show them, so you do not have to copy past the rows)
@goranschwarz
Some additional changes
b4051a3 
* Added 'MissingIndexForEagerIndexSpool' in the Missing index section -- That is if we have any "Eager Index Spool", then extract what a good index would look like for that...
* Added 'NodeId' at the top right corner of each operator
* In the base operator 'left most'.
  * Added "NonParallelPlanReason" in the tooltip section.
  * Added Timings for "ElapedTime", "CpuTime", "UdfElapsedTime" & "UdfCpuTime"
  * Top 10 Waits for the statement are printed in a table sorder by WaitTimeMs
  * Parameters Compile and Runtime are printed in a table
* In any operator
  * Added "CardinalityPct" and if the estimate and actual are 'way off', it will be printed with a red background
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@goranschwarz