shahar1 opened a new pull request, #70395: URL: https://github.com/apache/airflow/pull/70395
Reverts #69161. The change is a provably wire-identical no-op: proto-plus already converts nested dicts recursively, so the pre- and post-change `CreateWorkflowInvocationRequest` serialize byte-identically and the dict path never dropped `invocation_config`. Unknown fields raise `ValueError` in proto-plus, so a silent drop at this layer was never possible. The symptom reported in #53843 was caused outside Airflow: dataform-core 3.0.10–3.0.16 defaulted incremental tables to `protected: true` (dataform-co/dataform#1914), which silently disables full refresh server-side; fixed upstream in core 3.0.17 (dataform-co/dataform#1942). Repos pin `dataformCoreVersion`, which is why the symptom persisted after the upstream fix. #53843 has been closed as not planned accordingly. The accompanying test only pinned third-party proto-plus conversion behavior, which the project's testing standards exclude. related: #53843 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Fable 5) Generated-by: Claude Code (Fable 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- 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]
