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

Change subject: IMPALA-5351: Support storing column comment of kudu table
......................................................................


Patch Set 5:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/12977/5/tests/metadata/test_ddl.py
File tests/metadata/test_ddl.py:

http://gerrit.cloudera.org:8080/#/c/12977/5/tests/metadata/test_ddl.py@733
PS5, Line 733: )
flake8: E501 line too long (92 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12977/5/tests/metadata/test_ddl.py@753
PS5, Line 753: l
flake8: E501 line too long (94 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12977/5/tests/metadata/test_ddl.py@761
PS5, Line 761: b
flake8: E501 line too long (95 > 90 characters)


http://gerrit.cloudera.org:8080/#/c/12977/5/tests/query_test/test_kudu.py
File tests/query_test/test_kudu.py:

http://gerrit.cloudera.org:8080/#/c/12977/5/tests/query_test/test_kudu.py@884
PS5, Line 884: b
flake8: E501 line too long (96 > 90 characters)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb3b37eed364f12bdb3c1d7ef5be128f1475936c
Gerrit-Change-Number: 12977
Gerrit-PatchSet: 5
Gerrit-Owner: helifu <hzhel...@corp.netease.com>
Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Thomas Marshall <tmarsh...@cloudera.com>
Gerrit-Reviewer: helifu <hzhel...@corp.netease.com>
Gerrit-Comment-Date: Thu, 25 Apr 2019 07:21:41 +0000
Gerrit-HasComments: Yes

Reply via email to