[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-12-04 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-986044185 Finally ! -- 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.

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-11-25 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-979260998 Apologies for that but the problem was really difficult to diagnose and fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-11-25 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-979260608 We had some serious issues with memory (fixed yesterday with #19809 finally (two days of investigation!) if you could please rebase one more time @georborodin that woudl be gre

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-11-17 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-971914175 I am afraid we fixed some more tests (Integration) and since this is celery and likely around that, I have a kind requests of rebasing in one more (hopefully last :crossed_fing

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-11-15 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-969371093 Could you please rebase to latest main? I think the test failures have been fixed in the meantime. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-11-14 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-968399781 Sorry - missed that one. I am running tests and let's see. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-09-18 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-922346871 There are some serious problems here - but you are > 70 commits behind main. Can you please rebase @georborodin ? -- This is an automated message from the Apache Git Service.

[GitHub] [airflow] potiuk commented on pull request #18189: Allow using default celery commands for custom Celery executors subclassed from existing

2021-09-14 Thread GitBox
potiuk commented on pull request #18189: URL: https://github.com/apache/airflow/pull/18189#issuecomment-919109988 some doc failures -- 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 c