Repository navigation
Add Andrushika's June to October contribution report for apache-airflow - #28
Open
Andrushika wants to merge 2 commits into
Open
Andrushika wants to merge 2 commits into
Andrushika wants to merge 2 commits into
Conversation
Author
|
cc @jason810496 @Lee-W :) |
|
好耶，太強ㄌ |
|
好強！但是不是漏放 GitHub 跟 Slack 連結了? 😄 |
Author
|
對耶...等等來補 感謝！ |
chia7712
approved these changes
Oct 19, 2026
chia7712
reviewed
Oct 19, 2026
| - `breeze start-airflow` 每次啟動前會 hash UI 目錄，判斷要不要重新 build assets | ||
| - 但它連第三方套件庫 `.pnpm-store` 也一起掃，所以就算什麼都沒改，每次都要多等 16–30 秒 | ||
| - 改成掃描時直接跳過 `node_modules` 和 `.pnpm-store`，檢查時間降到 1 秒內 | ||
| - 以 3 票之姿~~灌票~~當選 **2026 年 9 月 PR of the Month** |
jason810496
approved these changes
Oct 20, 2026
Comment on lines
+170
to
+171
| - flaky test，airflowctl 的 integration test 常常隨機 timeout | ||
| - AI 追下去，發現是 Celery 預設平行起很多個 worker，在 4 vCPU 的 runner 上一次 fork 十幾個 task process 造成效能爆炸 |
Comment on lines
+189
to
+190
| - 因為算是莫名被罵（？）所以難過了一下，後來 Ash 也自己發現那真的是需要解的問題，我也開了 follow-up PR [#73862](https://github.com/apache/airflow/pull/73862) 修掉 | ||
| - 原本在 Airflow Slack 上面拜託 maintainers 幫忙看一下 PR 通常都會沒下文，從這次 review 之後 Ash 好像都會點進來幫我看兩眼 |
|
|
||
| #### UI | ||
|
|
||
| 這邊做了一系列關於 task log 視窗的 UX 改進，起因是 lazy loading 的 log rows 在被滑鼠 select 的時候行為會怪怪的（無法被選中、反白的區塊跳來跳去、複製貼上功能不正常等等） |
Member
There was a problem hiding this comment.
btw task log 的後端也很直得修 如果追過 e2e path 應該會想要整個重寫
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Apache Airflow Contribution Report by Andrushika
Summary
Merged PR 分類（依區域）