potiuk commented on issue #43171:
URL: https://github.com/apache/airflow/issues/43171#issuecomment-2435764433

   > but has to be done very carefully, because if it gives misleading advice 
it will lead users down chasing the wrong rabbit hole. For example this log in 
standard_task_runner.py is most of the time not due to memory running out: "Job 
%s was killed before it finished (likely due to running out of memory)",. I've 
seen our engineers chasing memory issues in vain countless of times because of 
that message. 
   
   I am big fan of "always tell the user what action from their side the error 
implies.". Agree things can be misleading and re the case you mentioned  - I 
cannot find it now but I think in case of such complicated and 
multi-possible-root-cause we should explain what's going on and link to a FAQ 
page on Airflow explaining possible reasons. This way when you have the error, 
and we find other reasons and more detailed explanations what could be wrong 
and how to remediate it - we can always update the docs and add more 
information that will be useful for many past versions of airflow that people 
will have.
   
   > (yes we should have filed a PR 😄)
   
   Absolutely :)
   


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

Reply via email to