[ https://issues.apache.org/jira/browse/HBASE-8305?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626658#comment-13626658 ]
Jonathan Hsieh commented on HBASE-8305: --------------------------------------- Looks good. I had the same question as jm -- why delete instead of convert to trace? How large are the logs after the change (what did we win?) nit: unrelated? {code} @@ -1873,7 +1873,7 @@ public class HConnectionManager { */ void deleteCachedLocation(HRegionInfo hri, HRegionLocation source) { boolean isStaleDelete = false; - HRegionLocation oldLocation = null; + HRegionLocation oldLocation; synchronized (this.cachedRegionLocations) { Map<byte[], HRegionLocation> tableLocations = getTableLocations(hri.getTableName()); {code} > Too much logs in the some tests > ------------------------------- > > Key: HBASE-8305 > URL: https://issues.apache.org/jira/browse/HBASE-8305 > Project: HBase > Issue Type: Bug > Components: build, test > Affects Versions: 0.95.0 > Reporter: Nicolas Liochon > Assignee: Nicolas Liochon > Attachments: 8305.v1.patch, 8305.v2.patch > > > org.apache.hadoop.hbase.client.TestAdmin-output.txt 235.45 MB > org.apache.hadoop.hbase.master.TestTableLockManager-output.txt 4.15 MB > org.apache.hadoop.hbase.regionserver.TestAtomicOperation-output.txt 5.47 MB > org.apache.hadoop.hbase.regionserver.TestEndToEndSplitTransaction-output.txt > 16.07 MB -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira