kamalcph opened a new pull request, #14778:
URL: https://github.com/apache/kafka/pull/14778

   DelayedRemoteFetch uses `fetch.max.wait.ms` config as a delay timeout for 
DelayedRemoteFetchPurgatory. `fetch.max.wait.ms` purpose is to wait for the 
given amount of time when there is no data available to serve the FETCH request.
   
   ```
   The maximum amount of time the server will block before answering the fetch 
request if there isn't sufficient data to immediately satisfy the requirement 
given by fetch.min.bytes.
   ```
   
   Using the same timeout in the DelayedRemoteFetchPurgatory can confuse the 
user on how to configure optimal value for each purpose. Moreover, the config 
is of LOW importance and most of the users won't configure it and use the 
default value of 500 ms.
   
   Having the delay timeout of 500 ms in DelayedRemoteFetchPurgatory can lead 
to higher number of expired delayed remote fetch requests when the remote 
storage have any degradation.
   
   Test: Existing unit and integration tests
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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: jira-unsubscr...@kafka.apache.org

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

Reply via email to