Skip to content

Commit

Permalink
[SPARK-49039][UI] Reset checkbox when executor metrics are loaded in …
Browse files Browse the repository at this point in the history
…the Stages tab

### What changes were proposed in this pull request?
This PR aims to reset checkbox when executor metrics are loaded in the Stages tab.

### Why are the changes needed?
The variable `executorSummaryTableSelector` in the `reselectCheckboxesBasedOnTaskTableState` function may not be initialized, which results in inconsistent UI display.

Sometimes `allexecutors` api calls are slow, which causes `executorSummaryTableSelector` to be initialized lazily.
```js
$.getJSON(createRESTEndPointForExecutorsPage(appId),
```

<img width="800" alt="image" src="https://github.com/user-attachments/assets/780705ee-71b2-4063-bee9-cac3a3d35f27">

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
local test

<img width="1299" alt="image" src="https://github.com/user-attachments/assets/b3ee9cc3-088f-490f-a52b-259e8b07c016">

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#47519 from cxzl25/SPARK-49039.

Authored-by: sychen <[email protected]>
Signed-off-by: Kent Yao <[email protected]>
(cherry picked from commit e3a4430)
Signed-off-by: Kent Yao <[email protected]>
  • Loading branch information
cxzl25 authored and yaooqinn committed Aug 16, 2024
1 parent bcd05a2 commit dcafafe
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -650,6 +650,7 @@ $(document).ready(function () {
}
executorSummaryTableSelector.column(13).visible(dataToShow.showBytesSpilledData);
executorSummaryTableSelector.column(14).visible(dataToShow.showBytesSpilledData);
reselectCheckboxesBasedOnTaskTableState();
});

// Prepare data for speculation metrics
Expand Down

0 comments on commit dcafafe

Please sign in to comment.