924060929 commented on pull request #5336: URL: https://github.com/apache/incubator-doris/pull/5336#issuecomment-786366712
> @924060929 This is a nice capture. I am curious about whether the lock is necessary when running in sync mode (default mode)? Unnecessary, just for simplify. Synchronized has little effect on performance in sync mode. Shall we remove synchronized in sync mode? ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org