[GitHub] [airflow] mhenc commented on pull request #29188: AIP-44 Migrate Dagbag.sync_to_db to internal API.

["mhenc (via GitHub)" <gi...@apache.org>]

mhenc commented on PR #29188:
URL: https://github.com/apache/airflow/pull/29188#issuecomment-1411989287

   > From the AIP, i got the impression that we would be sending the serialized dag to the internal api. Will `BaseSerialization.serialize` in the `internal_api_call` decorator take care about the serialization?
   
   Yes, the InternalApi decorator/server serializes/deserializes the objects using BaseSerialization.
   
   https://github.com/apache/airflow/blob/main/airflow/api_internal/internal_api_call.py#L107
   
    I see that you have problems serializing `dags`, which is `dict[String,DAG]`. We may need to extend the BaseSerialization to support it. Let me take a look.


-- 
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-unsubscribe@airflow.apache.org

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