potiuk commented on PR #71685: URL: https://github.com/apache/airflow/pull/71685#issuecomment-5316577957
Thanks for this, and sorry to close it. This duplicates #71279, which makes the identical production change — the same `_redact_all` guard becomes `not isinstance(item, (dict, tuple, set, list))` — and was opened on 7 August, nine days before this one. #71279 also carries a few things this PR doesn't: a newsfragment, a fail-closed backstop on the final `return` rather than returning `item`, and a test class covering int, float, bool, `None` and bytes, plus the `merge()` round trip that restores the original value. Airflow does allow parallel work on the same problem — "better PR wins" — but the convention is to check for an existing PR before starting. No criticism intended; the diagnosis here was right. Closing in favour of #71279. --- _Drafted by an AI assistant and may contain mistakes — if you think this call is wrong, say so and the PR will be reopened; a human maintainer has the final word._ Drafted-by: Claude Code (Opus 5); reviewed by @potiuk 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]
