potiuk commented on PR #59876: URL: https://github.com/apache/airflow/pull/59876#issuecomment-3698939721
BTW. To be perfectly honest, In this case I think performance is **not** as important (at least until we will start doing the communication very frequently - for example following @dabla optimisation / async task reporting back status of individual async coroutines back to scheduler). The comms overhead for inter-process communication and serialization of data involved (every such call needs to serialize data sent across the wire - even if shared memory is used to communicate between processes) is already likely order of magnitude slower than single method call in Python, so this should not be too much of a concern. In this case I think we should optimise for readability, I also do not like the extra `()` needed in the original proposal - that's why `supervisor_send(data)` is probably best approach. -- 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]
