dstandish commented on PR #33242: URL: https://github.com/apache/airflow/pull/33242#issuecomment-1894074083
This is a good optimization. We ran into this exact issue and came up with the same optimization independently. But we were thinking to just add an index since there are a couple other instances of if very similar query that I don't think can do the same thing. But even if we do add the index I don't think it hurts to have both. My one suggestion would be to move the query generation to a private function somewhere so that things are more readable at the call site. -- 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]
