Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13400 )

Change subject: (WIP) IMPALA-8507: Support DROP TABLE statement with Kudu/HMS 
integration
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13400/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
File fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java:

http://gerrit.cloudera.org:8080/#/c/13400/1/fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java@1616
PS1, Line 1616:               tableName.getDb(), tableName.getTbl(), true, 
params.if_exists, params.purge);
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/13400/1/tests/custom_cluster/test_kudu.py
File tests/custom_cluster/test_kudu.py:

http://gerrit.cloudera.org:8080/#/c/13400/1/tests/custom_cluster/test_kudu.py@169
PS1, Line 169:
flake8: W391 blank line at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6d3b93957cc66009ad7a67fc513be2068f156abc
Gerrit-Change-Number: 13400
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Comment-Date: Wed, 22 May 2019 06:30:27 +0000
Gerrit-HasComments: Yes

Reply via email to