avzamelek opened a new pull request, #70456:
URL: https://github.com/apache/airflow/pull/70456
## What
Remove the redundant `shell=True` argument from
`BeamAsyncHook.run_beam_command_async`.
`asyncio.create_subprocess_shell()` already starts the subprocess through a
shell. Passing `shell=True` again is forwarded to `subprocess.Popen`, where
it
conflicts with the value supplied internally and raises a `TypeError`.
## Test coverage
Add a focused async unit test that verifies the subprocess launcher is called
without the duplicate `shell` keyword.
## Testing
- `git diff --check`
- Breeze test setup was attempted.
- The targeted test could not be run locally because Docker is not installed;
Breeze requires Docker to create its CI environment.
## Scope
This is a bug fix for the Apache Beam provider's async execution path. It
does
not change command construction or shell-escaping behavior.
## Development assistance
Claude Code Fable 5 was used as a development assistant during investigation
and
implementation. The final diff was reviewed by human before submission.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
