SEPURI-SAI-KRISHNA commented on issue #74459:
URL: https://github.com/apache/airflow/issues/74459#issuecomment-6076594390
Correcting my earlier comment: my verification of #74199 in `dbt.cloud
4.11.0rc1` was wrong.
I checked that `hook_params` survives `serialize()`, but I used
`{"retry_limit": 7, "retry_delay": 3.5}`, two JSON serializable scalars.
`hook_params` is a free form dict and the one key whose documented values are
Python objects is `retry_args`, so my input could not exercise the case that
matters. With a tenacity object in `retry_args` the task fails when it defers,
and it fails after the dbt Cloud job has already started, which leaves the run
orphaned.
Thanks to @FrankYang0529 for catching it and for the fix in #74489.
Treating that as a release blocker for `dbt.cloud 4.11.0`: shipping rc1 as
is would regress any deferrable `DbtCloudRunJobOperator` whose `hook_params`
carries `retry_args`, which worked on 4.10.0. The google 22.7.0rc1 item I
reported is unaffected and my result there stands.
--
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]
