yuqian90 commented on issue #7723: [AIRFLOW-7063] Fix dag.clear() slowness 
caused by count
URL: https://github.com/apache/airflow/pull/7723#issuecomment-599082749
 
 
   > > Thanks for the review. The test I added does not play well with sqlite. 
I'll fix soon
   > 
   > The error is really interesting 🤔
   
   [The 
test](https://travis-ci.org/github/apache/airflow/jobs/662402191?utm_medium=notification&utm_source=github_status)
 failed in sqlite because of a hard limit on the level of nesting in the query. 
The test is specifically made to cause many levels of nesting in the generated 
sql to reproduce the sqlalchemy slowness. 
[Stackoverflow](https://stackoverflow.com/questions/32300191/django-db-utils-operationalerror-parser-stack-overflow
   ) suggests there are compiler options to increase this hard limit in sqlite, 
but I doubt anyone uses sqlite in production with a dag this complicated.  I 
think in this case skipping the test in sqlite backend is the right thing to do 
so I did that with `pytest.mark.backend`.
   

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to