YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-997885255
Refined the variable name in the latest commit. I think this is more
readable then.
--
This is an automated message from the Apache Git Service.
To respond to the message,
YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-994250531
Left some comments to explain in the corresponding ticket. @Reidd
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-994250198
Left comments
> LGTM. Can you explain more about the pics in the ticket you upload? I
don't quite catch what it tells
--
This is an automated message from the
YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-990925914
The failed UT is not related with the changes.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-990532276
Reconstructed the code and extracted the RpcHandler to avoid the redundant
coding of fastpath handler.
--
This is an automated message from the Apache Git Service.
To respond
YutSean commented on pull request #3929:
URL: https://github.com/apache/hbase/pull/3929#issuecomment-989577939
I have two considerations.
1. Wether we should make the fastpath as a new config that user can turn on
or off alternatively (Because it seems that when the number of handlers