[GitHub] [airflow] ephraimbuddy commented on pull request #9338: Get API request limit and offset from parameters.py

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9338: URL: https://github.com/apache/airflow/pull/9338#issuecomment-645717641 I didn't have any issue when I added maximum. You should add it immediately after minimum. See https://swagger.io/docs/specification/data-models/data-types/ ---

[GitHub] [airflow] ephraimbuddy commented on pull request #9338: Get API request limit and offset from parameters.py

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9338: URL: https://github.com/apache/airflow/pull/9338#issuecomment-645715798 The 400 for the limit=0 LGTM. I will investigate now for the maximum This is an automated message from the

[GitHub] [airflow] ephraimbuddy commented on pull request #9338: Get API request limit and offset from parameters.py

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9338: URL: https://github.com/apache/airflow/pull/9338#issuecomment-645337531 @mik-laj This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [airflow] ephraimbuddy commented on pull request #9338: Get API request limit and offset from parameters.py

2020-06-17 Thread GitBox
ephraimbuddy commented on pull request #9338: URL: https://github.com/apache/airflow/pull/9338#issuecomment-645320694 No. The default of 100 means that if limit is not applied then a limit of 100 will be applied. Users should be able to fetch more than 100 items if they want

[GitHub] [airflow] ephraimbuddy commented on pull request #9338: Get API request limit and offset from parameters.py

2020-06-16 Thread GitBox
ephraimbuddy commented on pull request #9338: URL: https://github.com/apache/airflow/pull/9338#issuecomment-645055416 I think it will be better to allow connexion pass it for us by adding it as keyword arguments to the views. What do you think? Take a look at this: https://github.com/ap