diff options
author | Alex Bozarth <ajbozart@us.ibm.com> | 2016-12-13 21:37:46 +0000 |
---|---|---|
committer | Sean Owen <sowen@cloudera.com> | 2016-12-13 21:37:46 +0000 |
commit | aebf44e50b6b04b848829adbbe08b0f74f31eb32 (patch) | |
tree | 8e225728faff8999b29781eb2952452c3d3761de /core | |
parent | 9e8a9d7c6a847bc5e77f9a1004029ec27616da9d (diff) | |
download | spark-aebf44e50b6b04b848829adbbe08b0f74f31eb32.tar.gz spark-aebf44e50b6b04b848829adbbe08b0f74f31eb32.tar.bz2 spark-aebf44e50b6b04b848829adbbe08b0f74f31eb32.zip |
[SPARK-18816][WEB UI] Executors Logs column only ran visibility check on initial table load
## What changes were proposed in this pull request?
When I added a visibility check for the logs column on the executors page in #14382 the method I used only ran the check on the initial DataTable creation and not subsequent page loads. I moved the check out of the table definition and instead it runs on each page load. The jQuery DataTable functionality used is the same.
## How was this patch tested?
Tested Manually
No visible UI changes to screenshot.
Author: Alex Bozarth <ajbozart@us.ibm.com>
Closes #16256 from ajbozarth/spark18816.
Diffstat (limited to 'core')
-rw-r--r-- | core/src/main/resources/org/apache/spark/ui/static/executorspage.js | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/core/src/main/resources/org/apache/spark/ui/static/executorspage.js b/core/src/main/resources/org/apache/spark/ui/static/executorspage.js index 1df67337ea..fe5db6aa26 100644 --- a/core/src/main/resources/org/apache/spark/ui/static/executorspage.js +++ b/core/src/main/resources/org/apache/spark/ui/static/executorspage.js @@ -412,10 +412,6 @@ $(document).ready(function () { ], "columnDefs": [ { - "targets": [ 15 ], - "visible": logsExist(response) - }, - { "targets": [ 16 ], "visible": getThreadDumpEnabled() } @@ -423,7 +419,8 @@ $(document).ready(function () { "order": [[0, "asc"]] }; - $(selector).DataTable(conf); + var dt = $(selector).DataTable(conf); + dt.column(15).visible(logsExist(response)); $('#active-executors [data-toggle="tooltip"]').tooltip(); var sumSelector = "#summary-execs-table"; |