jscheffl commented on code in PR #57610: URL: https://github.com/apache/airflow/pull/57610#discussion_r2761028878
########## providers/informatica/src/airflow/providers/informatica/operators/empty.py: ########## Review Comment: I see no value in adding a second EmptyOperator - I rather think this would confuse users if there are two different. To also say that the core/standard EmptyOperator also is optimized and is not routed to an executor but acknowledged in Scheduler directly as noop. It is not required that provider packages have operators. And just for demo purposes to showcase lineage I assume not needed. TLDR: Please remove this. (including RST docs) -- 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]
