Github user tgravescs commented on a diff in the pull request:

    https://github.com/apache/spark/pull/13670#discussion_r71041434
  
    --- Diff: 
core/src/main/resources/org/apache/spark/ui/static/executorspage.js ---
    @@ -0,0 +1,480 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *    http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +
    +function formatStatus(status, type) {
    +    if (type !== 'display') return status;
    +    if (status) {
    +        return "Active"
    +    } else {
    +        return "Dead"
    +    }
    +}
    +
    +jQuery.extend(jQuery.fn.dataTableExt.oSort, {
    +    "title-numeric-pre": function (a) {
    +        var x = a.match(/title="*(-?[0-9\.]+)/)[1];
    +        return parseFloat(x);
    +    },
    +
    +    "title-numeric-asc": function (a, b) {
    +        return ((a < b) ? -1 : ((a > b) ? 1 : 0));
    +    },
    +
    +    "title-numeric-desc": function (a, b) {
    +        return ((a < b) ? 1 : ((a > b) ? -1 : 0));
    +    }
    +});
    +
    +$(document).ajaxStop($.unblockUI);
    +$(document).ajaxStart(function () {
    +    $.blockUI({message: '<h3>Loading Executors Page...</h3>'});
    +});
    +
    +function createTemplateURI(appId) {
    +    var words = document.baseURI.split('/');
    +    var ind = words.indexOf("proxy");
    +    if (ind > 0) {
    +        var baseURI = words.slice(0, ind + 1).join('/') + '/' + appId + 
'/static/executorspage-template.html';
    +        return baseURI;
    +    }
    +    ind = words.indexOf("history");
    +    if(ind > 0) {
    +        var baseURI = words.slice(0, ind).join('/') + 
'/static/executorspage-template.html';
    +        return baseURI;
    +    }
    +    return location.origin + "/static/executorspage-template.html";
    +}
    +
    +function getStandAloneppId(cb) {
    +    var words = document.baseURI.split('/');
    +    var ind = words.indexOf("proxy");
    +    if (ind > 0) {
    +        var appId = words[ind + 1];
    +        cb(appId);
    +        return;
    +    }
    +    ind = words.indexOf("history");
    +    if (ind > 0) {
    +        var appId = words[ind + 1];
    +        cb(appId);
    +        return;
    +    }
    +    //Looks like Web UI is running in standalone mode
    +    //Let's get application-id using REST End Point
    +    $.getJSON(location.origin + "/api/v1/applications", function(response, 
status, jqXHR) {
    +        if (response && response.length > 0) {
    +            var appId = response[0].id
    +            cb(appId);
    +            return;
    +        }
    +    });
    +}
    +
    +function createRESTEndPoint(appId) {
    +    var words = document.baseURI.split('/');
    +    var ind = words.indexOf("proxy");
    +    if (ind > 0) {
    +        var appId = words[ind + 1];
    +        var newBaseURI = words.slice(0, ind + 2).join('/');
    +        return newBaseURI + "/api/v1/applications/" + appId + 
"/allexecutors"
    +    }
    +    ind = words.indexOf("history");
    +    if (ind > 0) {
    +        var appId = words[ind + 1];
    +        var attemptId = words[ind + 2];
    +        var newBaseURI = words.slice(0, ind).join('/');
    +        if (isNaN(attemptId)) {
    +            return newBaseURI + "/api/v1/applications/" + appId + 
"/allexecutors";
    +        } else {
    +            return newBaseURI + "/api/v1/applications/" + appId + "/" + 
attemptId + "/allexecutors";
    +        }
    +    }
    +    return location.origin + "/api/v1/applications/" + appId + 
"/allexecutors";
    +}
    +
    +function formatLogsCells(execLogs, type) {
    +    if (type !== 'display') return Object.keys(execLogs);
    +    if (!execLogs) return;
    +    var result = '';
    +    $.each(execLogs, function (logName, logUrl) {
    +        result += '<div><a href=' + logUrl + '>' + logName + '</a></div>'
    +    });
    +    return result;
    +}
    +
    +// Determine Color Opacity from 0.5-1
    +// activeTasks range from 0 to maxTasks
    +function activeTasksAlpha(activeTasks, maxTasks) {
    +    return maxTasks > 0 ? ((activeTasks / maxTasks) * 0.5 + 0.5) : 1;
    +}
    +
    +function activeTasksStyle(activeTasks, maxTasks) {
    +    return activeTasks > 0 ? ("hsla(240, 100%, 50%, " + 
activeTasksAlpha(activeTasks, maxTasks) + ")") : "";
    +}
    +
    +function activeTasksColor(activeTasks, maxTasks) {
    +    return activeTasks > 0 ? ("hsla(240, 100%, 50%, " + 
activeTasksAlpha(activeTasks, maxTasks) + ")") : "";
    +}
    +
    +// failedTasks range max at 10% failure, alpha max = 1
    +function failedTasksAlpha(failedTasks, totalTasks) {
    +    return totalTasks > 0 ?
    +        (Math.min(10 * failedTasks / totalTasks, 1) * 0.5 + 0.5) : 1;
    +}
    +
    +function failedTasksStyle(failedTasks, totalTasks) {
    +    return failedTasks > 0 ?
    +        ("hsla(0, 100%, 50%, " + failedTasksAlpha(failedTasks, totalTasks) 
+ ")") : "";
    +}
    +
    +function failedTasksColor(failedTasks, totalTasks) {
    +    return failedTasks > 0 ? "white" : "black";
    +}
    +
    +// totalDuration range from 0 to 50% GC time, alpha max = 1
    +function totalDurationAlpha(totalGCTime, totalDuration) {
    +    return totalDuration > 0 ?
    +        (Math.min(totalGCTime / totalDuration + 0.5, 1)) : 1;
    +}
    +
    +function totalDurationStyle(totalGCTime, totalDuration) {
    +    // Red if GC time over GCTimePercent of total time
    +    // When GCTimePercent is edited change ToolTips.TASK_TIME to match
    +    var GCTimePercent = 0.1;
    +    return (totalGCTime > GCTimePercent * totalDuration) ?
    +        ("hsla(0, 100%, 50%, " + totalDurationAlpha(totalGCTime, 
totalDuration) + ")") : "";
    +}
    +
    +function totalDurationColor(totalGCTime, totalDuration) {
    +    // Red if GC time over GCTimePercent of total time
    +    // When GCTimePercent is edited change ToolTips.TASK_TIME to match
    +    var GCTimePercent = 0.1;
    +    return (totalGCTime > GCTimePercent * totalDuration) ? "white" : 
"black";
    +}
    +
    +$(document).ready(function () {
    +    $.extend($.fn.dataTable.defaults, {
    +        stateSave: true,
    +        lengthMenu: [[20, 40, 60, 100, -1], [20, 40, 60, 100, "All"]],
    +        pageLength: 20
    +    });
    +
    +    executorsSummary = $("#active-executors");
    +    searchString = executorsSummary["context"]["location"]["search"];
    --- End diff --
    
    unused, remove


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to