nizhikov commented on a change in pull request #8695: URL: https://github.com/apache/kafka/pull/8695#discussion_r431677154
########## File path: tests/kafkatest/services/kafka/kafka.py ########## @@ -52,6 +51,7 @@ def advertised_listener(self, node): def listener_security_protocol(self): return "%s:%s" % (self.name, self.security_protocol) + Review comment: This change not necessary. But PyCharm and IDEA print warning without it: "PEP 8: E302 expected 2 blank lines, found 1" I will revert it. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org