| <!DOCTYPE HTML> |
| <html lang="en"> |
| <head> |
| <!-- Generated by javadoc (17) --> |
| <title>Source code</title> |
| <meta name="viewport" content="width=device-width, initial-scale=1"> |
| <meta name="description" content="source: package: org.apache.hadoop.hbase.master.assignment, class: TestOpenRegionProcedureHang"> |
| <meta name="generator" content="javadoc/SourceToHTMLConverter"> |
| <link rel="stylesheet" type="text/css" href="../../../../../../../stylesheet.css" title="Style"> |
| </head> |
| <body class="source-page"> |
| <main role="main"> |
| <div class="source-container"> |
| <pre><span class="source-line-no">001</span><span id="line-1">/*</span> |
| <span class="source-line-no">002</span><span id="line-2"> * Licensed to the Apache Software Foundation (ASF) under one</span> |
| <span class="source-line-no">003</span><span id="line-3"> * or more contributor license agreements. See the NOTICE file</span> |
| <span class="source-line-no">004</span><span id="line-4"> * distributed with this work for additional information</span> |
| <span class="source-line-no">005</span><span id="line-5"> * regarding copyright ownership. The ASF licenses this file</span> |
| <span class="source-line-no">006</span><span id="line-6"> * to you under the Apache License, Version 2.0 (the</span> |
| <span class="source-line-no">007</span><span id="line-7"> * "License"); you may not use this file except in compliance</span> |
| <span class="source-line-no">008</span><span id="line-8"> * with the License. You may obtain a copy of the License at</span> |
| <span class="source-line-no">009</span><span id="line-9"> *</span> |
| <span class="source-line-no">010</span><span id="line-10"> * http://www.apache.org/licenses/LICENSE-2.0</span> |
| <span class="source-line-no">011</span><span id="line-11"> *</span> |
| <span class="source-line-no">012</span><span id="line-12"> * Unless required by applicable law or agreed to in writing, software</span> |
| <span class="source-line-no">013</span><span id="line-13"> * distributed under the License is distributed on an "AS IS" BASIS,</span> |
| <span class="source-line-no">014</span><span id="line-14"> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.</span> |
| <span class="source-line-no">015</span><span id="line-15"> * See the License for the specific language governing permissions and</span> |
| <span class="source-line-no">016</span><span id="line-16"> * limitations under the License.</span> |
| <span class="source-line-no">017</span><span id="line-17"> */</span> |
| <span class="source-line-no">018</span><span id="line-18">package org.apache.hadoop.hbase.master.assignment;</span> |
| <span class="source-line-no">019</span><span id="line-19"></span> |
| <span class="source-line-no">020</span><span id="line-20">import java.io.IOException;</span> |
| <span class="source-line-no">021</span><span id="line-21">import java.util.concurrent.CountDownLatch;</span> |
| <span class="source-line-no">022</span><span id="line-22">import java.util.concurrent.TimeUnit;</span> |
| <span class="source-line-no">023</span><span id="line-23">import org.apache.hadoop.conf.Configuration;</span> |
| <span class="source-line-no">024</span><span id="line-24">import org.apache.hadoop.hbase.HBaseClassTestRule;</span> |
| <span class="source-line-no">025</span><span id="line-25">import org.apache.hadoop.hbase.HBaseTestingUtil;</span> |
| <span class="source-line-no">026</span><span id="line-26">import org.apache.hadoop.hbase.HConstants;</span> |
| <span class="source-line-no">027</span><span id="line-27">import org.apache.hadoop.hbase.PleaseHoldException;</span> |
| <span class="source-line-no">028</span><span id="line-28">import org.apache.hadoop.hbase.StartTestingClusterOption;</span> |
| <span class="source-line-no">029</span><span id="line-29">import org.apache.hadoop.hbase.TableName;</span> |
| <span class="source-line-no">030</span><span id="line-30">import org.apache.hadoop.hbase.client.RegionInfo;</span> |
| <span class="source-line-no">031</span><span id="line-31">import org.apache.hadoop.hbase.master.HMaster;</span> |
| <span class="source-line-no">032</span><span id="line-32">import org.apache.hadoop.hbase.master.MasterServices;</span> |
| <span class="source-line-no">033</span><span id="line-33">import org.apache.hadoop.hbase.master.RegionPlan;</span> |
| <span class="source-line-no">034</span><span id="line-34">import org.apache.hadoop.hbase.master.procedure.MasterProcedureEnv;</span> |
| <span class="source-line-no">035</span><span id="line-35">import org.apache.hadoop.hbase.master.region.MasterRegion;</span> |
| <span class="source-line-no">036</span><span id="line-36">import org.apache.hadoop.hbase.procedure2.ProcedureExecutor;</span> |
| <span class="source-line-no">037</span><span id="line-37">import org.apache.hadoop.hbase.regionserver.HRegionServer;</span> |
| <span class="source-line-no">038</span><span id="line-38">import org.apache.hadoop.hbase.testclassification.MasterTests;</span> |
| <span class="source-line-no">039</span><span id="line-39">import org.apache.hadoop.hbase.testclassification.MediumTests;</span> |
| <span class="source-line-no">040</span><span id="line-40">import org.apache.hadoop.hbase.util.Bytes;</span> |
| <span class="source-line-no">041</span><span id="line-41">import org.apache.hadoop.hbase.util.JVMClusterUtil.MasterThread;</span> |
| <span class="source-line-no">042</span><span id="line-42">import org.apache.zookeeper.KeeperException;</span> |
| <span class="source-line-no">043</span><span id="line-43">import org.junit.AfterClass;</span> |
| <span class="source-line-no">044</span><span id="line-44">import org.junit.BeforeClass;</span> |
| <span class="source-line-no">045</span><span id="line-45">import org.junit.ClassRule;</span> |
| <span class="source-line-no">046</span><span id="line-46">import org.junit.Test;</span> |
| <span class="source-line-no">047</span><span id="line-47">import org.junit.experimental.categories.Category;</span> |
| <span class="source-line-no">048</span><span id="line-48">import org.slf4j.Logger;</span> |
| <span class="source-line-no">049</span><span id="line-49">import org.slf4j.LoggerFactory;</span> |
| <span class="source-line-no">050</span><span id="line-50"></span> |
| <span class="source-line-no">051</span><span id="line-51">import org.apache.hadoop.hbase.shaded.protobuf.ProtobufUtil;</span> |
| <span class="source-line-no">052</span><span id="line-52">import org.apache.hadoop.hbase.shaded.protobuf.generated.RegionServerStatusProtos.RegionStateTransition;</span> |
| <span class="source-line-no">053</span><span id="line-53">import org.apache.hadoop.hbase.shaded.protobuf.generated.RegionServerStatusProtos.RegionStateTransition.TransitionCode;</span> |
| <span class="source-line-no">054</span><span id="line-54">import org.apache.hadoop.hbase.shaded.protobuf.generated.RegionServerStatusProtos.ReportRegionStateTransitionRequest;</span> |
| <span class="source-line-no">055</span><span id="line-55">import org.apache.hadoop.hbase.shaded.protobuf.generated.RegionServerStatusProtos.ReportRegionStateTransitionResponse;</span> |
| <span class="source-line-no">056</span><span id="line-56"></span> |
| <span class="source-line-no">057</span><span id="line-57">/**</span> |
| <span class="source-line-no">058</span><span id="line-58"> * See HBASE-22060 and HBASE-22074 for more details.</span> |
| <span class="source-line-no">059</span><span id="line-59"> */</span> |
| <span class="source-line-no">060</span><span id="line-60">@Category({ MasterTests.class, MediumTests.class })</span> |
| <span class="source-line-no">061</span><span id="line-61">public class TestOpenRegionProcedureHang {</span> |
| <span class="source-line-no">062</span><span id="line-62"></span> |
| <span class="source-line-no">063</span><span id="line-63"> @ClassRule</span> |
| <span class="source-line-no">064</span><span id="line-64"> public static final HBaseClassTestRule CLASS_RULE =</span> |
| <span class="source-line-no">065</span><span id="line-65"> HBaseClassTestRule.forClass(TestOpenRegionProcedureHang.class);</span> |
| <span class="source-line-no">066</span><span id="line-66"></span> |
| <span class="source-line-no">067</span><span id="line-67"> private static final Logger LOG = LoggerFactory.getLogger(TestOpenRegionProcedureHang.class);</span> |
| <span class="source-line-no">068</span><span id="line-68"></span> |
| <span class="source-line-no">069</span><span id="line-69"> private static CountDownLatch ARRIVE;</span> |
| <span class="source-line-no">070</span><span id="line-70"> private static CountDownLatch RESUME;</span> |
| <span class="source-line-no">071</span><span id="line-71"></span> |
| <span class="source-line-no">072</span><span id="line-72"> private static CountDownLatch FINISH;</span> |
| <span class="source-line-no">073</span><span id="line-73"></span> |
| <span class="source-line-no">074</span><span id="line-74"> private static CountDownLatch ABORT;</span> |
| <span class="source-line-no">075</span><span id="line-75"></span> |
| <span class="source-line-no">076</span><span id="line-76"> private static final class AssignmentManagerForTest extends AssignmentManager {</span> |
| <span class="source-line-no">077</span><span id="line-77"></span> |
| <span class="source-line-no">078</span><span id="line-78"> public AssignmentManagerForTest(MasterServices master, MasterRegion masterRegion) {</span> |
| <span class="source-line-no">079</span><span id="line-79"> super(master, masterRegion);</span> |
| <span class="source-line-no">080</span><span id="line-80"> }</span> |
| <span class="source-line-no">081</span><span id="line-81"></span> |
| <span class="source-line-no">082</span><span id="line-82"> @Override</span> |
| <span class="source-line-no">083</span><span id="line-83"> public ReportRegionStateTransitionResponse reportRegionStateTransition(</span> |
| <span class="source-line-no">084</span><span id="line-84"> ReportRegionStateTransitionRequest req) throws PleaseHoldException {</span> |
| <span class="source-line-no">085</span><span id="line-85"> RegionStateTransition transition = req.getTransition(0);</span> |
| <span class="source-line-no">086</span><span id="line-86"> if (</span> |
| <span class="source-line-no">087</span><span id="line-87"> transition.getTransitionCode() == TransitionCode.OPENED</span> |
| <span class="source-line-no">088</span><span id="line-88"> && ProtobufUtil.toTableName(transition.getRegionInfo(0).getTableName()).equals(NAME)</span> |
| <span class="source-line-no">089</span><span id="line-89"> && ARRIVE != null</span> |
| <span class="source-line-no">090</span><span id="line-90"> ) {</span> |
| <span class="source-line-no">091</span><span id="line-91"> ARRIVE.countDown();</span> |
| <span class="source-line-no">092</span><span id="line-92"> try {</span> |
| <span class="source-line-no">093</span><span id="line-93"> RESUME.await();</span> |
| <span class="source-line-no">094</span><span id="line-94"> RESUME = null;</span> |
| <span class="source-line-no">095</span><span id="line-95"> } catch (InterruptedException e) {</span> |
| <span class="source-line-no">096</span><span id="line-96"> throw new RuntimeException(e);</span> |
| <span class="source-line-no">097</span><span id="line-97"> }</span> |
| <span class="source-line-no">098</span><span id="line-98"> try {</span> |
| <span class="source-line-no">099</span><span id="line-99"> return super.reportRegionStateTransition(req);</span> |
| <span class="source-line-no">100</span><span id="line-100"> } finally {</span> |
| <span class="source-line-no">101</span><span id="line-101"> FINISH.countDown();</span> |
| <span class="source-line-no">102</span><span id="line-102"> }</span> |
| <span class="source-line-no">103</span><span id="line-103"> } else {</span> |
| <span class="source-line-no">104</span><span id="line-104"> return super.reportRegionStateTransition(req);</span> |
| <span class="source-line-no">105</span><span id="line-105"> }</span> |
| <span class="source-line-no">106</span><span id="line-106"> }</span> |
| <span class="source-line-no">107</span><span id="line-107"> }</span> |
| <span class="source-line-no">108</span><span id="line-108"></span> |
| <span class="source-line-no">109</span><span id="line-109"> public static final class HMasterForTest extends HMaster {</span> |
| <span class="source-line-no">110</span><span id="line-110"></span> |
| <span class="source-line-no">111</span><span id="line-111"> public HMasterForTest(Configuration conf) throws IOException {</span> |
| <span class="source-line-no">112</span><span id="line-112"> super(conf);</span> |
| <span class="source-line-no">113</span><span id="line-113"> }</span> |
| <span class="source-line-no">114</span><span id="line-114"></span> |
| <span class="source-line-no">115</span><span id="line-115"> @Override</span> |
| <span class="source-line-no">116</span><span id="line-116"> protected AssignmentManager createAssignmentManager(MasterServices master,</span> |
| <span class="source-line-no">117</span><span id="line-117"> MasterRegion masterRegion) {</span> |
| <span class="source-line-no">118</span><span id="line-118"> return new AssignmentManagerForTest(master, masterRegion);</span> |
| <span class="source-line-no">119</span><span id="line-119"> }</span> |
| <span class="source-line-no">120</span><span id="line-120"></span> |
| <span class="source-line-no">121</span><span id="line-121"> @Override</span> |
| <span class="source-line-no">122</span><span id="line-122"> public void abort(String reason, Throwable cause) {</span> |
| <span class="source-line-no">123</span><span id="line-123"> // hang here so we can finish the reportRegionStateTransition call, which is the most</span> |
| <span class="source-line-no">124</span><span id="line-124"> // important part to reproduce the bug</span> |
| <span class="source-line-no">125</span><span id="line-125"> if (ABORT != null) {</span> |
| <span class="source-line-no">126</span><span id="line-126"> try {</span> |
| <span class="source-line-no">127</span><span id="line-127"> ABORT.await();</span> |
| <span class="source-line-no">128</span><span id="line-128"> ABORT = null;</span> |
| <span class="source-line-no">129</span><span id="line-129"> } catch (InterruptedException e) {</span> |
| <span class="source-line-no">130</span><span id="line-130"> throw new RuntimeException(e);</span> |
| <span class="source-line-no">131</span><span id="line-131"> }</span> |
| <span class="source-line-no">132</span><span id="line-132"> }</span> |
| <span class="source-line-no">133</span><span id="line-133"> super.abort(reason, cause);</span> |
| <span class="source-line-no">134</span><span id="line-134"> }</span> |
| <span class="source-line-no">135</span><span id="line-135"> }</span> |
| <span class="source-line-no">136</span><span id="line-136"></span> |
| <span class="source-line-no">137</span><span id="line-137"> private static final HBaseTestingUtil UTIL = new HBaseTestingUtil();</span> |
| <span class="source-line-no">138</span><span id="line-138"></span> |
| <span class="source-line-no">139</span><span id="line-139"> private static TableName NAME = TableName.valueOf("Open");</span> |
| <span class="source-line-no">140</span><span id="line-140"></span> |
| <span class="source-line-no">141</span><span id="line-141"> private static byte[] CF = Bytes.toBytes("cf");</span> |
| <span class="source-line-no">142</span><span id="line-142"></span> |
| <span class="source-line-no">143</span><span id="line-143"> @BeforeClass</span> |
| <span class="source-line-no">144</span><span id="line-144"> public static void setUp() throws Exception {</span> |
| <span class="source-line-no">145</span><span id="line-145"> Configuration conf = UTIL.getConfiguration();</span> |
| <span class="source-line-no">146</span><span id="line-146"> conf.setClass(HConstants.MASTER_IMPL, HMasterForTest.class, HMaster.class);</span> |
| <span class="source-line-no">147</span><span id="line-147"></span> |
| <span class="source-line-no">148</span><span id="line-148"> // make sure we do not timeout when caling reportRegionStateTransition</span> |
| <span class="source-line-no">149</span><span id="line-149"> conf.setInt(HConstants.HBASE_CLIENT_OPERATION_TIMEOUT, 10 * 60 * 1000);</span> |
| <span class="source-line-no">150</span><span id="line-150"> conf.setInt(HConstants.HBASE_RPC_SHORTOPERATION_TIMEOUT_KEY, 10 * 60 * 1000);</span> |
| <span class="source-line-no">151</span><span id="line-151"> UTIL.startMiniCluster(</span> |
| <span class="source-line-no">152</span><span id="line-152"> StartTestingClusterOption.builder().numMasters(2).numRegionServers(3).build());</span> |
| <span class="source-line-no">153</span><span id="line-153"> UTIL.createTable(NAME, CF);</span> |
| <span class="source-line-no">154</span><span id="line-154"> UTIL.waitTableAvailable(NAME);</span> |
| <span class="source-line-no">155</span><span id="line-155"> UTIL.getAdmin().balancerSwitch(false, true);</span> |
| <span class="source-line-no">156</span><span id="line-156"> }</span> |
| <span class="source-line-no">157</span><span id="line-157"></span> |
| <span class="source-line-no">158</span><span id="line-158"> @AfterClass</span> |
| <span class="source-line-no">159</span><span id="line-159"> public static void tearDown() throws Exception {</span> |
| <span class="source-line-no">160</span><span id="line-160"> UTIL.shutdownMiniCluster();</span> |
| <span class="source-line-no">161</span><span id="line-161"> }</span> |
| <span class="source-line-no">162</span><span id="line-162"></span> |
| <span class="source-line-no">163</span><span id="line-163"> @Test</span> |
| <span class="source-line-no">164</span><span id="line-164"> public void test() throws InterruptedException, KeeperException, IOException {</span> |
| <span class="source-line-no">165</span><span id="line-165"> RegionInfo region = UTIL.getMiniHBaseCluster().getRegions(NAME).get(0).getRegionInfo();</span> |
| <span class="source-line-no">166</span><span id="line-166"> AssignmentManager am = UTIL.getMiniHBaseCluster().getMaster().getAssignmentManager();</span> |
| <span class="source-line-no">167</span><span id="line-167"></span> |
| <span class="source-line-no">168</span><span id="line-168"> HRegionServer rs1 = UTIL.getRSForFirstRegionInTable(NAME);</span> |
| <span class="source-line-no">169</span><span id="line-169"> HRegionServer rs2 = UTIL.getOtherRegionServer(rs1);</span> |
| <span class="source-line-no">170</span><span id="line-170"></span> |
| <span class="source-line-no">171</span><span id="line-171"> ARRIVE = new CountDownLatch(1);</span> |
| <span class="source-line-no">172</span><span id="line-172"> RESUME = new CountDownLatch(1);</span> |
| <span class="source-line-no">173</span><span id="line-173"> FINISH = new CountDownLatch(1);</span> |
| <span class="source-line-no">174</span><span id="line-174"> ABORT = new CountDownLatch(1);</span> |
| <span class="source-line-no">175</span><span id="line-175"> am.moveAsync(new RegionPlan(region, rs1.getServerName(), rs2.getServerName()));</span> |
| <span class="source-line-no">176</span><span id="line-176"></span> |
| <span class="source-line-no">177</span><span id="line-177"> ARRIVE.await();</span> |
| <span class="source-line-no">178</span><span id="line-178"> ARRIVE = null;</span> |
| <span class="source-line-no">179</span><span id="line-179"> HMaster master = UTIL.getMiniHBaseCluster().getMaster();</span> |
| <span class="source-line-no">180</span><span id="line-180"> master.getZooKeeper().close();</span> |
| <span class="source-line-no">181</span><span id="line-181"> UTIL.waitFor(30000, () -> {</span> |
| <span class="source-line-no">182</span><span id="line-182"> for (MasterThread mt : UTIL.getMiniHBaseCluster().getMasterThreads()) {</span> |
| <span class="source-line-no">183</span><span id="line-183"> if (mt.getMaster() != master && mt.getMaster().isActiveMaster()) {</span> |
| <span class="source-line-no">184</span><span id="line-184"> return mt.getMaster().isInitialized();</span> |
| <span class="source-line-no">185</span><span id="line-185"> }</span> |
| <span class="source-line-no">186</span><span id="line-186"> }</span> |
| <span class="source-line-no">187</span><span id="line-187"> return false;</span> |
| <span class="source-line-no">188</span><span id="line-188"> });</span> |
| <span class="source-line-no">189</span><span id="line-189"> ProcedureExecutor<MasterProcedureEnv> procExec =</span> |
| <span class="source-line-no">190</span><span id="line-190"> UTIL.getMiniHBaseCluster().getMaster().getMasterProcedureExecutor();</span> |
| <span class="source-line-no">191</span><span id="line-191"> UTIL.waitFor(30000,</span> |
| <span class="source-line-no">192</span><span id="line-192"> () -> procExec.getProcedures().stream().filter(p -> p instanceof OpenRegionProcedure)</span> |
| <span class="source-line-no">193</span><span id="line-193"> .map(p -> (OpenRegionProcedure) p).anyMatch(p -> p.region.getTable().equals(NAME)));</span> |
| <span class="source-line-no">194</span><span id="line-194"> OpenRegionProcedure proc = procExec.getProcedures().stream()</span> |
| <span class="source-line-no">195</span><span id="line-195"> .filter(p -> p instanceof OpenRegionProcedure).map(p -> (OpenRegionProcedure) p)</span> |
| <span class="source-line-no">196</span><span id="line-196"> .filter(p -> p.region.getTable().equals(NAME)).findFirst().get();</span> |
| <span class="source-line-no">197</span><span id="line-197"> // wait a bit to let the OpenRegionProcedure send out the request</span> |
| <span class="source-line-no">198</span><span id="line-198"> Thread.sleep(2000);</span> |
| <span class="source-line-no">199</span><span id="line-199"> RESUME.countDown();</span> |
| <span class="source-line-no">200</span><span id="line-200"> if (!FINISH.await(15, TimeUnit.SECONDS)) {</span> |
| <span class="source-line-no">201</span><span id="line-201"> LOG.info("Wait reportRegionStateTransition to finish timed out, this is possible if"</span> |
| <span class="source-line-no">202</span><span id="line-202"> + " we update the procedure store, as the WALProcedureStore"</span> |
| <span class="source-line-no">203</span><span id="line-203"> + " will retry forever to roll the writer if it is not closed");</span> |
| <span class="source-line-no">204</span><span id="line-204"> }</span> |
| <span class="source-line-no">205</span><span id="line-205"> FINISH = null;</span> |
| <span class="source-line-no">206</span><span id="line-206"> // if the reportRegionTransition is finished, wait a bit to let it return the data to RS</span> |
| <span class="source-line-no">207</span><span id="line-207"> Thread.sleep(2000);</span> |
| <span class="source-line-no">208</span><span id="line-208"> ABORT.countDown();</span> |
| <span class="source-line-no">209</span><span id="line-209"></span> |
| <span class="source-line-no">210</span><span id="line-210"> UTIL.waitFor(30000, () -> procExec.isFinished(proc.getProcId()));</span> |
| <span class="source-line-no">211</span><span id="line-211"> UTIL.waitFor(30000, () -> procExec.isFinished(proc.getParentProcId()));</span> |
| <span class="source-line-no">212</span><span id="line-212"> }</span> |
| <span class="source-line-no">213</span><span id="line-213">}</span> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| </pre> |
| </div> |
| </main> |
| </body> |
| </html> |
