stevenschaerer commented on PR #38689: URL: https://github.com/apache/airflow/pull/38689#issuecomment-2034640475
> I don’t think we can just remove TaskStateTrigger, but let’s deprecate it for future removal. > > It would also be a good idea to implement an version of `_get_count` to use the async SQLAlchemy API if available, and fallback to the `sync_to_async` version otherwise. Most people likely won’t have the async component (`pip install sqlalchemy[asyncio]`) but we should start to slowly add support to it in triggerers. I have deprecated `TaskStateTrigger`. Following the other comments I'll leave using async SQLAlchemy for a separate PR. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
