[ https://issues.apache.org/jira/browse/HBASE-6071?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13283301#comment-13283301 ]
Zhihong Yu commented on HBASE-6071: ----------------------------------- I used my script to scan https://builds.apache.org/job/PreCommit-HBASE-Build/1991/console and didn't find hanging test. {code} + String message = String.format("Exception during try #%d (out of %d):",tries+1, numRetries); + LOG.debug(message,t); {code} nit: space should be inserted between comma and tries, between comma and t. My original intention was using the above debug log to facilitate finding root cause. It would be nice if debug log is added to 0.90 release you're using and see what we get. > getRegionServerWithRetires, should log unsuccessful attempts and exceptions. > ---------------------------------------------------------------------------- > > Key: HBASE-6071 > URL: https://issues.apache.org/jira/browse/HBASE-6071 > Project: HBase > Issue Type: Improvement > Components: client, ipc > Affects Versions: 0.92.0, 0.94.0 > Reporter: Igal Shilman > Priority: Minor > Labels: client, ipc > Attachments: HBASE-6071.patch, > HConnectionManager_HBASE-6071-0.90.0.patch > > > HConnectionImplementation.getRegionServerWithRetries might terminate w/ an > exception different then a DoNotRetryIOException, thus silently drops > exceptions from previous attempts. > [~ted_yu] suggested > ([here|http://mail-archives.apache.org/mod_mbox/hbase-user/201205.mbox/%3CCAFebPXBq9V9BVdzRTNr-MB3a1Lz78SZj6gvP6On0b%2Bajt9StAg%40mail.gmail.com%3E]) > adding a log message inside the catch block describing the exception type > and details. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira