Riza Suminto has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22950 )
Change subject: IMPALA-13986: Update JS scripts to align with linting rules ...................................................................... IMPALA-13986: Update JS scripts to align with linting rules For large JS scripts, code-analysis and linting rules were added to standardize changes for the webUI scripts in IMPALA-13473. This change corrects the scripts to align with these linting and code-analysis rules. Change-Id: I5c8851c3d2f4e95b7eebd61f379361818afca7ea Reviewed-on: http://gerrit.cloudera.org:8080/22950 Tested-by: Impala Public Jenkins <[email protected]> Reviewed-by: Riza Suminto <[email protected]> --- M tests/webui/js_tests/queries/profileParseWorker.test.js M tests/webui/js_tests/query_timeline/chart_commons.test.js M tests/webui/js_tests/query_timeline/fragment_diagram.test.js M tests/webui/js_tests/query_timeline/fragment_metrics_diagram.test.js M tests/webui/js_tests/query_timeline/host_utilization_diagram.test.js M www/scripts/common_util.js M www/scripts/queries/profileParseWorker.js M www/scripts/query_timeline/chart_commons.js M www/scripts/query_timeline/fragment_diagram.js M www/scripts/query_timeline/fragment_metrics_diagram.js M www/scripts/query_timeline/global_dom.js M www/scripts/query_timeline/host_utilization_diagram.js 12 files changed, 371 insertions(+), 360 deletions(-) Approvals: Impala Public Jenkins: Verified Riza Suminto: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22950 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5c8851c3d2f4e95b7eebd61f379361818afca7ea Gerrit-Change-Number: 22950 Gerrit-PatchSet: 11 Gerrit-Owner: Surya Hebbar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Surya Hebbar <[email protected]>
