[ https://issues.apache.org/jira/browse/HBASE-5708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13247095#comment-13247095 ]
Phabricator commented on HBASE-5708: ------------------------------------ mbautin has commented on the revision "[jira] [HBASE-5708] [89-fb] Improving robustness of map-reduce-related and other unit tests". INLINE COMMENTS src/main/java/org/apache/hadoop/hbase/master/OldLogsCleaner.java:98-100 Saw a NullPointerException on the next line. src/main/java/org/apache/hadoop/hbase/master/SplitLogManager.java:984-987 Saw an infinite loop with a SESSIONEXPIRED here in TestSplitLogManager. src/main/java/org/apache/hadoop/hbase/master/SplitLogManager.java:1070-1073 Saw an infinite loop with a SESSIONEXPIRED here in TestSplitLogManager. src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java:1591 Previously, we would proceed to process the open region message, even though we have re-queued it for later. src/test/java/org/apache/hadoop/hbase/HBaseTestCase.java:52 A lot of stuff in this deprecated class should eventually be delegated to HBaseTestingUtility. src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java:138-139 Sometimes we just want to reuse the mini-map-reduce-cluster-starting capabilities of HBaseTestingUtility, but provide a NameNode URI ourselves. src/test/java/org/apache/hadoop/hbase/regionserver/wal/TestHLog.java:364 Observed failures to start namenode's trash cleaning thread when it failed to connect as a client to the newly started namenode. Adding a delay fixed the problem in some of those cases. src/test/java/org/apache/hadoop/hbase/util/TestMiniClusterLoadSequential.java:80 The load balancer would unassign some regions in a load-test unit test, resulting in a failure, which is totally ridiculous. src/test/resources/hbase-site.xml:145 A lot of tests used to fail with ZK session expiration errors. I have set ZK session timeout to 1 sec in a few places where it seemed necessary. src/test/resources/hbase-site.xml:155 We did not have enough retries in some unit tests. This will hopefully add some robustness. REVISION DETAIL https://reviews.facebook.net/D2601 > [89-fb] Make MiniMapRedCluster directory a subdirectory of target/test > ---------------------------------------------------------------------- > > Key: HBASE-5708 > URL: https://issues.apache.org/jira/browse/HBASE-5708 > Project: HBase > Issue Type: Bug > Reporter: Mikhail Bautin > Priority: Minor > Attachments: D2601.1.patch, D2601.2.patch, D2601.3.patch > > > Some map-reduce-based tests are failing when executed concurrently in 89-fb > because mini-map-reduce cluster uses /tmp/hadoop-<username> for temporary > data. -- 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