ferruzzi commented on PR #50677:
URL: https://github.com/apache/airflow/pull/50677#issuecomment-2914568562

   @ashb - When you get a second can you have a look and see if I broke this 
down right?   The part of the code that the user directly interacts with (and 
might be imported into a DAG) now lives in 
`airflow/sdk/definitions/deadline.py` and everything else that I don't want 
them using directly is in `airflow/models/deadline.py`.  
   
   I  think when I asked about it previously, the breakdown was "required db" 
vs "non-db" but this feels like a more natural break for this... is it still 
alright or do i need to put it back to all in the sdk path except the 
_fetch_from_db method?


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