Surya Hebbar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22496 )

Change subject: IMPALA-13074: WRITE TO HDFS node is omitted from Web UI graphic 
plan
......................................................................


Patch Set 8:

(3 comments)

Just wanted an update example, if necessary on L1205

Sorry for excessively adding myself as a new reviewer here.

http://gerrit.cloudera.org:8080/#/c/22496/8//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22496/8//COMMIT_MSG@7
PS8, Line 7: WRITE TO HDFS node is omitted from Web UI graphic plan
Please use something like...
Add WRITE TO HDFS nodes to webUI's graphical plan


http://gerrit.cloudera.org:8080/#/c/22496/8//COMMIT_MSG@14
PS8, Line 14: * tested manually on WebUI with CREATE TABLE .. AS SELECT 
statements
            :    and UPDATE on Iceberg tables
Breakdown to more points if necessary


http://gerrit.cloudera.org:8080/#/c/22496/8/be/src/service/impala-http-handler.cc
File be/src/service/impala-http-handler.cc:

http://gerrit.cloudera.org:8080/#/c/22496/8/be/src/service/impala-http-handler.cc@1181
PS8, Line 1181: // Helper method which converts a list of plan fragments into a 
single JSON document, with
              : // the following schema:
              : // "plan_nodes": [
              : //     {
              : //       "label": "12:AGGREGATE",
              : //       "label_detail": "FINALIZE",
              : //       "output_card": 23456,
              : //       "num_instances": 34,
              : //       "max_time": "1m23s",
              : //       "avg_time": "1.3ms",
              : //       "children": [
              : //           {
              : //             "label": "11:EXCHANGE",
              : //             "label_detail": "UNPARTITIONED",
              : //             "children": []
              : //           }
              : //       ]
              : //     },
              : //     {
              : //       "label": "07:AGGREGATE",
              : //       "label_detail": "",
              : //       "children": [],
              : //       "data_stream_target": "11:EXCHANGE"
              : //     }
              : // ]
Please update the example here, unless redundant



--
To view, visit http://gerrit.cloudera.org:8080/22496
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib2bd442f6499efde7406d87c2b1fd1b46a45381b
Gerrit-Change-Number: 22496
Gerrit-PatchSet: 8
Gerrit-Owner: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Daniel Vanko <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Tue, 11 Mar 2025 23:09:55 +0000
Gerrit-HasComments: Yes

Reply via email to