You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "hussein-awala (via GitHub)" <gi...@apache.org> on 2023/03/04 22:07:15 UTC

[GitHub] [airflow] hussein-awala commented on pull request #29762: Add an example for dynamic task mapping with non-TaskFlow operator

hussein-awala commented on PR #29762:
URL: https://github.com/apache/airflow/pull/29762#issuecomment-1454902445

   @josh-fell @potiuk  The test `test_deserialization_across_process` failes because of [this assertion](https://github.com/apache/airflow/blob/main/tests/serialization/test_dag_serialization.py#L619), where in  serialized_partial_kwargs, python_callable is a string:
   ```
   'def add_one(x: int):\n    return x + 1\n'
   ```
   but in the original_partial_kwargs it's a function.
   
   Is it supposed to keep the function as it is in the serialized mapped task or it's bug in test?


-- 
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