Pil0tXia commented on PR #4837: URL: https://github.com/apache/eventmesh/pull/4837#issuecomment-2057007941
@cnzakii >That's a good idea, should we retry any failed request? I would recommend this as a useful feature, but it is not a necessary task. >Re-initiating a request that failed at the network level may work, but re-initiating a request that returned an error status code will usually not change the result You've found a good point. In that case, you can put network error requests into the third queue and only retry network errors. This also applies to ordinary HTTP requests. The number of retry attempts can be set as a YAML parameter, and when its value is set to 0, it means no retries. Additionally, we can introduce another parameter that allows the user to decide whether to only retry network errors or retry all requests with non-200 status codes. By default, only network errors are retried. -- 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: issues-unsubscr...@eventmesh.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@eventmesh.apache.org For additional commands, e-mail: issues-h...@eventmesh.apache.org
