kaxil commented on PR #56660: URL: https://github.com/apache/airflow/pull/56660#issuecomment-3412503407
> > > Fundamental question: Should `dag.test()` create a temporary serialized DAG version like it was doing in one of the early 3.0.x version, or should it bypass the serialization requirement entirely? > > > > > > `dag.test` uses an inmemory serialized dag now which I think it was doing in the earlier version. The reason it was working before was that there was no requirement to have serialized dag in db before creating a dagrun. Now we have that requirement. > > The command option( `airflow dag test`) creates serialized dags in DB. And I think it's ok to create it. Helps us ensure we are testing with the real value. We can do the same for `dag.test` if it's prefered > > yeah is there much downside to just creating it? Yeah I am in favor of making it consistent to `airflow dag test` . Which means change will be very isolated and not change other parts of the codebase -- 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]
