YoannAbriel opened a new pull request, #63797:
URL: https://github.com/apache/airflow/pull/63797

   Migration 0101 paginates over `serialized_dag` using `ORDER BY dag_id`, but 
`dag_id` has no index. On MySQL with large tables (100K+ rows), the inner 
subquery's filesort exceeds `sort_buffer_size` and fails with 
`OperationalError: (1038, 'Out of sort memory')`.
   
   The existing subquery optimization (selecting only `id, dag_id` in the inner 
query) isn't enough — without an index, MySQL still has to sort the full result 
set in memory.
   
   Fix: create a temporary index on `serialized_dag(dag_id)` before the 
pagination loop and drop it after, in both upgrade and downgrade paths. The 
index lets MySQL use an index scan instead of a filesort, eliminating the sort 
buffer dependency entirely.
   
   Closes: #63786
   
   <!-- SPDX-License-Identifier: Apache-2.0
         https://www.apache.org/licenses/LICENSE-2.0 -->
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4, claude-opus-4-6)
   
   Generated-by: Claude Code (Opus 4, claude-opus-4-6) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   ---
   
   * Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information. Note: commit author/co-author name and email in commits 
become permanently public when merged.
   * For fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   * When adding dependency, check compliance with the [ASF 3rd Party License 
Policy](https://www.apache.org/legal/resolved.html#category-x).
   * For significant user-facing changes create newsfragment: 
`{pr_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
 You can add this file in a follow-up commit after the PR is created so you 
know the PR number.
   


-- 
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]

Reply via email to