| <!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.procedure, class: TestZKProcedureControllers, class: StartControllers"> |
| <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.procedure;</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 static org.junit.Assert.assertArrayEquals;</span> |
| <span class="source-line-no">021</span><span id="line-21">import static org.junit.Assert.assertEquals;</span> |
| <span class="source-line-no">022</span><span id="line-22">import static org.mockito.Mockito.never;</span> |
| <span class="source-line-no">023</span><span id="line-23">import static org.mockito.Mockito.spy;</span> |
| <span class="source-line-no">024</span><span id="line-24">import static org.mockito.Mockito.times;</span> |
| <span class="source-line-no">025</span><span id="line-25">import static org.mockito.Mockito.verify;</span> |
| <span class="source-line-no">026</span><span id="line-26"></span> |
| <span class="source-line-no">027</span><span id="line-27">import java.util.ArrayList;</span> |
| <span class="source-line-no">028</span><span id="line-28">import java.util.List;</span> |
| <span class="source-line-no">029</span><span id="line-29">import java.util.concurrent.CountDownLatch;</span> |
| <span class="source-line-no">030</span><span id="line-30">import org.apache.hadoop.hbase.HBaseClassTestRule;</span> |
| <span class="source-line-no">031</span><span id="line-31">import org.apache.hadoop.hbase.HBaseTestingUtil;</span> |
| <span class="source-line-no">032</span><span id="line-32">import org.apache.hadoop.hbase.errorhandling.ForeignExceptionDispatcher;</span> |
| <span class="source-line-no">033</span><span id="line-33">import org.apache.hadoop.hbase.testclassification.MasterTests;</span> |
| <span class="source-line-no">034</span><span id="line-34">import org.apache.hadoop.hbase.testclassification.MediumTests;</span> |
| <span class="source-line-no">035</span><span id="line-35">import org.apache.hadoop.hbase.util.Bytes;</span> |
| <span class="source-line-no">036</span><span id="line-36">import org.apache.hadoop.hbase.util.Pair;</span> |
| <span class="source-line-no">037</span><span id="line-37">import org.apache.hadoop.hbase.zookeeper.ZKUtil;</span> |
| <span class="source-line-no">038</span><span id="line-38">import org.apache.hadoop.hbase.zookeeper.ZKWatcher;</span> |
| <span class="source-line-no">039</span><span id="line-39">import org.junit.AfterClass;</span> |
| <span class="source-line-no">040</span><span id="line-40">import org.junit.BeforeClass;</span> |
| <span class="source-line-no">041</span><span id="line-41">import org.junit.ClassRule;</span> |
| <span class="source-line-no">042</span><span id="line-42">import org.junit.Test;</span> |
| <span class="source-line-no">043</span><span id="line-43">import org.junit.experimental.categories.Category;</span> |
| <span class="source-line-no">044</span><span id="line-44">import org.mockito.Mockito;</span> |
| <span class="source-line-no">045</span><span id="line-45">import org.mockito.invocation.InvocationOnMock;</span> |
| <span class="source-line-no">046</span><span id="line-46">import org.mockito.stubbing.Answer;</span> |
| <span class="source-line-no">047</span><span id="line-47">import org.mockito.verification.VerificationMode;</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.hbase.thirdparty.com.google.common.collect.Lists;</span> |
| <span class="source-line-no">052</span><span id="line-52"></span> |
| <span class="source-line-no">053</span><span id="line-53">import org.apache.hadoop.hbase.shaded.protobuf.ProtobufUtil;</span> |
| <span class="source-line-no">054</span><span id="line-54"></span> |
| <span class="source-line-no">055</span><span id="line-55">/**</span> |
| <span class="source-line-no">056</span><span id="line-56"> * Test zookeeper-based, procedure controllers</span> |
| <span class="source-line-no">057</span><span id="line-57"> */</span> |
| <span class="source-line-no">058</span><span id="line-58">@Category({ MasterTests.class, MediumTests.class })</span> |
| <span class="source-line-no">059</span><span id="line-59">public class TestZKProcedureControllers {</span> |
| <span class="source-line-no">060</span><span id="line-60"></span> |
| <span class="source-line-no">061</span><span id="line-61"> @ClassRule</span> |
| <span class="source-line-no">062</span><span id="line-62"> public static final HBaseClassTestRule CLASS_RULE =</span> |
| <span class="source-line-no">063</span><span id="line-63"> HBaseClassTestRule.forClass(TestZKProcedureControllers.class);</span> |
| <span class="source-line-no">064</span><span id="line-64"></span> |
| <span class="source-line-no">065</span><span id="line-65"> private static final Logger LOG = LoggerFactory.getLogger(TestZKProcedureControllers.class);</span> |
| <span class="source-line-no">066</span><span id="line-66"> private final static HBaseTestingUtil UTIL = new HBaseTestingUtil();</span> |
| <span class="source-line-no">067</span><span id="line-67"> private static final String COHORT_NODE_NAME = "expected";</span> |
| <span class="source-line-no">068</span><span id="line-68"> private static final String CONTROLLER_NODE_NAME = "controller";</span> |
| <span class="source-line-no">069</span><span id="line-69"> private static final VerificationMode once = Mockito.times(1);</span> |
| <span class="source-line-no">070</span><span id="line-70"></span> |
| <span class="source-line-no">071</span><span id="line-71"> private final byte[] memberData = Bytes.toBytes("data from member");</span> |
| <span class="source-line-no">072</span><span id="line-72"></span> |
| <span class="source-line-no">073</span><span id="line-73"> @BeforeClass</span> |
| <span class="source-line-no">074</span><span id="line-74"> public static void setupTest() throws Exception {</span> |
| <span class="source-line-no">075</span><span id="line-75"> UTIL.startMiniZKCluster();</span> |
| <span class="source-line-no">076</span><span id="line-76"> }</span> |
| <span class="source-line-no">077</span><span id="line-77"></span> |
| <span class="source-line-no">078</span><span id="line-78"> @AfterClass</span> |
| <span class="source-line-no">079</span><span id="line-79"> public static void cleanupTest() throws Exception {</span> |
| <span class="source-line-no">080</span><span id="line-80"> UTIL.shutdownMiniZKCluster();</span> |
| <span class="source-line-no">081</span><span id="line-81"> }</span> |
| <span class="source-line-no">082</span><span id="line-82"></span> |
| <span class="source-line-no">083</span><span id="line-83"> /**</span> |
| <span class="source-line-no">084</span><span id="line-84"> * Smaller test to just test the actuation on the cohort member</span> |
| <span class="source-line-no">085</span><span id="line-85"> * @throws Exception on failure</span> |
| <span class="source-line-no">086</span><span id="line-86"> */</span> |
| <span class="source-line-no">087</span><span id="line-87"> @Test</span> |
| <span class="source-line-no">088</span><span id="line-88"> public void testSimpleZKCohortMemberController() throws Exception {</span> |
| <span class="source-line-no">089</span><span id="line-89"> ZKWatcher watcher = UTIL.getZooKeeperWatcher();</span> |
| <span class="source-line-no">090</span><span id="line-90"> final String operationName = "instanceTest";</span> |
| <span class="source-line-no">091</span><span id="line-91"></span> |
| <span class="source-line-no">092</span><span id="line-92"> final Subprocedure sub = Mockito.mock(Subprocedure.class);</span> |
| <span class="source-line-no">093</span><span id="line-93"> Mockito.when(sub.getName()).thenReturn(operationName);</span> |
| <span class="source-line-no">094</span><span id="line-94"></span> |
| <span class="source-line-no">095</span><span id="line-95"> final byte[] data = new byte[] { 1, 2, 3 };</span> |
| <span class="source-line-no">096</span><span id="line-96"> final CountDownLatch prepared = new CountDownLatch(1);</span> |
| <span class="source-line-no">097</span><span id="line-97"> final CountDownLatch committed = new CountDownLatch(1);</span> |
| <span class="source-line-no">098</span><span id="line-98"></span> |
| <span class="source-line-no">099</span><span id="line-99"> final ForeignExceptionDispatcher monitor = spy(new ForeignExceptionDispatcher());</span> |
| <span class="source-line-no">100</span><span id="line-100"> final ZKProcedureMemberRpcs controller = new ZKProcedureMemberRpcs(watcher, "testSimple");</span> |
| <span class="source-line-no">101</span><span id="line-101"></span> |
| <span class="source-line-no">102</span><span id="line-102"> // mock out cohort member callbacks</span> |
| <span class="source-line-no">103</span><span id="line-103"> final ProcedureMember member = Mockito.mock(ProcedureMember.class);</span> |
| <span class="source-line-no">104</span><span id="line-104"> Mockito.doReturn(sub).when(member).createSubprocedure(operationName, data);</span> |
| <span class="source-line-no">105</span><span id="line-105"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">106</span><span id="line-106"> @Override</span> |
| <span class="source-line-no">107</span><span id="line-107"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">108</span><span id="line-108"> controller.sendMemberAcquired(sub);</span> |
| <span class="source-line-no">109</span><span id="line-109"> prepared.countDown();</span> |
| <span class="source-line-no">110</span><span id="line-110"> return null;</span> |
| <span class="source-line-no">111</span><span id="line-111"> }</span> |
| <span class="source-line-no">112</span><span id="line-112"> }).when(member).submitSubprocedure(sub);</span> |
| <span class="source-line-no">113</span><span id="line-113"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">114</span><span id="line-114"> @Override</span> |
| <span class="source-line-no">115</span><span id="line-115"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">116</span><span id="line-116"> controller.sendMemberCompleted(sub, memberData);</span> |
| <span class="source-line-no">117</span><span id="line-117"> committed.countDown();</span> |
| <span class="source-line-no">118</span><span id="line-118"> return null;</span> |
| <span class="source-line-no">119</span><span id="line-119"> }</span> |
| <span class="source-line-no">120</span><span id="line-120"> }).when(member).receivedReachedGlobalBarrier(operationName);</span> |
| <span class="source-line-no">121</span><span id="line-121"></span> |
| <span class="source-line-no">122</span><span id="line-122"> // start running the listener</span> |
| <span class="source-line-no">123</span><span id="line-123"> controller.start(COHORT_NODE_NAME, member);</span> |
| <span class="source-line-no">124</span><span id="line-124"></span> |
| <span class="source-line-no">125</span><span id="line-125"> // set a prepare node from a 'coordinator'</span> |
| <span class="source-line-no">126</span><span id="line-126"> String prepare =</span> |
| <span class="source-line-no">127</span><span id="line-127"> ZKProcedureUtil.getAcquireBarrierNode(controller.getZkController(), operationName);</span> |
| <span class="source-line-no">128</span><span id="line-128"> ZKUtil.createSetData(watcher, prepare, ProtobufUtil.prependPBMagic(data));</span> |
| <span class="source-line-no">129</span><span id="line-129"> // wait for the operation to be prepared</span> |
| <span class="source-line-no">130</span><span id="line-130"> prepared.await();</span> |
| <span class="source-line-no">131</span><span id="line-131"></span> |
| <span class="source-line-no">132</span><span id="line-132"> // create the commit node so we update the operation to enter the commit phase</span> |
| <span class="source-line-no">133</span><span id="line-133"> String commit =</span> |
| <span class="source-line-no">134</span><span id="line-134"> ZKProcedureUtil.getReachedBarrierNode(controller.getZkController(), operationName);</span> |
| <span class="source-line-no">135</span><span id="line-135"> LOG.debug("Found prepared, posting commit node:" + commit);</span> |
| <span class="source-line-no">136</span><span id="line-136"> ZKUtil.createAndFailSilent(watcher, commit);</span> |
| <span class="source-line-no">137</span><span id="line-137"> LOG.debug("Commit node:" + commit + ", exists:" + ZKUtil.checkExists(watcher, commit));</span> |
| <span class="source-line-no">138</span><span id="line-138"> committed.await();</span> |
| <span class="source-line-no">139</span><span id="line-139"></span> |
| <span class="source-line-no">140</span><span id="line-140"> verify(monitor, never()).receive(Mockito.any());</span> |
| <span class="source-line-no">141</span><span id="line-141"> // XXX: broken due to composition.</span> |
| <span class="source-line-no">142</span><span id="line-142"> // verify(member, never()).getManager().controllerConnectionFailure(Mockito.anyString(),</span> |
| <span class="source-line-no">143</span><span id="line-143"> // Mockito.any());</span> |
| <span class="source-line-no">144</span><span id="line-144"> // cleanup after the test</span> |
| <span class="source-line-no">145</span><span id="line-145"> ZKUtil.deleteNodeRecursively(watcher, controller.getZkController().getBaseZnode());</span> |
| <span class="source-line-no">146</span><span id="line-146"> assertEquals("Didn't delete prepare node", -1, ZKUtil.checkExists(watcher, prepare));</span> |
| <span class="source-line-no">147</span><span id="line-147"> assertEquals("Didn't delete commit node", -1, ZKUtil.checkExists(watcher, commit));</span> |
| <span class="source-line-no">148</span><span id="line-148"> }</span> |
| <span class="source-line-no">149</span><span id="line-149"></span> |
| <span class="source-line-no">150</span><span id="line-150"> @Test</span> |
| <span class="source-line-no">151</span><span id="line-151"> public void testZKCoordinatorControllerWithNoCohort() throws Exception {</span> |
| <span class="source-line-no">152</span><span id="line-152"> final String operationName = "no cohort controller test";</span> |
| <span class="source-line-no">153</span><span id="line-153"> final byte[] data = new byte[] { 1, 2, 3 };</span> |
| <span class="source-line-no">154</span><span id="line-154"></span> |
| <span class="source-line-no">155</span><span id="line-155"> runMockCommitWithOrchestratedControllers(startCoordinatorFirst, operationName, data);</span> |
| <span class="source-line-no">156</span><span id="line-156"> runMockCommitWithOrchestratedControllers(startCohortFirst, operationName, data);</span> |
| <span class="source-line-no">157</span><span id="line-157"> }</span> |
| <span class="source-line-no">158</span><span id="line-158"></span> |
| <span class="source-line-no">159</span><span id="line-159"> @Test</span> |
| <span class="source-line-no">160</span><span id="line-160"> public void testZKCoordinatorControllerWithSingleMemberCohort() throws Exception {</span> |
| <span class="source-line-no">161</span><span id="line-161"> final String operationName = "single member controller test";</span> |
| <span class="source-line-no">162</span><span id="line-162"> final byte[] data = new byte[] { 1, 2, 3 };</span> |
| <span class="source-line-no">163</span><span id="line-163"></span> |
| <span class="source-line-no">164</span><span id="line-164"> runMockCommitWithOrchestratedControllers(startCoordinatorFirst, operationName, data, "cohort");</span> |
| <span class="source-line-no">165</span><span id="line-165"> runMockCommitWithOrchestratedControllers(startCohortFirst, operationName, data, "cohort");</span> |
| <span class="source-line-no">166</span><span id="line-166"> }</span> |
| <span class="source-line-no">167</span><span id="line-167"></span> |
| <span class="source-line-no">168</span><span id="line-168"> @Test</span> |
| <span class="source-line-no">169</span><span id="line-169"> public void testZKCoordinatorControllerMultipleCohort() throws Exception {</span> |
| <span class="source-line-no">170</span><span id="line-170"> final String operationName = "multi member controller test";</span> |
| <span class="source-line-no">171</span><span id="line-171"> final byte[] data = new byte[] { 1, 2, 3 };</span> |
| <span class="source-line-no">172</span><span id="line-172"></span> |
| <span class="source-line-no">173</span><span id="line-173"> runMockCommitWithOrchestratedControllers(startCoordinatorFirst, operationName, data, "cohort",</span> |
| <span class="source-line-no">174</span><span id="line-174"> "cohort2", "cohort3");</span> |
| <span class="source-line-no">175</span><span id="line-175"> runMockCommitWithOrchestratedControllers(startCohortFirst, operationName, data, "cohort",</span> |
| <span class="source-line-no">176</span><span id="line-176"> "cohort2", "cohort3");</span> |
| <span class="source-line-no">177</span><span id="line-177"> }</span> |
| <span class="source-line-no">178</span><span id="line-178"></span> |
| <span class="source-line-no">179</span><span id="line-179"> private void runMockCommitWithOrchestratedControllers(StartControllers controllers,</span> |
| <span class="source-line-no">180</span><span id="line-180"> String operationName, byte[] data, String... cohort) throws Exception {</span> |
| <span class="source-line-no">181</span><span id="line-181"> ZKWatcher watcher = UTIL.getZooKeeperWatcher();</span> |
| <span class="source-line-no">182</span><span id="line-182"> List<String> expected = Lists.newArrayList(cohort);</span> |
| <span class="source-line-no">183</span><span id="line-183"></span> |
| <span class="source-line-no">184</span><span id="line-184"> final Subprocedure sub = Mockito.mock(Subprocedure.class);</span> |
| <span class="source-line-no">185</span><span id="line-185"> Mockito.when(sub.getName()).thenReturn(operationName);</span> |
| <span class="source-line-no">186</span><span id="line-186"></span> |
| <span class="source-line-no">187</span><span id="line-187"> CountDownLatch prepared = new CountDownLatch(expected.size());</span> |
| <span class="source-line-no">188</span><span id="line-188"> CountDownLatch committed = new CountDownLatch(expected.size());</span> |
| <span class="source-line-no">189</span><span id="line-189"> ArrayList<byte[]> dataFromMembers = new ArrayList<>();</span> |
| <span class="source-line-no">190</span><span id="line-190"></span> |
| <span class="source-line-no">191</span><span id="line-191"> // mock out coordinator so we can keep track of zk progress</span> |
| <span class="source-line-no">192</span><span id="line-192"> ProcedureCoordinator coordinator =</span> |
| <span class="source-line-no">193</span><span id="line-193"> setupMockCoordinator(operationName, prepared, committed, dataFromMembers);</span> |
| <span class="source-line-no">194</span><span id="line-194"></span> |
| <span class="source-line-no">195</span><span id="line-195"> ProcedureMember member = Mockito.mock(ProcedureMember.class);</span> |
| <span class="source-line-no">196</span><span id="line-196"></span> |
| <span class="source-line-no">197</span><span id="line-197"> Pair<ZKProcedureCoordinator, List<ZKProcedureMemberRpcs>> pair = controllers.start(watcher,</span> |
| <span class="source-line-no">198</span><span id="line-198"> operationName, coordinator, CONTROLLER_NODE_NAME, member, expected);</span> |
| <span class="source-line-no">199</span><span id="line-199"> ZKProcedureCoordinator controller = pair.getFirst();</span> |
| <span class="source-line-no">200</span><span id="line-200"> List<ZKProcedureMemberRpcs> cohortControllers = pair.getSecond();</span> |
| <span class="source-line-no">201</span><span id="line-201"> // start the operation</span> |
| <span class="source-line-no">202</span><span id="line-202"> Procedure p = Mockito.mock(Procedure.class);</span> |
| <span class="source-line-no">203</span><span id="line-203"> Mockito.when(p.getName()).thenReturn(operationName);</span> |
| <span class="source-line-no">204</span><span id="line-204"></span> |
| <span class="source-line-no">205</span><span id="line-205"> controller.sendGlobalBarrierAcquire(p, data, expected);</span> |
| <span class="source-line-no">206</span><span id="line-206"></span> |
| <span class="source-line-no">207</span><span id="line-207"> // post the prepare node for each expected node</span> |
| <span class="source-line-no">208</span><span id="line-208"> for (ZKProcedureMemberRpcs cc : cohortControllers) {</span> |
| <span class="source-line-no">209</span><span id="line-209"> cc.sendMemberAcquired(sub);</span> |
| <span class="source-line-no">210</span><span id="line-210"> }</span> |
| <span class="source-line-no">211</span><span id="line-211"></span> |
| <span class="source-line-no">212</span><span id="line-212"> // wait for all the notifications to reach the coordinator</span> |
| <span class="source-line-no">213</span><span id="line-213"> prepared.await();</span> |
| <span class="source-line-no">214</span><span id="line-214"> // make sure we got the all the nodes and no more</span> |
| <span class="source-line-no">215</span><span id="line-215"> Mockito.verify(coordinator, times(expected.size()))</span> |
| <span class="source-line-no">216</span><span id="line-216"> .memberAcquiredBarrier(Mockito.eq(operationName), Mockito.anyString());</span> |
| <span class="source-line-no">217</span><span id="line-217"></span> |
| <span class="source-line-no">218</span><span id="line-218"> // kick off the commit phase</span> |
| <span class="source-line-no">219</span><span id="line-219"> controller.sendGlobalBarrierReached(p, expected);</span> |
| <span class="source-line-no">220</span><span id="line-220"></span> |
| <span class="source-line-no">221</span><span id="line-221"> // post the committed node for each expected node</span> |
| <span class="source-line-no">222</span><span id="line-222"> for (ZKProcedureMemberRpcs cc : cohortControllers) {</span> |
| <span class="source-line-no">223</span><span id="line-223"> cc.sendMemberCompleted(sub, memberData);</span> |
| <span class="source-line-no">224</span><span id="line-224"> }</span> |
| <span class="source-line-no">225</span><span id="line-225"></span> |
| <span class="source-line-no">226</span><span id="line-226"> // wait for all commit notifications to reach the coordinator</span> |
| <span class="source-line-no">227</span><span id="line-227"> committed.await();</span> |
| <span class="source-line-no">228</span><span id="line-228"> // make sure we got the all the nodes and no more</span> |
| <span class="source-line-no">229</span><span id="line-229"> Mockito.verify(coordinator, times(expected.size())).memberFinishedBarrier(</span> |
| <span class="source-line-no">230</span><span id="line-230"> Mockito.eq(operationName), Mockito.anyString(), Mockito.eq(memberData));</span> |
| <span class="source-line-no">231</span><span id="line-231"></span> |
| <span class="source-line-no">232</span><span id="line-232"> assertEquals("Incorrect number of members returnd data", expected.size(),</span> |
| <span class="source-line-no">233</span><span id="line-233"> dataFromMembers.size());</span> |
| <span class="source-line-no">234</span><span id="line-234"> for (byte[] result : dataFromMembers) {</span> |
| <span class="source-line-no">235</span><span id="line-235"> assertArrayEquals("Incorrect data from member", memberData, result);</span> |
| <span class="source-line-no">236</span><span id="line-236"> }</span> |
| <span class="source-line-no">237</span><span id="line-237"></span> |
| <span class="source-line-no">238</span><span id="line-238"> controller.resetMembers(p);</span> |
| <span class="source-line-no">239</span><span id="line-239"></span> |
| <span class="source-line-no">240</span><span id="line-240"> // verify all behavior</span> |
| <span class="source-line-no">241</span><span id="line-241"> verifyZooKeeperClean(operationName, watcher, controller.getZkProcedureUtil());</span> |
| <span class="source-line-no">242</span><span id="line-242"> verifyCohort(member, cohortControllers.size(), operationName, data);</span> |
| <span class="source-line-no">243</span><span id="line-243"> verifyCoordinator(operationName, coordinator, expected);</span> |
| <span class="source-line-no">244</span><span id="line-244"> }</span> |
| <span class="source-line-no">245</span><span id="line-245"></span> |
| <span class="source-line-no">246</span><span id="line-246"> // TODO Broken by composition.</span> |
| <span class="source-line-no">247</span><span id="line-247"> // @Test</span> |
| <span class="source-line-no">248</span><span id="line-248"> // public void testCoordinatorControllerHandlesEarlyPrepareNodes() throws Exception {</span> |
| <span class="source-line-no">249</span><span id="line-249"> // runEarlyPrepareNodes(startCoordinatorFirst, "testEarlyPreparenodes", new byte[] { 1, 2, 3 },</span> |
| <span class="source-line-no">250</span><span id="line-250"> // "cohort1", "cohort2");</span> |
| <span class="source-line-no">251</span><span id="line-251"> // runEarlyPrepareNodes(startCohortFirst, "testEarlyPreparenodes", new byte[] { 1, 2, 3 },</span> |
| <span class="source-line-no">252</span><span id="line-252"> // "cohort1", "cohort2");</span> |
| <span class="source-line-no">253</span><span id="line-253"> // }</span> |
| <span class="source-line-no">254</span><span id="line-254"></span> |
| <span class="source-line-no">255</span><span id="line-255"> public void runEarlyPrepareNodes(StartControllers controllers, String operationName, byte[] data,</span> |
| <span class="source-line-no">256</span><span id="line-256"> String... cohort) throws Exception {</span> |
| <span class="source-line-no">257</span><span id="line-257"> ZKWatcher watcher = UTIL.getZooKeeperWatcher();</span> |
| <span class="source-line-no">258</span><span id="line-258"> List<String> expected = Lists.newArrayList(cohort);</span> |
| <span class="source-line-no">259</span><span id="line-259"></span> |
| <span class="source-line-no">260</span><span id="line-260"> final Subprocedure sub = Mockito.mock(Subprocedure.class);</span> |
| <span class="source-line-no">261</span><span id="line-261"> Mockito.when(sub.getName()).thenReturn(operationName);</span> |
| <span class="source-line-no">262</span><span id="line-262"></span> |
| <span class="source-line-no">263</span><span id="line-263"> final CountDownLatch prepared = new CountDownLatch(expected.size());</span> |
| <span class="source-line-no">264</span><span id="line-264"> final CountDownLatch committed = new CountDownLatch(expected.size());</span> |
| <span class="source-line-no">265</span><span id="line-265"> ArrayList<byte[]> dataFromMembers = new ArrayList<>();</span> |
| <span class="source-line-no">266</span><span id="line-266"></span> |
| <span class="source-line-no">267</span><span id="line-267"> // mock out coordinator so we can keep track of zk progress</span> |
| <span class="source-line-no">268</span><span id="line-268"> ProcedureCoordinator coordinator =</span> |
| <span class="source-line-no">269</span><span id="line-269"> setupMockCoordinator(operationName, prepared, committed, dataFromMembers);</span> |
| <span class="source-line-no">270</span><span id="line-270"></span> |
| <span class="source-line-no">271</span><span id="line-271"> ProcedureMember member = Mockito.mock(ProcedureMember.class);</span> |
| <span class="source-line-no">272</span><span id="line-272"> Procedure p = Mockito.mock(Procedure.class);</span> |
| <span class="source-line-no">273</span><span id="line-273"> Mockito.when(p.getName()).thenReturn(operationName);</span> |
| <span class="source-line-no">274</span><span id="line-274"></span> |
| <span class="source-line-no">275</span><span id="line-275"> Pair<ZKProcedureCoordinator, List<ZKProcedureMemberRpcs>> pair = controllers.start(watcher,</span> |
| <span class="source-line-no">276</span><span id="line-276"> operationName, coordinator, CONTROLLER_NODE_NAME, member, expected);</span> |
| <span class="source-line-no">277</span><span id="line-277"> ZKProcedureCoordinator controller = pair.getFirst();</span> |
| <span class="source-line-no">278</span><span id="line-278"> List<ZKProcedureMemberRpcs> cohortControllers = pair.getSecond();</span> |
| <span class="source-line-no">279</span><span id="line-279"></span> |
| <span class="source-line-no">280</span><span id="line-280"> // post 1/2 the prepare nodes early</span> |
| <span class="source-line-no">281</span><span id="line-281"> for (int i = 0; i < cohortControllers.size() / 2; i++) {</span> |
| <span class="source-line-no">282</span><span id="line-282"> cohortControllers.get(i).sendMemberAcquired(sub);</span> |
| <span class="source-line-no">283</span><span id="line-283"> }</span> |
| <span class="source-line-no">284</span><span id="line-284"></span> |
| <span class="source-line-no">285</span><span id="line-285"> // start the operation</span> |
| <span class="source-line-no">286</span><span id="line-286"> controller.sendGlobalBarrierAcquire(p, data, expected);</span> |
| <span class="source-line-no">287</span><span id="line-287"></span> |
| <span class="source-line-no">288</span><span id="line-288"> // post the prepare node for each expected node</span> |
| <span class="source-line-no">289</span><span id="line-289"> for (ZKProcedureMemberRpcs cc : cohortControllers) {</span> |
| <span class="source-line-no">290</span><span id="line-290"> cc.sendMemberAcquired(sub);</span> |
| <span class="source-line-no">291</span><span id="line-291"> }</span> |
| <span class="source-line-no">292</span><span id="line-292"></span> |
| <span class="source-line-no">293</span><span id="line-293"> // wait for all the notifications to reach the coordinator</span> |
| <span class="source-line-no">294</span><span id="line-294"> prepared.await();</span> |
| <span class="source-line-no">295</span><span id="line-295"> // make sure we got the all the nodes and no more</span> |
| <span class="source-line-no">296</span><span id="line-296"> Mockito.verify(coordinator, times(expected.size()))</span> |
| <span class="source-line-no">297</span><span id="line-297"> .memberAcquiredBarrier(Mockito.eq(operationName), Mockito.anyString());</span> |
| <span class="source-line-no">298</span><span id="line-298"></span> |
| <span class="source-line-no">299</span><span id="line-299"> // kick off the commit phase</span> |
| <span class="source-line-no">300</span><span id="line-300"> controller.sendGlobalBarrierReached(p, expected);</span> |
| <span class="source-line-no">301</span><span id="line-301"></span> |
| <span class="source-line-no">302</span><span id="line-302"> // post the committed node for each expected node</span> |
| <span class="source-line-no">303</span><span id="line-303"> for (ZKProcedureMemberRpcs cc : cohortControllers) {</span> |
| <span class="source-line-no">304</span><span id="line-304"> cc.sendMemberCompleted(sub, memberData);</span> |
| <span class="source-line-no">305</span><span id="line-305"> }</span> |
| <span class="source-line-no">306</span><span id="line-306"></span> |
| <span class="source-line-no">307</span><span id="line-307"> // wait for all commit notifications to reach the coordiantor</span> |
| <span class="source-line-no">308</span><span id="line-308"> committed.await();</span> |
| <span class="source-line-no">309</span><span id="line-309"> // make sure we got the all the nodes and no more</span> |
| <span class="source-line-no">310</span><span id="line-310"> Mockito.verify(coordinator, times(expected.size())).memberFinishedBarrier(</span> |
| <span class="source-line-no">311</span><span id="line-311"> Mockito.eq(operationName), Mockito.anyString(), Mockito.eq(memberData));</span> |
| <span class="source-line-no">312</span><span id="line-312"></span> |
| <span class="source-line-no">313</span><span id="line-313"> controller.resetMembers(p);</span> |
| <span class="source-line-no">314</span><span id="line-314"></span> |
| <span class="source-line-no">315</span><span id="line-315"> // verify all behavior</span> |
| <span class="source-line-no">316</span><span id="line-316"> verifyZooKeeperClean(operationName, watcher, controller.getZkProcedureUtil());</span> |
| <span class="source-line-no">317</span><span id="line-317"> verifyCohort(member, cohortControllers.size(), operationName, data);</span> |
| <span class="source-line-no">318</span><span id="line-318"> verifyCoordinator(operationName, coordinator, expected);</span> |
| <span class="source-line-no">319</span><span id="line-319"> }</span> |
| <span class="source-line-no">320</span><span id="line-320"></span> |
| <span class="source-line-no">321</span><span id="line-321"> /**</span> |
| <span class="source-line-no">322</span><span id="line-322"> * @return a mock {@link ProcedureCoordinator} that just counts down the prepared and committed</span> |
| <span class="source-line-no">323</span><span id="line-323"> * latch for called to the respective method</span> |
| <span class="source-line-no">324</span><span id="line-324"> */</span> |
| <span class="source-line-no">325</span><span id="line-325"> private ProcedureCoordinator setupMockCoordinator(String operationName,</span> |
| <span class="source-line-no">326</span><span id="line-326"> final CountDownLatch prepared, final CountDownLatch committed,</span> |
| <span class="source-line-no">327</span><span id="line-327"> final ArrayList<byte[]> dataFromMembers) {</span> |
| <span class="source-line-no">328</span><span id="line-328"> ProcedureCoordinator coordinator = Mockito.mock(ProcedureCoordinator.class);</span> |
| <span class="source-line-no">329</span><span id="line-329"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">330</span><span id="line-330"> @Override</span> |
| <span class="source-line-no">331</span><span id="line-331"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">332</span><span id="line-332"> prepared.countDown();</span> |
| <span class="source-line-no">333</span><span id="line-333"> return null;</span> |
| <span class="source-line-no">334</span><span id="line-334"> }</span> |
| <span class="source-line-no">335</span><span id="line-335"> }).when(coordinator).memberAcquiredBarrier(Mockito.eq(operationName), Mockito.anyString());</span> |
| <span class="source-line-no">336</span><span id="line-336"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">337</span><span id="line-337"> @Override</span> |
| <span class="source-line-no">338</span><span id="line-338"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">339</span><span id="line-339"> dataFromMembers.add(memberData);</span> |
| <span class="source-line-no">340</span><span id="line-340"> committed.countDown();</span> |
| <span class="source-line-no">341</span><span id="line-341"> return null;</span> |
| <span class="source-line-no">342</span><span id="line-342"> }</span> |
| <span class="source-line-no">343</span><span id="line-343"> }).when(coordinator).memberFinishedBarrier(Mockito.eq(operationName), Mockito.anyString(),</span> |
| <span class="source-line-no">344</span><span id="line-344"> Mockito.eq(memberData));</span> |
| <span class="source-line-no">345</span><span id="line-345"> return coordinator;</span> |
| <span class="source-line-no">346</span><span id="line-346"> }</span> |
| <span class="source-line-no">347</span><span id="line-347"></span> |
| <span class="source-line-no">348</span><span id="line-348"> /**</span> |
| <span class="source-line-no">349</span><span id="line-349"> * Verify that the prepare, commit and abort nodes for the operation are removed from zookeeper</span> |
| <span class="source-line-no">350</span><span id="line-350"> */</span> |
| <span class="source-line-no">351</span><span id="line-351"> private void verifyZooKeeperClean(String operationName, ZKWatcher watcher,</span> |
| <span class="source-line-no">352</span><span id="line-352"> ZKProcedureUtil controller) throws Exception {</span> |
| <span class="source-line-no">353</span><span id="line-353"> String prepare = ZKProcedureUtil.getAcquireBarrierNode(controller, operationName);</span> |
| <span class="source-line-no">354</span><span id="line-354"> String commit = ZKProcedureUtil.getReachedBarrierNode(controller, operationName);</span> |
| <span class="source-line-no">355</span><span id="line-355"> String abort = ZKProcedureUtil.getAbortNode(controller, operationName);</span> |
| <span class="source-line-no">356</span><span id="line-356"> assertEquals("Didn't delete prepare node", -1, ZKUtil.checkExists(watcher, prepare));</span> |
| <span class="source-line-no">357</span><span id="line-357"> assertEquals("Didn't delete commit node", -1, ZKUtil.checkExists(watcher, commit));</span> |
| <span class="source-line-no">358</span><span id="line-358"> assertEquals("Didn't delete abort node", -1, ZKUtil.checkExists(watcher, abort));</span> |
| <span class="source-line-no">359</span><span id="line-359"> }</span> |
| <span class="source-line-no">360</span><span id="line-360"></span> |
| <span class="source-line-no">361</span><span id="line-361"> /**</span> |
| <span class="source-line-no">362</span><span id="line-362"> * Verify the cohort controller got called once per expected node to start the operation</span> |
| <span class="source-line-no">363</span><span id="line-363"> */</span> |
| <span class="source-line-no">364</span><span id="line-364"> private void verifyCohort(ProcedureMember member, int cohortSize, String operationName,</span> |
| <span class="source-line-no">365</span><span id="line-365"> byte[] data) {</span> |
| <span class="source-line-no">366</span><span id="line-366"> // verify(member, Mockito.times(cohortSize)).submitSubprocedure(Mockito.eq(operationName),</span> |
| <span class="source-line-no">367</span><span id="line-367"> // (byte[]) Mockito.argThat(new ArrayEquals(data)));</span> |
| <span class="source-line-no">368</span><span id="line-368"> Mockito.verify(member, Mockito.atLeast(cohortSize)).submitSubprocedure(Mockito.any());</span> |
| <span class="source-line-no">369</span><span id="line-369"></span> |
| <span class="source-line-no">370</span><span id="line-370"> }</span> |
| <span class="source-line-no">371</span><span id="line-371"></span> |
| <span class="source-line-no">372</span><span id="line-372"> /**</span> |
| <span class="source-line-no">373</span><span id="line-373"> * Verify that the coordinator only got called once for each expected node</span> |
| <span class="source-line-no">374</span><span id="line-374"> */</span> |
| <span class="source-line-no">375</span><span id="line-375"> private void verifyCoordinator(String operationName, ProcedureCoordinator coordinator,</span> |
| <span class="source-line-no">376</span><span id="line-376"> List<String> expected) {</span> |
| <span class="source-line-no">377</span><span id="line-377"> // verify that we got all the expected nodes</span> |
| <span class="source-line-no">378</span><span id="line-378"> for (String node : expected) {</span> |
| <span class="source-line-no">379</span><span id="line-379"> verify(coordinator, once).memberAcquiredBarrier(operationName, node);</span> |
| <span class="source-line-no">380</span><span id="line-380"> verify(coordinator, once).memberFinishedBarrier(operationName, node, memberData);</span> |
| <span class="source-line-no">381</span><span id="line-381"> }</span> |
| <span class="source-line-no">382</span><span id="line-382"> }</span> |
| <span class="source-line-no">383</span><span id="line-383"></span> |
| <span class="source-line-no">384</span><span id="line-384"> /**</span> |
| <span class="source-line-no">385</span><span id="line-385"> * Specify how the controllers that should be started (not spy/mockable) for the test.</span> |
| <span class="source-line-no">386</span><span id="line-386"> */</span> |
| <span class="source-line-no">387</span><span id="line-387"> private abstract class StartControllers {</span> |
| <span class="source-line-no">388</span><span id="line-388"> public abstract Pair<ZKProcedureCoordinator, List<ZKProcedureMemberRpcs>> start(</span> |
| <span class="source-line-no">389</span><span id="line-389"> ZKWatcher watcher, String operationName, ProcedureCoordinator coordinator,</span> |
| <span class="source-line-no">390</span><span id="line-390"> String controllerName, ProcedureMember member, List<String> cohortNames) throws Exception;</span> |
| <span class="source-line-no">391</span><span id="line-391"> }</span> |
| <span class="source-line-no">392</span><span id="line-392"></span> |
| <span class="source-line-no">393</span><span id="line-393"> private final StartControllers startCoordinatorFirst = new StartControllers() {</span> |
| <span class="source-line-no">394</span><span id="line-394"></span> |
| <span class="source-line-no">395</span><span id="line-395"> @Override</span> |
| <span class="source-line-no">396</span><span id="line-396"> public Pair<ZKProcedureCoordinator, List<ZKProcedureMemberRpcs>> start(ZKWatcher watcher,</span> |
| <span class="source-line-no">397</span><span id="line-397"> String operationName, ProcedureCoordinator coordinator, String controllerName,</span> |
| <span class="source-line-no">398</span><span id="line-398"> ProcedureMember member, List<String> expected) throws Exception {</span> |
| <span class="source-line-no">399</span><span id="line-399"> // start the controller</span> |
| <span class="source-line-no">400</span><span id="line-400"> ZKProcedureCoordinator controller =</span> |
| <span class="source-line-no">401</span><span id="line-401"> new ZKProcedureCoordinator(watcher, operationName, CONTROLLER_NODE_NAME);</span> |
| <span class="source-line-no">402</span><span id="line-402"> controller.start(coordinator);</span> |
| <span class="source-line-no">403</span><span id="line-403"></span> |
| <span class="source-line-no">404</span><span id="line-404"> // make a cohort controller for each expected node</span> |
| <span class="source-line-no">405</span><span id="line-405"></span> |
| <span class="source-line-no">406</span><span id="line-406"> List<ZKProcedureMemberRpcs> cohortControllers = new ArrayList<>();</span> |
| <span class="source-line-no">407</span><span id="line-407"> for (String nodeName : expected) {</span> |
| <span class="source-line-no">408</span><span id="line-408"> ZKProcedureMemberRpcs cc = new ZKProcedureMemberRpcs(watcher, operationName);</span> |
| <span class="source-line-no">409</span><span id="line-409"> cc.start(nodeName, member);</span> |
| <span class="source-line-no">410</span><span id="line-410"> cohortControllers.add(cc);</span> |
| <span class="source-line-no">411</span><span id="line-411"> }</span> |
| <span class="source-line-no">412</span><span id="line-412"> return new Pair<>(controller, cohortControllers);</span> |
| <span class="source-line-no">413</span><span id="line-413"> }</span> |
| <span class="source-line-no">414</span><span id="line-414"> };</span> |
| <span class="source-line-no">415</span><span id="line-415"></span> |
| <span class="source-line-no">416</span><span id="line-416"> /**</span> |
| <span class="source-line-no">417</span><span id="line-417"> * Check for the possible race condition where a cohort member starts after the controller and</span> |
| <span class="source-line-no">418</span><span id="line-418"> * therefore could miss a new operation</span> |
| <span class="source-line-no">419</span><span id="line-419"> */</span> |
| <span class="source-line-no">420</span><span id="line-420"> private final StartControllers startCohortFirst = new StartControllers() {</span> |
| <span class="source-line-no">421</span><span id="line-421"></span> |
| <span class="source-line-no">422</span><span id="line-422"> @Override</span> |
| <span class="source-line-no">423</span><span id="line-423"> public Pair<ZKProcedureCoordinator, List<ZKProcedureMemberRpcs>> start(ZKWatcher watcher,</span> |
| <span class="source-line-no">424</span><span id="line-424"> String operationName, ProcedureCoordinator coordinator, String controllerName,</span> |
| <span class="source-line-no">425</span><span id="line-425"> ProcedureMember member, List<String> expected) throws Exception {</span> |
| <span class="source-line-no">426</span><span id="line-426"></span> |
| <span class="source-line-no">427</span><span id="line-427"> // make a cohort controller for each expected node</span> |
| <span class="source-line-no">428</span><span id="line-428"> List<ZKProcedureMemberRpcs> cohortControllers = new ArrayList<>();</span> |
| <span class="source-line-no">429</span><span id="line-429"> for (String nodeName : expected) {</span> |
| <span class="source-line-no">430</span><span id="line-430"> ZKProcedureMemberRpcs cc = new ZKProcedureMemberRpcs(watcher, operationName);</span> |
| <span class="source-line-no">431</span><span id="line-431"> cc.start(nodeName, member);</span> |
| <span class="source-line-no">432</span><span id="line-432"> cohortControllers.add(cc);</span> |
| <span class="source-line-no">433</span><span id="line-433"> }</span> |
| <span class="source-line-no">434</span><span id="line-434"></span> |
| <span class="source-line-no">435</span><span id="line-435"> // start the controller</span> |
| <span class="source-line-no">436</span><span id="line-436"> ZKProcedureCoordinator controller =</span> |
| <span class="source-line-no">437</span><span id="line-437"> new ZKProcedureCoordinator(watcher, operationName, CONTROLLER_NODE_NAME);</span> |
| <span class="source-line-no">438</span><span id="line-438"> controller.start(coordinator);</span> |
| <span class="source-line-no">439</span><span id="line-439"></span> |
| <span class="source-line-no">440</span><span id="line-440"> return new Pair<>(controller, cohortControllers);</span> |
| <span class="source-line-no">441</span><span id="line-441"> }</span> |
| <span class="source-line-no">442</span><span id="line-442"> };</span> |
| <span class="source-line-no">443</span><span id="line-443">}</span> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| </pre> |
| </div> |
| </main> |
| </body> |
| </html> |