zuston commented on PR #1715: URL: https://github.com/apache/incubator-uniffle/pull/1715#issuecomment-2124325896
> > This looks strange that it replace the partial blockMaxRetryTimes abilities and introduces the extra config option to meet something, but this change is not compatible with the previous logic. > > What do you recommend? Or continue to reuse blockMaxRetryTimes After rethinking, I think we don't need this. If you want to increase retry count, you just need to increase the block retry max times. -- 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]
