Sai Hemanth Gantasala has posted comments on this change. ( http://gerrit.cloudera.org:8080/20986 )
Change subject: IMPALA-12782: Show info of the event processing in /events webUI ...................................................................... Patch Set 2: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/20986/2/www/events.tmpl File www/events.tmpl: http://gerrit.cloudera.org:8080/#/c/20986/2/www/events.tmpl@53 PS2, Line 53: Start processing the current batch at {{progress-info.start_time}} ({{progress-info.elapsed_time}} ago).</br> Nit: does 'Started processing' sound good? http://gerrit.cloudera.org:8080/#/c/20986/2/www/events.tmpl@54 PS2, Line 54: Start processing the current event at {{progress-info.start_time_of_event}} ({{progress-info.elapsed_time_current_event}} ago). Nit: Should we also show the current event ID here? -- To view, visit http://gerrit.cloudera.org:8080/20986 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2e7d4952c7fd04ae89b6751204499bf9dd99f57c Gerrit-Change-Number: 20986 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Comment-Date: Tue, 13 Feb 2024 23:38:12 +0000 Gerrit-HasComments: Yes
