ephraimbuddy commented on a change in pull request #9556:
URL: https://github.com/apache/airflow/pull/9556#discussion_r448810142
##########
File path: airflow/api_connexion/schemas/dag_run_schema.py
##########
@@ -72,5 +72,25 @@ class DAGRunCollectionSchema(Schema):
total_entries = fields.Int()
+class DagRunsBatchFormSchema(Schema):
+ """ Schema to validate and deserialize the Form(request payload) submitted
to DagRun Batch endpoint"""
+
+ class Meta:
+ """ Meta """
+ datetimeformat = 'iso'
+ strict = True
+
+ page_offset = fields.Int(required=False, missing=0, min=0)
+ page_limit = fields.Int(required=False, missing=100, min=1)
Review comment:
I think it's better left out. Some servers can handle more than 1000
page request limit. I'm working on configurable page request limit and we can
apply it here. Then it'll not be a hard limit but a limit that the users can
configure.
You can see the commit here
https://github.com/apache/airflow/commit/a37ac22c179e5dccc95b05c862bffc542dad125e
----------------------------------------------------------------
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:
[email protected]
