chamikaramj commented on a change in pull request #11048: [BEAM-9433] Create 
expansion service artifact for common Java IOs.
URL: https://github.com/apache/beam/pull/11048#discussion_r388382088
 
 

 ##########
 File path: sdks/python/apache_beam/io/external/kafka.py
 ##########
 @@ -166,6 +171,8 @@ def __init__(
                                serialization.ByteArraySerializer'.
     :param expansion_service: The address (host:port) of the ExpansionService.
     """
+    if expansion_service is None:
 
 Review comment:
   Prob. we don't need this since we default to default_io_expansion_service() 
below ?

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to