mokai87 commented on a change in pull request #2219:
URL: https://github.com/apache/hbase/pull/2219#discussion_r470995249
##########
File path:
hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java
##########
@@ -199,6 +199,10 @@ public void close() {
if (!closed.compareAndSet(false, true)) {
return;
}
+ LOG.info("Connection has been closed by {}.",
Thread.currentThread().getName());
Review comment:
I think the stacktrace is an option if the thread name is helpless. It's
almost enough with meaningful name. I'm also worried log stack at info level is
confusing since the format is like an exception.
----------------------------------------------------------------
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]
