mik-laj commented on a change in pull request #9503:
URL: https://github.com/apache/airflow/pull/9503#discussion_r449274053



##########
File path: airflow/api_connexion/endpoints/dag_run_endpoint.py
##########
@@ -69,24 +61,24 @@ def get_dag_runs(session, dag_id, start_date_gte=None, 
start_date_lte=None,
 
     # filter start date
     if start_date_gte:
-        query = query.filter(DagRun.start_date >= start_date_gte)
+        query = query.filter(DagRun.start_date >= 
timezone.parse(start_date_gte))

Review comment:
       We cannot make such a change to the specification because API clients 
must support this type of field. It will not be easy to add custom type support 
for all generated clients. We can try to make such a change only in memory when 
loading the file, but I'm not sure if we need it. The current solution meets 
all our requirements.




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to