jason810496 commented on PR #71213: URL: https://github.com/apache/airflow/pull/71213#issuecomment-5312042346
> When do we plan to actually use this? I wonder if a property similar to the SDK DAG class’s property is enough. When introducing the native Dag DSL. We have to introduce at "serialization Dag level" so the native Dag is able to set the flag. (native Dag DSL -> Serialized Dag JSON -> DagFileParsingResult -> LazyDeserializedDAG) > > * Only the Lang SDK Dag can explicitly set the `is_mixed_language_dag` flag at Dag level argument. > > This does not seem to be a part of the PR? Why are you putting this here? Even if it’s true, why is it the case? IMO, we need to define the spec first (which is the purpose of this PR) before wiring the native Dag part up. -- 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]
