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 2022/01/29 03:41:32 UTC
[GitHub] [airflow] chenglongyan opened a new pull request #21204: Fix backfill job crash when use DebugScheduler
chenglongyan opened a new pull request #21204:
URL: https://github.com/apache/airflow/pull/21204
related: #13712
--
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.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] chenglongyan closed pull request #21204: Fix backfill job crash when use DebugExecutor
Posted by GitBox <gi...@apache.org>.
chenglongyan closed pull request #21204:
URL: https://github.com/apache/airflow/pull/21204
--
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.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #21204: Fix backfill job crash when use DebugScheduler
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #21204:
URL: https://github.com/apache/airflow/pull/21204#issuecomment-1024925623
> I will create a new pr about those tests
You should add them here. If for anything - for cherry-picking, this is extremely important to have tests together with the code in a single PR not separately.
--
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.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #21204: Fix backfill job crash when use DebugScheduler
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #21204:
URL: https://github.com/apache/airflow/pull/21204#issuecomment-1024878982
That doesn't sound right to remove the tests. We should fix them instead I think - unless there is a good reason we do not want to ?
--
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.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] chenglongyan commented on pull request #21204: Fix backfill job crash when use DebugScheduler
Posted by GitBox <gi...@apache.org>.
chenglongyan commented on pull request #21204:
URL: https://github.com/apache/airflow/pull/21204#issuecomment-1024901664
> That doesn't sound right to remove the tests. We should fix them instead I think - unless there is a good reason we do not want to ?
I will create a new pr about those tests
--
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.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org