bugraoz93 commented on code in PR #55744:
URL: https://github.com/apache/airflow/pull/55744#discussion_r2356783703
##########
airflow-ctl/src/airflowctl/api/operations.py:
##########
@@ -244,7 +244,7 @@ def create_event(
"""Create an asset event."""
try:
self.response = self.client.post(
Review Comment:
```suggestion
if asset_event_body.extra is None:
asset_event_body.extra = {}
self.response = self.client.post(
```
I think this will fix the bug :) This is some kind of integration problem we
are having with dictionary fields in airflowctl (There are a couple of cases we
have dict extra fields, such as in `connections`. In some cases, in the
database, they cannot be `null`, but it can be `null` on the datamodel, which
ends up in error if it is passed as `None`
Please see the example below; it is the same problem. Let's patch it like
this for now and we can come up with more generic solution to this
https://github.com/apache/airflow/blob/8d891bb98f318a15b063d9e9faa4a4fd6b49ef3d/airflow-ctl/src/airflowctl/api/operations.py#L573-L585
--
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]
