I took jstack (which I didn't keep) of a hanging surefire process and saw RegionServerThread. I should have examined other threads in the trace.
Will do that next time I see similar test failure. BTW how can we know whether there were hanging surefire processes on Jenkins ? On Mon, Aug 8, 2011 at 3:15 PM, Stack <[email protected]> wrote: > How does your suggested change relate to the lock Ted? You are > daemonizing hbase servers but seems like its an outstanding hdfs > server that is the prob? > St.Ack > > On Mon, Aug 8, 2011 at 9:18 AM, Ted Yu <[email protected]> wrote: > > Hi, > > You may have noticed unit test failures with message similar to the > > following: > > testInfoServersRedirect(org.apache.hadoop.hbase.TestInfoServers): Cannot > > lock storage /home/hadoop/hbase/build/hbase/test/dfs/name1. The directory > is > > already locked. > > testInfoServersStatusPages(org.apache.hadoop.hbase.TestInfoServers): > > Cannot lock storage /home/hadoop/hbase/build/hbase/test/dfs/name1. The > > directory is already locked. > > This indicated that certain JVMClusterUtil was hanging after the > underlying > > unit test finished. > > > > I suggest making the following change to JVMClusterUtil: > > > > Index: src/main/java/org/apache/hadoop/hbase/util/JVMClusterUtil.java > > =================================================================== > > --- src/main/java/org/apache/hadoop/hbase/util/JVMClusterUtil.java > > (revision 1154705) > > +++ src/main/java/org/apache/hadoop/hbase/util/JVMClusterUtil.java > > (working copy) > > @@ -44,6 +44,7 @@ > > public RegionServerThread(final HRegionServer r, final int index) { > > super(r, "RegionServer:" + index + ";" + r.getServerName()); > > this.regionServer = r; > > + this.setDaemon(true); > > } > > > > /** @return the region server */ > > @@ -110,6 +111,7 @@ > > public MasterThread(final HMaster m, final int index) { > > super(m, "Master:" + index + ";" + m.getServerName()); > > this.master = m; > > + this.setDaemon(true); > > } > > > > /** @return the master */ > > > > Please comment. > > >
