dylanbstorey commented on code in PR #30175: URL: https://github.com/apache/airflow/pull/30175#discussion_r1167626974
########## airflow/providers/apache/kafka/sensors/kafka.py: ########## Review Comment: I've had these patterns around since ~1.10.2 and chose not to use them even in limited settings until deferrable existed because they were catastrophically bad. Kafka and Airflow already have plenty of footguns available - I'd prefer not to hand out more w/o need. We can revisit this conversation as KIP-500 reaches more widespread adoption and I get some time to test some how Airflow 2.0 does with this pattern though. Also in over a year in operation not a single user has asked for this feature so I'm happy to wait until we see a demand. -- 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]
