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

Change subject: IMPALA-9632: Implement ds_hll_sketch() and ds_hll_estimate()
......................................................................


Patch Set 1:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/16000/1/tests/query_test/test_datasketches.py
File tests/query_test/test_datasketches.py:

http://gerrit.cloudera.org:8080/#/c/16000/1/tests/query_test/test_datasketches.py@1
PS1, Line 1: import pytest
flake8: F401 'pytest' imported but unused


http://gerrit.cloudera.org:8080/#/c/16000/1/tests/query_test/test_datasketches.py@8
PS1, Line 8: class TestDatasketches(ImpalaTestSuite):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/16000/1/tests/query_test/test_datasketches.py@24
PS1, Line 24:
flake8: W391 blank line at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic602cb6eb2bfbeab37e5e4cba11fbf0ca40b03fe
Gerrit-Change-Number: 16000
Gerrit-PatchSet: 1
Gerrit-Owner: Gabor Kaszab <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Fri, 29 May 2020 14:33:35 +0000
Gerrit-HasComments: Yes

Reply via email to