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


Reply via email to