mik-laj commented on pull request #9338:
URL: https://github.com/apache/airflow/pull/9338#issuecomment-645486285


   @OmairK Rate limit is another problem that can be solved on a different 
layer, e.g. Load Balancer, so we don't have to deal with it for now.
   
   @ephraimbuddy @svetli-n We should have a limit that should be configured in 
`airflow.cfg`, but for now we can set it as a fixed value - 100. This is to 
protect against a DOS attack. Fetching one billion records into memory can 
cause all server memory to be consumed. This problem will not occur if the user 
fetches only a subset of items in one request. What do you think 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.

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


Reply via email to