jstern commented on issue #7133: [AIRFLOW-6535] add exception to fail without 
retry
URL: https://github.com/apache/airflow/pull/7133#issuecomment-605339729
 
 
   Thanks @KevinYang21 - I definitely would like to see it get merged, but I'm 
stuck on a strange problem with the docs build throwing warnings due to the 
problem I described in 
https://github.com/apache/airflow/pull/7133#discussion_r389372194 above. 
   
   The short version of the issue is that my PR adds `airflow.exceptions` to 
the API docs, but this causes warnings in the doc build because 
`AirflowException` is already included (incorrectly) in the yandex provider API 
docs. I have not yet had the time or inspiration to figure out why that bug 
exists in master or how to correct it.
   
   Another option that would let the build succeed would be to revert the 
change that adds the exceptions module to the api docs, but since I believe the 
docs in master are wrong I have been reluctant to take this step.
   
   Perhaps I should file a JIRA ticket for the docs bug in master and hope 
someone who understands Sphinx and autoapi better than I do knows how to fix it?

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