[GitHub] [airflow] ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of fresh DAGRun

2020-02-25 Thread GitBox
ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of 
fresh DAGRun
URL: https://github.com/apache/airflow/pull/7510#issuecomment-591112184
 
 
   @Fokko `run.refresh_from_db()` just loads the id and state columns. The case 
you are talking about is still handled correctly (by the sligtly badly named 
`verify_integrity` function.


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


With regards,
Apache Git Services


[GitHub] [airflow] ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of fresh DAGRun

2020-02-24 Thread GitBox
ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of 
fresh DAGRun
URL: https://github.com/apache/airflow/pull/7510#issuecomment-590469732
 
 
   Ah yeah,  I have a similar notepad.
   
   And which helper fn for instance in this case?


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


With regards,
Apache Git Services


[GitHub] [airflow] ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of fresh DAGRun

2020-02-24 Thread GitBox
ashb commented on issue #7510: [AIRFLOW-6887][WIP] Do not check the state of 
fresh DAGRun
URL: https://github.com/apache/airflow/pull/7510#issuecomment-590453194
 
 
   What script are you using to get the numbers btw @mik-laj?


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


With regards,
Apache Git Services