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

   > 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.
   
   yes completely agree with you on this point @potiuk There is high chance for 
user to miss out reading the description. Throwing this information in the logs 
is the best way to prevent such confusion. I will do that part. 


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

Reply via email to