bolkedebruin commented on issue #6627: [AIRFLOW-5931] Use os.fork when 
appropriate to speed up task execution.
URL: https://github.com/apache/airflow/pull/6627#issuecomment-557300526
 
 
   Aside from from this performance enhancement I think a much bigger gain can 
be gotten from removing the raw task excecution. Ie. the process is now as 
follows Executor -> Task -> Rawtask. Which doesn't make sense. It should just 
be Executor -> Task. Ideally the task would then signal to the executor what 
its state is rather than setting its own state, which is either undeterministic 
(in case of a crash of the task) and insecure (it requires airflow db access by 
the task which is available to all operators)

----------------------------------------------------------------
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