Brunda10 commented on PR #55408:
URL: https://github.com/apache/airflow/pull/55408#issuecomment-3332334084
Hi @maintainers,
I see that several CI jobs are failing, and after checking the logs, the
failures seem related to serialization tests in test_utils.py.
Specifically, the new event_extra_template field I added is showing up in
the serialized asset output (as an empty dict by default), which causes the
assertion mismatch.
For example:
```
{
'__type': 'asset',
- 'extra': {'a': 1},
+ 'event_extra_template': {},
+ 'extra': {'a': 1},
'group': 'asset',
'name': 'uri1',
'uri': 'uri1',
}
```
Before I update the tests, could you confirm if it’s acceptable to always
serialize event_extra_template (defaulting to {} when not set)?
Or would you prefer I exclude it from serialization when it’s empty, to
avoid breaking existing expectations?
Thanks!
--
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]
