You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/02/26 16:19:47 UTC

[GitHub] [airflow] potiuk commented on pull request #29773: better description for limit in api

potiuk commented on PR #29773:
URL: https://github.com/apache/airflow/pull/29773#issuecomment-1445400834

   I agree it was confusing and the way it is described now is vey accurate. Thanks @bowrna for that.
   
   Additionally - maybe that would be an option to consider as a follow-up we could print (if not already done) warning in webserver's logs in case someone uses page limit higher than maximum? This way there would be at least an indication in the logs that someone is doing something not really expected and the admins of Airflow could potentially warn the user of the API about it.


-- 
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: commits-unsubscribe@airflow.apache.org

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