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

Change subject: IMPALA-10509: Add tool to visualize Impala query plan from 
profile
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/17077/1/bin/diagnostics/experimental/plan-graph.py
File bin/diagnostics/experimental/plan-graph.py:

http://gerrit.cloudera.org:8080/#/c/17077/1/bin/diagnostics/experimental/plan-graph.py@50
PS1, Line 50:
> flake8: E305 expected 2 blank lines after class or function definition, fou
Done


http://gerrit.cloudera.org:8080/#/c/17077/1/bin/diagnostics/experimental/plan-graph.py@630
PS1, Line 630:
> flake8: E305 expected 2 blank lines after class or function definition, fou
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic62e3c2b061dd2b84589336fb9e68fa1eccf76f2
Gerrit-Change-Number: 17077
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: David Rorke <dro...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Comment-Date: Wed, 17 Feb 2021 21:01:14 +0000
Gerrit-HasComments: Yes

Reply via email to