Codingaditya17 commented on PR #67550: URL: https://github.com/apache/airflow/pull/67550#issuecomment-4547929591
Good point, you’re right. The current change fixes `total_entries`, but pagination is still applied to the joined rows before grouping, so `limit` can still behave incorrectly when one import-error file maps to multiple DAG rows. I’ll update the PR so pagination is applied to distinct `ParseImportError.id` values first, then fetch all joined DAG rows for those paginated IDs before grouping. I’ll also extend the regression test to cover `limit=1` for a file mapped to multiple DAGs. -- 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]
