Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22626 )

Change subject: IMPALA-13864: Implement ImpylaHS2ResultSet.exec_summary
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22626/5/shell/impala_client.py
File shell/impala_client.py:

http://gerrit.cloudera.org:8080/#/c/22626/5/shell/impala_client.py@113
PS5, Line 113: def build_exec_summary_table(summary, idx, indent_level, 
new_indent_level, output,
             :                              is_prettyprint=True, 
separate_prefix_column=False):
Got a feedback from Csaba that this ideally should be in its own python file.
I'll check if I can do that.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7d88f78e58eeda29ce21e7828884c7a129d7efe6
Gerrit-Change-Number: 22626
Gerrit-PatchSet: 5
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Thu, 20 Mar 2025 18:10:04 +0000
Gerrit-HasComments: Yes

Reply via email to