amoghrajesh commented on code in PR #52693:
URL: https://github.com/apache/airflow/pull/52693#discussion_r2179538603
##########
airflow-core/src/airflow/serialization/serialized_objects.py:
##########
@@ -317,19 +317,20 @@ def decode_asset_condition(var: dict[str, Any]) ->
BaseAsset:
raise ValueError(f"deserialization not implemented for DAT {dat!r}")
-def decode_asset(var: dict[str, Any]):
- def _smart_decode_trigger_kwargs(d):
- """
- Slightly clean up kwargs for display.
+def smart_decode_trigger_kwargs(d):
+ """
+ Slightly clean up kwargs for display.
- This detects one level of BaseSerialization and tries to deserialize
the
- content, removing some __type __var ugliness when the value is
displayed
- in UI to the user.
- """
- if not isinstance(d, dict) or Encoding.TYPE not in d:
- return d
- return BaseSerialization.deserialize(d)
+ This detects one level of BaseSerialization and tries to deserialize the
+ content, removing some __type __var ugliness when the value is displayed
+ in UI to the user.
Review Comment:
Yeah that's misleading, pushed a change in bb9f258b7f
--
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]
