shivaam commented on issue #71577: URL: https://github.com/apache/airflow/issues/71577#issuecomment-5296983174
@ferruzzi Loved the idea of a detector only model and sharing the list with enforcer. I benchmarked a detector prototype looking at 6 tables and it added 1ms per test which is acceptable. I think a narrow detector monitoring a handful of tables look practical. My suggested next steps: - Keep the enforcer as a separate small PR. - Add an optional detector that reports leaks but does not fail tests. I have the code locally for this right now. - Run tests and find out any unit tests that are leaking using the detector as well. - Use your analysis and the report to fix through small PRs -- 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]
