Nataneljpwd commented on code in PR #62114:
URL: https://github.com/apache/airflow/pull/62114#discussion_r2867469908
##########
airflow-core/tests/unit/jobs/test_scheduler_job.py:
##########
@@ -7435,8 +7435,8 @@ def
test_activate_referenced_assets_with_no_existing_warning(self, session, test
dag1 = DAG(dag_id=dag_id1, start_date=DEFAULT_DATE, schedule=[asset1,
asset1_1, asset1_2])
sync_dag_to_db(dag1, session=session)
- asset_models = session.scalars(select(AssetModel)).all()
- assert len(asset_models) == 3
+ asset_models = select(AssetModel).cte()
Review Comment:
How do you think this can be added? As it does not cause failure when using
in, rather just cause some slowdown
The only think I can think of is to check for the keyword 'in' for the str
of the query
--
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]
