[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread Greg Rahn (Code Review)
Greg Rahn has posted comments on this change. Change subject: Add thousands separator to cardinality in plan diagram .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5587/5/www/query_plan.tmpl File

[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: Add thousands separator to cardinality in plan diagram .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/5587/5/www/query_plan.tmpl File

[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread Henry Robinson (Code Review)
Henry Robinson has submitted this change and it was merged. Change subject: Add thousands separator to cardinality in plan diagram .. Add thousands separator to cardinality in plan diagram Tested manually by visually

[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread Henry Robinson (Code Review)
Henry Robinson has posted comments on this change. Change subject: Add thousands separator to cardinality in plan diagram .. Patch Set 4: Code-Review+2 Carry +2 -- To view, visit http://gerrit.cloudera.org:8080/5587 To

[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Add thousands separator to cardinality in plan diagram .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5587 To

[Impala-ASF-CR] Add thousands separator to cardinality in plan diagram

2017-01-05 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: Add thousands separator to cardinality in plan diagram .. Patch Set 4: Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/147/ --