anmolnar commented on pull request #4125: URL: https://github.com/apache/hbase/pull/4125#issuecomment-1073970823
> adding sync call in the code path will cause serious performance impact I'll take another look at `BufferCallBeforeInitHandler.java` and try to leverage that instead of using the `sync()` call, but would you please clarify why do you say it will have serious performance impact? As mentioned, the client doesn't handle many connections and it must wait for the connection to be fully usable before doing anything. I don't see big difference in _where_ we actually pile up the incoming calls: in a handler or force the client stop until we're ready. The client has to wait either way. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
