Sailesh Mukil has posted comments on this change. ( http://gerrit.cloudera.org:8080/10523 )
Change subject: IMPALA-7082: Show human readable size in query backend page ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/10523/2/www/common-header.tmpl File www/common-header.tmpl: http://gerrit.cloudera.org:8080/#/c/10523/2/www/common-header.tmpl@54 PS2, Line 54: <script> Is this is the right file to put these utility functions in? Shouldn't we create a "util" file instead? -- To view, visit http://gerrit.cloudera.org:8080/10523 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I04afb4091bb8b6bc9dedfeceaf9284a8c65b16a1 Gerrit-Change-Number: 10523 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Comment-Date: Tue, 29 May 2018 19:28:18 +0000 Gerrit-HasComments: Yes
