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

Change subject: IMPALA-8847: Ignore add partition events with empty partition 
list
......................................................................


Patch Set 2:

(12 comments)

http://gerrit.cloudera.org:8080/#/c/14049/2/tests/custom_cluster/test_event_processing.py
File tests/custom_cluster/test_event_processing.py:

http://gerrit.cloudera.org:8080/#/c/14049/2/tests/custom_cluster/test_event_processing.py@156
PS2, Line 156:
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/custom_cluster/test_event_processing.py@156
PS2, Line 156:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py
File tests/util/event_processor_utils.py:

http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@22
PS2, Line 22: import logging
flake8: F401 'logging' imported but unused


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@28
PS2, Line 28: class EventProcessorUtils(object):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@31
PS2, Line 31: @
flake8: E301 expected 1 blank line, found 0


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@47
PS2, Line 47: T
flake8: F821 undefined name 'TimeoutError'


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@49
PS2, Line 49: .
flake8: E131 continuation line unaligned for hanging indent


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@53
PS2, Line 53: 
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@53
PS2, Line 53:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@62
PS2, Line 62:
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@62
PS2, Line 62:
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/14049/2/tests/util/event_processor_utils.py@87
PS2, Line 87: @
flake8: E303 too many blank lines (2)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I877ce6233934e7090cd18e497f748bc6479838cb
Gerrit-Change-Number: 14049
Gerrit-PatchSet: 2
Gerrit-Owner: Vihang Karajgaonkar <[email protected]>
Gerrit-Reviewer: Anurag Mantripragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Mon, 12 Aug 2019 20:30:53 +0000
Gerrit-HasComments: Yes

Reply via email to