josh-fell commented on pull request #20180: URL: https://github.com/apache/airflow/pull/20180#issuecomment-990262975
> Nice. I wonder how many of those we will find in other operators.. > > BTW. for the multi-tenancy work we are planning to add testing the DB method calls and we are planning to run all the tests we have for all the hooks and operators through a test harness tthat will be catching basically any DB operations and flagging those that are "not expected". > > I think that might be great opportunity to catch and fix all similar cases too. Would be as easy as to check if there is a db call in any of the `initts` of any of the operators instantiated during tests. Sounds pretty doable. After seeing this I poked around, there are a good number of instances where, at the very least, `get_connection()` is called in an operator's constructor. I can compile a list and create an issue for folks to tackle. WDYT? -- 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]
