You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "amoghrajesh (via GitHub)" <gi...@apache.org> on 2025/04/09 08:12:20 UTC

Re: [PR] Rework socket setup for supervisor and subprocess to handle large messages [airflow]

amoghrajesh commented on code in PR #48949:
URL: https://github.com/apache/airflow/pull/48949#discussion_r2034749934


##########
task-sdk/src/airflow/sdk/execution_time/supervisor.py:
##########
@@ -129,31 +125,24 @@
 ]
 
 
-@overload
-def mkpipe() -> tuple[socket, socket]: ...
-
-
-@overload
-def mkpipe(remote_read: Literal[True]) -> tuple[socket, BinaryIO]: ...
+BUFFER_SIZE = 4096

Review Comment:
   Handled in 8ca80d55b4475b4b635fc0b94d9e86f1090f8c06



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org