jiajunwang edited a comment on pull request #974: URL: https://github.com/apache/helix/pull/974#issuecomment-628838174
> There are too many places that previously read partial data but now thrown exception. We need to reason for each case that when exception thrown, what is the difference of behavior and what should be right behavior. I believe, on the contrary, we need a reason for each case that does not throw an exception. The partial reads cause many problems. We need to be very strict. Moreover, the batch retry logic has been checked in to ensure connectivity issues won't be a concern anymore. And about "what should be right behavior", my take is that what was there is not necessary the right behavior. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
