[BEAM-9422] Missing comma in worker_pool_main causing syntax error whenever worker is started

• Type: Bug
• Status: Resolved
• Priority: P1
• Resolution: Fixed
• Fix Version/s: Not applicable
• Component/s: sdk-py-harness
• Labels: newbie
• Flags: Important
• Assignee: Kyle Weaver
• Reporter: Frederik B
• Created: 02/Mar/20 13:55
• Updated: 16/May/20 13:28
• Resolved: 15/May/20 23:33

Description

Missing comma in worker_pool_main causing syntax error when python worker is started, resulting in: 

"Starting worker with command ['python', '-c', 'from apache_beam.runners.worker.sdk_worker import SdkHarness; SdkHarness("localhost:33083",worker_id="1-1",state_cache_size=0data_buffer_time_limit_ms=-1).run()']
File "<string>", line 1
from apache_beam.runners.worker.sdk_worker import SdkHarness; SdkHarness("localhost:33083",worker_id="1-1",state_cache_size=0data_buffer_time_limit_ms=-1).run()
^
SyntaxError: invalid syntax"

 

Should be trivial to fix; add comma to line 116 in worker_pool_main.py.

 

Note. I am not a Beam contributer, I am merely a user. I would appreciate if anyone could fix this. Thank you!

Issue Links

• is duplicated by: BEAM-9503 SyntaxError in process worker startup
• links to: https://github.com/apache/beam/pull/11123