bipinprasad commented on a change in pull request #3420:
URL: https://github.com/apache/storm/pull/3420#discussion_r739335344
##########
File path:
storm-webapp/src/main/java/org/apache/storm/daemon/ui/WEB-INF/topology.html
##########
@@ -331,22 +331,27 @@ <h2 id="topology-resources-header">Topology resources</h2>
info: false,
searching: false,
columnDefs: [
- {type: "num", targets: [1, 2, 3, 4, 5]},
+ {type: "num", targets: [1, 2, 4, 5], render:
$.fn.dataTable.render.number( ',', '.', 0)},
+ {type: "num", targets: [3]},
{type: "time-str", targets: [0]}
]
});
spoutStats.append(Mustache.render($(template).filter("#spout-stats-template").html(),response));
+ // id, executors, tasks, emitted, transferred, latency, acked,
failed, error host, error port, last error, error time
dtAutoPage("#spout-stats-table", {
columnDefs: [
- {type: "num", targets: 'table-num'}
+ {type: "num", targets: [1, 2, 5]},
+ {type: "num", targets: [3, 4, 6, 7], render:
$.fn.dataTable.render.number( ',', '.', 0)}
]
});
boltStats.append(Mustache.render($(template).filter("#bolt-stats-template").html(),response));
+ // id, executors, tasks, emitted, transferred, capacity, exec
latency, executed, process latency, acked, failed, error host, error port, last
error, error time
Review comment:
The columns match without the conditional columns. Open to what the
comment line should say. Column 10 should not be formatted with "," as a large
number. Will change this.
The columns are ordered thusly:
Id
Executors
Tasks
{{#schedulerDisplayResource}} (conditional columns below)
Req On-heap Mem (MB)
Req Off-heap Mem (MB)
Req CPU
Req Generic
{{/schedulerDisplayResource}} (conditional columns above)
Emitted
Transferred
Capacity
Execute latency (ms)
Executed
Process latency (ms)
Acked
Failed
Error Host
Error Port
Last error
Error Time
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
