From dcafafea61b9074067a5a15fca4b6a836b9e6145 Mon Sep 17 00:00:00 2001 From: sychen Date: Fri, 16 Aug 2024 18:08:26 +0800 Subject: [PATCH] [SPARK-49039][UI] Reset checkbox when executor metrics are loaded in 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), ``` image ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? local test image ### Was this patch authored or co-authored using generative AI tooling? No Closes #47519 from cxzl25/SPARK-49039. Authored-by: sychen Signed-off-by: Kent Yao (cherry picked from commit e3a44307fe73c4bff253c734aafbbc019ed8448f) Signed-off-by: Kent Yao --- core/src/main/resources/org/apache/spark/ui/static/stagepage.js | 1 + 1 file changed, 1 insertion(+) diff --git a/core/src/main/resources/org/apache/spark/ui/static/stagepage.js b/core/src/main/resources/org/apache/spark/ui/static/stagepage.js index 50bf959d3aa96..c7513c8268b26 100644 --- a/core/src/main/resources/org/apache/spark/ui/static/stagepage.js +++ b/core/src/main/resources/org/apache/spark/ui/static/stagepage.js @@ -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