Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13259 )

Change subject: Improve output for generate-schema-statements.py
......................................................................


Patch Set 3: Code-Review+2

(1 comment)

Looks good

http://gerrit.cloudera.org:8080/#/c/13259/1/testdata/bin/generate-schema-statements.py
File testdata/bin/generate-schema-statements.py:

http://gerrit.cloudera.org:8080/#/c/13259/1/testdata/bin/generate-schema-statements.py@861
PS1, Line 861:       load_table_
> How about deferring this change to later? Want to make a quick incremental
Yes, this is not a blocker.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Idf53fde209bb7ee319a4c13667f3fa2783507072
Gerrit-Change-Number: 13259
Gerrit-PatchSet: 3
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Tue, 14 May 2019 00:22:06 +0000
Gerrit-HasComments: Yes

Reply via email to