amoghrajesh commented on code in PR #67418:
URL: https://github.com/apache/airflow/pull/67418#discussion_r3294970251
##########
airflow-core/src/airflow/api_fastapi/execution_api/routes/asset_state.py:
##########
@@ -105,7 +106,7 @@ def set_asset_state_by_name(
) -> None:
"""Set an asset state value by asset name."""
asset_id = _resolve_asset_id_by_name(name, session)
- get_state_backend().set(AssetScope(asset_id=asset_id), key, body.value,
session=session)
+ get_state_backend().set(AssetScope(asset_id=asset_id), key,
json.dumps(body.value), session=session)
Review Comment:
Handled it in dd4f62afd2
##########
airflow-core/src/airflow/api_fastapi/execution_api/datamodels/task_state.py:
##########
@@ -19,17 +19,26 @@
from datetime import datetime
+from pydantic import JsonValue, field_validator
+
from airflow.api_fastapi.core_api.base import StrictBaseModel
class TaskStateResponse(StrictBaseModel):
"""Task state value returned to a worker."""
- value: str
+ value: JsonValue
Review Comment:
Handled it in
https://github.com/apache/airflow/commit/dd4f62afd2ecf47189351c151022ee68eada2326
--
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]
