ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-13 03:28:33 -05:00
Code Issues Packages Projects Releases Wiki Activity

Bumps azdataGraph version to 0.0.56 (#21089)

Browse Source 
* Bumps azdataGraph version to 0.0.56

* Removed warning popup since widget only contains avail metrics
This commit is contained in:
Lewis Sanchez
2022-11-03 08:16:20 -07:00
committed by GitHub
parent 580f53d851
commit ca78491238
7 changed files with 15 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/sql/workbench/contrib/executionPlan/browser/widgets/highlightExpensiveNodeWidget.ts
View File

@@ -301,11 +301,9 @@ export class HighlightExpensiveOperationAction extends Action {
const expensiveOperationDelegate: (cell: AzDataGraphCell) => number | undefined = context.getExpensiveOperationDelegate();
context.executionPlanDiagram.clearExpensiveOperatorHighlighting();
let result = context.executionPlanDiagram.highlightExpensiveOperator(expensiveOperationDelegate);
if (!result) {
const metric = context.expenseMetricSelectBox.value;
context.notificationService.warn(localize('invalidPropertyExecutionPlanMetric', 'No nodes found with the {0} metric.', metric));
}
context.executionPlanDiagram.highlightExpensiveOperator(expensiveOperationDelegate);
// lewissanchez TODO: Add focus logic to center the highlighted node. Removed the error message logic since the expensive operation widget
// is only populated with plan metrics that are contained in a plan.
}
}