potiuk commented on PR #28968: URL: https://github.com/apache/airflow/pull/28968#issuecomment-1383985093
I think this is more than just a small change - the unit tests have to be updated (and added) to cover all cases and I am not sure yet - without looking- what exactly was wrong and how it **SHOULD** be fixed, starting from unit tests and understanding them and adding more cases is the way to start working on it. @msumit maybe you can also comment on that? I think you remember best the intentions here and what will be the best way forward. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
