slfan1989 commented on PR #6404: URL: https://github.com/apache/hadoop/pull/6404#issuecomment-1878445162
@simbadzina Thanks for helping review the code! @LiuGuH Thanks for the contribution! but I think it's better if we don't introduce a new one if the original one works. Personally, I'm worried about the synchronization state of a single thread, it doesn't seem to be a good design. If some boundaries occur (such as NN restart, hang, etc.), how do we deal with it? If we are going to design a more complex mechanism to handle boundary issues, I think it is better to use the mechanisms that are already available. -- 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]
