mhenc commented on code in PR #29355:
URL: https://github.com/apache/airflow/pull/29355#discussion_r1112983708


##########
airflow/serialization/serialized_objects.py:
##########
@@ -502,6 +504,8 @@ def deserialize(cls, encoded_var: Any) -> Any:
             return Dataset(**var)
         elif type_ == DAT.SIMPLE_TASK_INSTANCE:
             return SimpleTaskInstance(**cls.deserialize(var))
+        elif type_ == DAT.TASK_INSTANCE:

Review Comment:
   But this works only for client -> Internal API side.
   
   What about Internal API -> client, e.g. worker. We need to have TaskInstance 
object in worker to run the task, e.g.
   
https://github.com/apache/airflow/blob/main/airflow/cli/commands/task_command.py#L187
   
   unless of course we are able to refactor it completely



-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to