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/02 09:18:48 UTC

[GitHub] [airflow] turbaszek commented on pull request #12660: Add upgrade check rule for unrecognized arguments to Operators

turbaszek commented on pull request #12660:
URL: https://github.com/apache/airflow/pull/12660#issuecomment-737101102


   > > Weird, passed for me locally too
   > 
   > As discussed here: https://apache-airflow.slack.com/archives/C0146STM600/p1606855518349800 - I belive this might side-effect from another test using the same "test" dag_id. We have a few of those and there are a number of tests that are likely leaving some remnants in the DB. My guess would be, this is the case and cleaning in setUp/tearDown might solve the issue.
   
   Sounds familiar. I usually prefer to use tests names as dag/task ids as this make sure that we are not affected by side effects. But cleanup would be also nice.


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