amoghrajesh commented on issue #43872: URL: https://github.com/apache/airflow/issues/43872#issuecomment-2567733935
> Wouldn't this confuse users when they specify "limit" as part of the request body/param, unaware of the maximum_page_limit? As in, if limit > maximum_page_limit, the API response returns the maximum_page_limit number of records. The user needs to refer to the warning logs to understand that the reason for the truncation is that they have set maximum_page_limit. I think, along with the warning log, we should also update REST API documentation. Yeah, that's a fair point. However, we do mention this in the API doc https://github.com/apache/airflow/blob/f7da5e48a2740a2bfb1c3dd57bb463ae29599a26/docs/apache-airflow/security/api.rst#page-size-limit about default size. I am all for adding it for clarification purpose, but probably that should be a different issue, unrelated to this @rawwar? -- 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]
