Lee-W commented on code in PR #74695: URL: https://github.com/apache/airflow/pull/74695#discussion_r4398318249
########## providers/common/ai/docs/structured_output.rst: ########## @@ -34,6 +35,8 @@ classes nested inside a function or ``@dag``-decorated body, parameterized generics, and dynamically-built classes whose ``__name__`` does not match the attribute they are bound to cannot be re-imported, so they are skipped with a warning at worker startup and the value fails to deserialize at the consumer. +A list of types (``output_type=[MyModel, int]``) registers no class, so the operator pushes the +model as a ``dict`` even when ``serialize_output`` is ``False``. Review Comment: `structured_output.rst`, `stability.rst` and the changelog note now mentions `LLMOperator`, `@task.llm` and `LLMFileAnalysisOperator` next to `AgentOperator`, and the pre-3.3 bullet mentions the LLM operators still push `list[MyModel]` as instances. -- 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]
