Skip to content

Commit

Permalink
[SPARK-20813][WEB UI] Fixed Web UI executor page tab search by status…
Browse files Browse the repository at this point in the history
… not working

## What changes were proposed in this pull request?
On status column of the table, I removed the condition  that forced only the display value to take on values Active, Blacklisted and Dead.

Before the removal, values used for sort and filter for that particular column was True and False.
## How was this patch tested?

Tested with Active, Blacklisted and Dead present as current status.

Author: John Lee <jlee2@yahoo-inc.com>

Closes #18036 from yoonlee95/SPARK-20813.
  • Loading branch information
John Lee authored and srowen committed May 22, 2017
1 parent f1ffc6e commit aea73be
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ function getThreadDumpEnabled() {
}

function formatStatus(status, type) {
if (type !== 'display') return status;
if (status) {
return "Active"
} else {
Expand Down Expand Up @@ -417,7 +416,6 @@ $(document).ready(function () {
},
{data: 'hostPort'},
{data: 'isActive', render: function (data, type, row) {
if (type !== 'display') return data;
if (row.isBlacklisted) return "Blacklisted";
else return formatStatus (data, type);
}
Expand Down

0 comments on commit aea73be

Please sign in to comment.