yuseok89 commented on code in PR #69675: URL: https://github.com/apache/airflow/pull/69675#discussion_r3558959801
########## providers/common/sql/src/airflow/providers/common/sql/operators/sql.py: ########## Review Comment: The `_get_match` negative-`tolerance` fix itself looks correct to me. That said, this change seems unrelated to the PR's topic (quoting the JSON path for asset-event `extra` filtering), and it isn't mentioned anywhere in the title, description, newsfragment, or the regression list. Is it included intentionally, or did it slip in from another branch? It's a valid fix on its own, so splitting it into its own PR would probably keep things cleaner. -- 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]
