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

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/12/03 13:34:44 UTC

[GitHub] [airflow] potiuk commented on issue #12587: dagrun object doesn't exist in the TriggerDagRunOperator

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


   > <img alt="Screen Shot 2020-12-02 at 7 17 36 PM" width="437" src="https://user-images.githubusercontent.com/2644098/100959466-2a753400-34d3-11eb-958a-feeb5bae4b9f.png">
   > 
   > crazy that it shows up in my IDE but not in our static checks.
   
   It is likely, because we have  "dagrun.py" still excluded from pylint in pylint_tdo.txt
   
   ```
   ./airflow/models/dag.py
   ./airflow/models/dagrun.py
   ./airflow/www/utils.py
   ./airflow/configuration.py
   ./airflow/models/taskinstance.py
   ```
   
   Until we remove all the  unneeded 'cyclic dependencies' (note! cyclic dependencies, not cyclic imports!) whcih I am boasting about for quite some time, we cannot remove those from the list.
   
   The root cause of it is the way how inter-twined those "core" classes are and how they depend on each other in all directions. I hope I will give it a shot after 2.0 again.
   
   


----------------------------------------------------------------
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:
users@infra.apache.org