ZanderXu commented on PR #4660: URL: https://github.com/apache/hadoop/pull/4660#issuecomment-1200156797
@slfan1989 Thanks for your review. > I feel that this change is a bit risky, will this lead to instability of the service? Sometimes it is reasonable to configure without timeout Sorry, I didn't get your idea. Can you share more detailed information or cases? > The main question is is it reasonable to use the same timeout configuration for different protocols? According to my practical experience, using one configuration `ipc.client.rpc-timeout.ms` is enough. Of course, if some masters feel it is necessary to distinguish this configuration, I will modify this patch. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
