Repository: incubator-impala Updated Branches: refs/heads/master 4ce5213d1 -> c4ee03a7e
Add thousands separator to cardinality in plan diagram Tested manually by visually validating the cardinality numbers have thousands separators. Change-Id: I23bfbd8d235c7f1ab7a22d640e557d8ae9a00947 Reviewed-on: http://gerrit.cloudera.org:8080/5587 Tested-by: Impala Public Jenkins Reviewed-by: Henry Robinson <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/ede22bad Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/ede22bad Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/ede22bad Branch: refs/heads/master Commit: ede22bad980447dcb8967295911f4d8026a57b85 Parents: 4ce5213 Author: Greg Rahn <[email protected]> Authored: Thu Dec 29 16:51:47 2016 -0800 Committer: Henry Robinson <[email protected]> Committed: Thu Jan 5 21:38:23 2017 +0000 ---------------------------------------------------------------------- www/query_plan.tmpl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/ede22bad/www/query_plan.tmpl ---------------------------------------------------------------------- diff --git a/www/query_plan.tmpl b/www/query_plan.tmpl index 421c2df..0487529 100644 --- a/www/query_plan.tmpl +++ b/www/query_plan.tmpl @@ -95,7 +95,7 @@ function build(node, parent, edges, states, colour_idx, max_node_time) { "max_time_val": node["max_time_val"], "style": "fill: " + colours[colour_idx]}); if (parent != null) { - var label_val = "" + node["output_card"]; + var label_val = "" + node["output_card"].toLocaleString(); edges.push({ start: node["label"], end: parent, style: { label: label_val }}); } @@ -104,7 +104,7 @@ function build(node, parent, edges, states, colour_idx, max_node_time) { if (node["data_stream_target"]) { edges.push({ "start": node["label"], "end": node["data_stream_target"], - "style": { label: "" + node["output_card"], + "style": { label: "" + node["output_card"].toLocaleString(), style: "stroke: #f66; stroke-dasharray: 5, 5;"}}); } max_node_time = Math.max(node["max_time_val"], max_node_time)
