saintstack commented on a change in pull request #3325:
URL: https://github.com/apache/hbase/pull/3325#discussion_r641240369
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
##########
@@ -486,7 +488,9 @@ boolean incNextCallSeq(long currentSeq) {
// cache the String once made.
@Override
public String toString() {
- return this.clientIPAndPort + ", " +
this.r.getRegionInfo().getRegionNameAsString();
+ return "clientIPAndPort=" + this.clientIPAndPort +
+ ", userName=" + this.userName +
+ ", regionInfo=" + this.r.getRegionInfo().getRegionNameAsString();
Review comment:
I like this change but I think you should do the encoded regionname here
rather than the full thing? The former is 32 bytes. The latter could be
anything. On other hand, full name would be more convenient. If only emitted
on scan lease expiration, then maybe this is ok. What do you think?
--
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]
