Eason09053360 commented on issue #73089:
URL: https://github.com/apache/airflow/issues/73089#issuecomment-5673363179
Tested all 11 of my backports on 3.3.2rc1. All verified fixed.
- #71481 ✅ `airflow dags list` prints a Dag once. With two `dag_version`
rows for the same `dag_id`, 3.3.1 printed 2 lines, rc1 prints 1.
- #71651 ✅ `airflow config lint` now reports `email.email_backend` and
`elasticsearch.log_id_template`. 3.3.1 said "No issues found" for both.
- #71791 ✅ `variables export` → `import` round-trip preserves `{"value":
"a", "description": "b"}`. 3.3.1 collapsed it to `a`.
- #72320 ✅ `airflow tasks test` exports `AIRFLOW_TEST_MODE=True` without
`--env-vars`.
3.3.1 exported nothing — and `--env-vars '{}'` did not help either, since
an empty dict is falsy.
- #72365 ✅ Rebuilding the Dag parser keeps every argument (`dags pause` 5→5,
`dags test`
11→11, `tasks test` 9→9). On 3.3.1 a rebuild dropped all of them to 1
(just `-h`).
- #72565 ✅ `python my_dag.py dags pause` / `unpause` exit 0. 3.3.1 raised
`TypeError: dag_pause() takes 1 positional argument but 2 were given`.
- #72568 ✅ With one component failing to spawn, the shutdown loop completes
and the
surviving sibling is terminated. On 3.3.1 the loop aborted with
`AttributeError: 'SubCommand' object has no attribute 'process'` and the
sibling kept running after standalone exited.
- #72601 ✅ `providers get --full` leaves `ProvidersManager` metadata
untouched when called
in-process. 3.3.1 mutated the cached description.
- #72661 ✅ With `--daemon`, the memray capture spans the actual run (10.3s
over 8 Dags ×
20 tasks). On 3.3.1 `start_time == end_time` — a 0-second capture
containing only
interpreter startup. Note this needs `--daemon` or `--dev` to reproduce;
the plain path does not fork.
- #72744 ✅ `jobs check --allow-multiple --limit 0` passes the guard;
`--limit 1` is still rejected, with a clearer message.
- #72832 ✅ `AirflowInfo.render_text()` returns the real report (3531 / 8403
/ 3527 chars
for json / table / yaml, no ANSI). On 3.3.1 all three returned an empty
string, which is
what was being uploaded. Verified offline rather than uploading to file.io.
Not covered: postgres/mysql backends, the `--dev` reloader path for #72661,
and the ANSI
half of #72832 (the check ran in a non-tty subprocess, where
`color_system="auto"` would
not emit escape codes anyway).
---
Drafted-by: Claude Opus 5 ; reviewed by @Eason09053360 before posting
--
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]
