| <!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: TestZKProcedure"> |
| <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.assertEquals;</span> |
| <span class="source-line-no">021</span><span id="line-21">import static org.junit.Assert.assertTrue;</span> |
| <span class="source-line-no">022</span><span id="line-22">import static org.mockito.ArgumentMatchers.any;</span> |
| <span class="source-line-no">023</span><span id="line-23">import static org.mockito.ArgumentMatchers.anyList;</span> |
| <span class="source-line-no">024</span><span id="line-24">import static org.mockito.ArgumentMatchers.eq;</span> |
| <span class="source-line-no">025</span><span id="line-25">import static org.mockito.Mockito.atMost;</span> |
| <span class="source-line-no">026</span><span id="line-26">import static org.mockito.Mockito.never;</span> |
| <span class="source-line-no">027</span><span id="line-27">import static org.mockito.Mockito.spy;</span> |
| <span class="source-line-no">028</span><span id="line-28">import static org.mockito.Mockito.when;</span> |
| <span class="source-line-no">029</span><span id="line-29"></span> |
| <span class="source-line-no">030</span><span id="line-30">import java.io.IOException;</span> |
| <span class="source-line-no">031</span><span id="line-31">import java.util.ArrayList;</span> |
| <span class="source-line-no">032</span><span id="line-32">import java.util.Arrays;</span> |
| <span class="source-line-no">033</span><span id="line-33">import java.util.List;</span> |
| <span class="source-line-no">034</span><span id="line-34">import java.util.concurrent.CountDownLatch;</span> |
| <span class="source-line-no">035</span><span id="line-35">import java.util.concurrent.ThreadPoolExecutor;</span> |
| <span class="source-line-no">036</span><span id="line-36">import java.util.concurrent.atomic.AtomicInteger;</span> |
| <span class="source-line-no">037</span><span id="line-37">import org.apache.hadoop.hbase.Abortable;</span> |
| <span class="source-line-no">038</span><span id="line-38">import org.apache.hadoop.hbase.HBaseClassTestRule;</span> |
| <span class="source-line-no">039</span><span id="line-39">import org.apache.hadoop.hbase.HBaseTestingUtil;</span> |
| <span class="source-line-no">040</span><span id="line-40">import org.apache.hadoop.hbase.errorhandling.ForeignException;</span> |
| <span class="source-line-no">041</span><span id="line-41">import org.apache.hadoop.hbase.errorhandling.ForeignExceptionDispatcher;</span> |
| <span class="source-line-no">042</span><span id="line-42">import org.apache.hadoop.hbase.errorhandling.TimeoutException;</span> |
| <span class="source-line-no">043</span><span id="line-43">import org.apache.hadoop.hbase.procedure.Subprocedure.SubprocedureImpl;</span> |
| <span class="source-line-no">044</span><span id="line-44">import org.apache.hadoop.hbase.testclassification.MasterTests;</span> |
| <span class="source-line-no">045</span><span id="line-45">import org.apache.hadoop.hbase.testclassification.MediumTests;</span> |
| <span class="source-line-no">046</span><span id="line-46">import org.apache.hadoop.hbase.util.Pair;</span> |
| <span class="source-line-no">047</span><span id="line-47">import org.apache.hadoop.hbase.zookeeper.ZKWatcher;</span> |
| <span class="source-line-no">048</span><span id="line-48">import org.junit.AfterClass;</span> |
| <span class="source-line-no">049</span><span id="line-49">import org.junit.BeforeClass;</span> |
| <span class="source-line-no">050</span><span id="line-50">import org.junit.ClassRule;</span> |
| <span class="source-line-no">051</span><span id="line-51">import org.junit.Test;</span> |
| <span class="source-line-no">052</span><span id="line-52">import org.junit.experimental.categories.Category;</span> |
| <span class="source-line-no">053</span><span id="line-53">import org.mockito.Mockito;</span> |
| <span class="source-line-no">054</span><span id="line-54">import org.mockito.internal.matchers.ArrayEquals;</span> |
| <span class="source-line-no">055</span><span id="line-55">import org.mockito.invocation.InvocationOnMock;</span> |
| <span class="source-line-no">056</span><span id="line-56">import org.mockito.stubbing.Answer;</span> |
| <span class="source-line-no">057</span><span id="line-57">import org.mockito.verification.VerificationMode;</span> |
| <span class="source-line-no">058</span><span id="line-58">import org.slf4j.Logger;</span> |
| <span class="source-line-no">059</span><span id="line-59">import org.slf4j.LoggerFactory;</span> |
| <span class="source-line-no">060</span><span id="line-60"></span> |
| <span class="source-line-no">061</span><span id="line-61">import org.apache.hbase.thirdparty.com.google.common.collect.Lists;</span> |
| <span class="source-line-no">062</span><span id="line-62"></span> |
| <span class="source-line-no">063</span><span id="line-63">/**</span> |
| <span class="source-line-no">064</span><span id="line-64"> * Cluster-wide testing of a distributed three-phase commit using a 'real' zookeeper cluster</span> |
| <span class="source-line-no">065</span><span id="line-65"> */</span> |
| <span class="source-line-no">066</span><span id="line-66">@Category({ MasterTests.class, MediumTests.class })</span> |
| <span class="source-line-no">067</span><span id="line-67">public class TestZKProcedure {</span> |
| <span class="source-line-no">068</span><span id="line-68"></span> |
| <span class="source-line-no">069</span><span id="line-69"> @ClassRule</span> |
| <span class="source-line-no">070</span><span id="line-70"> public static final HBaseClassTestRule CLASS_RULE =</span> |
| <span class="source-line-no">071</span><span id="line-71"> HBaseClassTestRule.forClass(TestZKProcedure.class);</span> |
| <span class="source-line-no">072</span><span id="line-72"></span> |
| <span class="source-line-no">073</span><span id="line-73"> private static final Logger LOG = LoggerFactory.getLogger(TestZKProcedure.class);</span> |
| <span class="source-line-no">074</span><span id="line-74"> private static HBaseTestingUtil UTIL = new HBaseTestingUtil();</span> |
| <span class="source-line-no">075</span><span id="line-75"> private static final String COORDINATOR_NODE_NAME = "coordinator";</span> |
| <span class="source-line-no">076</span><span id="line-76"> private static final long KEEP_ALIVE = 100; // seconds</span> |
| <span class="source-line-no">077</span><span id="line-77"> private static final int POOL_SIZE = 1;</span> |
| <span class="source-line-no">078</span><span id="line-78"> private static final long TIMEOUT = 10000; // when debugging make this larger for debugging</span> |
| <span class="source-line-no">079</span><span id="line-79"> private static final long WAKE_FREQUENCY = 500;</span> |
| <span class="source-line-no">080</span><span id="line-80"> private static final String opName = "op";</span> |
| <span class="source-line-no">081</span><span id="line-81"> private static final byte[] data = new byte[] { 1, 2 }; // TODO what is this used for?</span> |
| <span class="source-line-no">082</span><span id="line-82"> private static final VerificationMode once = Mockito.times(1);</span> |
| <span class="source-line-no">083</span><span id="line-83"></span> |
| <span class="source-line-no">084</span><span id="line-84"> @BeforeClass</span> |
| <span class="source-line-no">085</span><span id="line-85"> public static void setupTest() throws Exception {</span> |
| <span class="source-line-no">086</span><span id="line-86"> UTIL.startMiniZKCluster();</span> |
| <span class="source-line-no">087</span><span id="line-87"> }</span> |
| <span class="source-line-no">088</span><span id="line-88"></span> |
| <span class="source-line-no">089</span><span id="line-89"> @AfterClass</span> |
| <span class="source-line-no">090</span><span id="line-90"> public static void cleanupTest() throws Exception {</span> |
| <span class="source-line-no">091</span><span id="line-91"> UTIL.shutdownMiniZKCluster();</span> |
| <span class="source-line-no">092</span><span id="line-92"> }</span> |
| <span class="source-line-no">093</span><span id="line-93"></span> |
| <span class="source-line-no">094</span><span id="line-94"> private static ZKWatcher newZooKeeperWatcher() throws IOException {</span> |
| <span class="source-line-no">095</span><span id="line-95"> return new ZKWatcher(UTIL.getConfiguration(), "testing utility", new Abortable() {</span> |
| <span class="source-line-no">096</span><span id="line-96"> @Override</span> |
| <span class="source-line-no">097</span><span id="line-97"> public void abort(String why, Throwable e) {</span> |
| <span class="source-line-no">098</span><span id="line-98"> throw new RuntimeException("Unexpected abort in distributed three phase commit test:" + why,</span> |
| <span class="source-line-no">099</span><span id="line-99"> e);</span> |
| <span class="source-line-no">100</span><span id="line-100"> }</span> |
| <span class="source-line-no">101</span><span id="line-101"></span> |
| <span class="source-line-no">102</span><span id="line-102"> @Override</span> |
| <span class="source-line-no">103</span><span id="line-103"> public boolean isAborted() {</span> |
| <span class="source-line-no">104</span><span id="line-104"> return false;</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"> @Test</span> |
| <span class="source-line-no">110</span><span id="line-110"> public void testEmptyMemberSet() throws Exception {</span> |
| <span class="source-line-no">111</span><span id="line-111"> runCommit();</span> |
| <span class="source-line-no">112</span><span id="line-112"> }</span> |
| <span class="source-line-no">113</span><span id="line-113"></span> |
| <span class="source-line-no">114</span><span id="line-114"> @Test</span> |
| <span class="source-line-no">115</span><span id="line-115"> public void testSingleMember() throws Exception {</span> |
| <span class="source-line-no">116</span><span id="line-116"> runCommit("one");</span> |
| <span class="source-line-no">117</span><span id="line-117"> }</span> |
| <span class="source-line-no">118</span><span id="line-118"></span> |
| <span class="source-line-no">119</span><span id="line-119"> @Test</span> |
| <span class="source-line-no">120</span><span id="line-120"> public void testMultipleMembers() throws Exception {</span> |
| <span class="source-line-no">121</span><span id="line-121"> runCommit("one", "two", "three", "four");</span> |
| <span class="source-line-no">122</span><span id="line-122"> }</span> |
| <span class="source-line-no">123</span><span id="line-123"></span> |
| <span class="source-line-no">124</span><span id="line-124"> private void runCommit(String... members) throws Exception {</span> |
| <span class="source-line-no">125</span><span id="line-125"> // make sure we just have an empty list</span> |
| <span class="source-line-no">126</span><span id="line-126"> if (members == null) {</span> |
| <span class="source-line-no">127</span><span id="line-127"> members = new String[0];</span> |
| <span class="source-line-no">128</span><span id="line-128"> }</span> |
| <span class="source-line-no">129</span><span id="line-129"> List<String> expected = Arrays.asList(members);</span> |
| <span class="source-line-no">130</span><span id="line-130"></span> |
| <span class="source-line-no">131</span><span id="line-131"> // setup the constants</span> |
| <span class="source-line-no">132</span><span id="line-132"> ZKWatcher coordZkw = newZooKeeperWatcher();</span> |
| <span class="source-line-no">133</span><span id="line-133"> String opDescription = "coordination test - " + members.length + " cohort members";</span> |
| <span class="source-line-no">134</span><span id="line-134"></span> |
| <span class="source-line-no">135</span><span id="line-135"> // start running the controller</span> |
| <span class="source-line-no">136</span><span id="line-136"> ZKProcedureCoordinator coordinatorComms =</span> |
| <span class="source-line-no">137</span><span id="line-137"> new ZKProcedureCoordinator(coordZkw, opDescription, COORDINATOR_NODE_NAME);</span> |
| <span class="source-line-no">138</span><span id="line-138"> ThreadPoolExecutor pool =</span> |
| <span class="source-line-no">139</span><span id="line-139"> ProcedureCoordinator.defaultPool(COORDINATOR_NODE_NAME, POOL_SIZE, KEEP_ALIVE);</span> |
| <span class="source-line-no">140</span><span id="line-140"> ProcedureCoordinator coordinator = new ProcedureCoordinator(coordinatorComms, pool) {</span> |
| <span class="source-line-no">141</span><span id="line-141"> @Override</span> |
| <span class="source-line-no">142</span><span id="line-142"> public Procedure createProcedure(ForeignExceptionDispatcher fed, String procName,</span> |
| <span class="source-line-no">143</span><span id="line-143"> byte[] procArgs, List<String> expectedMembers) {</span> |
| <span class="source-line-no">144</span><span id="line-144"> return Mockito.spy(super.createProcedure(fed, procName, procArgs, expectedMembers));</span> |
| <span class="source-line-no">145</span><span id="line-145"> }</span> |
| <span class="source-line-no">146</span><span id="line-146"> };</span> |
| <span class="source-line-no">147</span><span id="line-147"></span> |
| <span class="source-line-no">148</span><span id="line-148"> // build and start members</span> |
| <span class="source-line-no">149</span><span id="line-149"> // NOTE: There is a single subprocedure builder for all members here.</span> |
| <span class="source-line-no">150</span><span id="line-150"> SubprocedureFactory subprocFactory = Mockito.mock(SubprocedureFactory.class);</span> |
| <span class="source-line-no">151</span><span id="line-151"> List<Pair<ProcedureMember, ZKProcedureMemberRpcs>> procMembers =</span> |
| <span class="source-line-no">152</span><span id="line-152"> new ArrayList<>(members.length);</span> |
| <span class="source-line-no">153</span><span id="line-153"> // start each member</span> |
| <span class="source-line-no">154</span><span id="line-154"> for (String member : members) {</span> |
| <span class="source-line-no">155</span><span id="line-155"> ZKWatcher watcher = newZooKeeperWatcher();</span> |
| <span class="source-line-no">156</span><span id="line-156"> ZKProcedureMemberRpcs comms = new ZKProcedureMemberRpcs(watcher, opDescription);</span> |
| <span class="source-line-no">157</span><span id="line-157"> ThreadPoolExecutor pool2 = ProcedureMember.defaultPool(member, 1, KEEP_ALIVE);</span> |
| <span class="source-line-no">158</span><span id="line-158"> ProcedureMember procMember = new ProcedureMember(comms, pool2, subprocFactory);</span> |
| <span class="source-line-no">159</span><span id="line-159"> procMembers.add(new Pair<>(procMember, comms));</span> |
| <span class="source-line-no">160</span><span id="line-160"> comms.start(member, procMember);</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"> // setup mock member subprocedures</span> |
| <span class="source-line-no">164</span><span id="line-164"> final List<Subprocedure> subprocs = new ArrayList<>();</span> |
| <span class="source-line-no">165</span><span id="line-165"> for (int i = 0; i < procMembers.size(); i++) {</span> |
| <span class="source-line-no">166</span><span id="line-166"> ForeignExceptionDispatcher cohortMonitor = new ForeignExceptionDispatcher();</span> |
| <span class="source-line-no">167</span><span id="line-167"> Subprocedure commit = Mockito.spy(new SubprocedureImpl(procMembers.get(i).getFirst(), opName,</span> |
| <span class="source-line-no">168</span><span id="line-168"> cohortMonitor, WAKE_FREQUENCY, TIMEOUT));</span> |
| <span class="source-line-no">169</span><span id="line-169"> subprocs.add(commit);</span> |
| <span class="source-line-no">170</span><span id="line-170"> }</span> |
| <span class="source-line-no">171</span><span id="line-171"></span> |
| <span class="source-line-no">172</span><span id="line-172"> // link subprocedure to buildNewOperation invocation.</span> |
| <span class="source-line-no">173</span><span id="line-173"> final AtomicInteger i = new AtomicInteger(0); // NOTE: would be racy if not an AtomicInteger</span> |
| <span class="source-line-no">174</span><span id="line-174"> Mockito.when(subprocFactory.buildSubprocedure(Mockito.eq(opName),</span> |
| <span class="source-line-no">175</span><span id="line-175"> (byte[]) Mockito.argThat(new ArrayEquals(data)))).thenAnswer(new Answer<Subprocedure>() {</span> |
| <span class="source-line-no">176</span><span id="line-176"> @Override</span> |
| <span class="source-line-no">177</span><span id="line-177"> public Subprocedure answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">178</span><span id="line-178"> int index = i.getAndIncrement();</span> |
| <span class="source-line-no">179</span><span id="line-179"> LOG.debug("Task size:" + subprocs.size() + ", getting:" + index);</span> |
| <span class="source-line-no">180</span><span id="line-180"> Subprocedure commit = subprocs.get(index);</span> |
| <span class="source-line-no">181</span><span id="line-181"> return commit;</span> |
| <span class="source-line-no">182</span><span id="line-182"> }</span> |
| <span class="source-line-no">183</span><span id="line-183"> });</span> |
| <span class="source-line-no">184</span><span id="line-184"></span> |
| <span class="source-line-no">185</span><span id="line-185"> // setup spying on the coordinator</span> |
| <span class="source-line-no">186</span><span id="line-186"> // Procedure proc = Mockito.spy(procBuilder.createProcedure(coordinator, opName, data,</span> |
| <span class="source-line-no">187</span><span id="line-187"> // expected));</span> |
| <span class="source-line-no">188</span><span id="line-188"> // Mockito.when(procBuilder.build(coordinator, opName, data, expected)).thenReturn(proc);</span> |
| <span class="source-line-no">189</span><span id="line-189"></span> |
| <span class="source-line-no">190</span><span id="line-190"> // start running the operation</span> |
| <span class="source-line-no">191</span><span id="line-191"> Procedure task =</span> |
| <span class="source-line-no">192</span><span id="line-192"> coordinator.startProcedure(new ForeignExceptionDispatcher(), opName, data, expected);</span> |
| <span class="source-line-no">193</span><span id="line-193"> // assertEquals("Didn't mock coordinator task", proc, task);</span> |
| <span class="source-line-no">194</span><span id="line-194"></span> |
| <span class="source-line-no">195</span><span id="line-195"> // verify all things ran as expected</span> |
| <span class="source-line-no">196</span><span id="line-196"> // waitAndVerifyProc(proc, once, once, never(), once, false);</span> |
| <span class="source-line-no">197</span><span id="line-197"> waitAndVerifyProc(task, once, once, never(), once, false);</span> |
| <span class="source-line-no">198</span><span id="line-198"> verifyCohortSuccessful(expected, subprocFactory, subprocs, once, once, never(), once, false);</span> |
| <span class="source-line-no">199</span><span id="line-199"></span> |
| <span class="source-line-no">200</span><span id="line-200"> // close all the things</span> |
| <span class="source-line-no">201</span><span id="line-201"> closeAll(coordinator, coordinatorComms, procMembers);</span> |
| <span class="source-line-no">202</span><span id="line-202"> }</span> |
| <span class="source-line-no">203</span><span id="line-203"></span> |
| <span class="source-line-no">204</span><span id="line-204"> /**</span> |
| <span class="source-line-no">205</span><span id="line-205"> * Test a distributed commit with multiple cohort members, where one of the cohort members has a</span> |
| <span class="source-line-no">206</span><span id="line-206"> * timeout exception during the prepare stage.</span> |
| <span class="source-line-no">207</span><span id="line-207"> */</span> |
| <span class="source-line-no">208</span><span id="line-208"> @Test</span> |
| <span class="source-line-no">209</span><span id="line-209"> public void testMultiCohortWithMemberTimeoutDuringPrepare() throws Exception {</span> |
| <span class="source-line-no">210</span><span id="line-210"> String opDescription = "error injection coordination";</span> |
| <span class="source-line-no">211</span><span id="line-211"> String[] cohortMembers = new String[] { "one", "two", "three" };</span> |
| <span class="source-line-no">212</span><span id="line-212"> List<String> expected = Lists.newArrayList(cohortMembers);</span> |
| <span class="source-line-no">213</span><span id="line-213"> // error constants</span> |
| <span class="source-line-no">214</span><span id="line-214"> final int memberErrorIndex = 2;</span> |
| <span class="source-line-no">215</span><span id="line-215"> final CountDownLatch coordinatorReceivedErrorLatch = new CountDownLatch(1);</span> |
| <span class="source-line-no">216</span><span id="line-216"></span> |
| <span class="source-line-no">217</span><span id="line-217"> // start running the coordinator and its controller</span> |
| <span class="source-line-no">218</span><span id="line-218"> ZKWatcher coordinatorWatcher = newZooKeeperWatcher();</span> |
| <span class="source-line-no">219</span><span id="line-219"> ZKProcedureCoordinator coordinatorController =</span> |
| <span class="source-line-no">220</span><span id="line-220"> new ZKProcedureCoordinator(coordinatorWatcher, opDescription, COORDINATOR_NODE_NAME);</span> |
| <span class="source-line-no">221</span><span id="line-221"> ThreadPoolExecutor pool =</span> |
| <span class="source-line-no">222</span><span id="line-222"> ProcedureCoordinator.defaultPool(COORDINATOR_NODE_NAME, POOL_SIZE, KEEP_ALIVE);</span> |
| <span class="source-line-no">223</span><span id="line-223"> ProcedureCoordinator coordinator = spy(new ProcedureCoordinator(coordinatorController, pool));</span> |
| <span class="source-line-no">224</span><span id="line-224"></span> |
| <span class="source-line-no">225</span><span id="line-225"> // start a member for each node</span> |
| <span class="source-line-no">226</span><span id="line-226"> SubprocedureFactory subprocFactory = Mockito.mock(SubprocedureFactory.class);</span> |
| <span class="source-line-no">227</span><span id="line-227"> List<Pair<ProcedureMember, ZKProcedureMemberRpcs>> members = new ArrayList<>(expected.size());</span> |
| <span class="source-line-no">228</span><span id="line-228"> for (String member : expected) {</span> |
| <span class="source-line-no">229</span><span id="line-229"> ZKWatcher watcher = newZooKeeperWatcher();</span> |
| <span class="source-line-no">230</span><span id="line-230"> ZKProcedureMemberRpcs controller = new ZKProcedureMemberRpcs(watcher, opDescription);</span> |
| <span class="source-line-no">231</span><span id="line-231"> ThreadPoolExecutor pool2 = ProcedureMember.defaultPool(member, 1, KEEP_ALIVE);</span> |
| <span class="source-line-no">232</span><span id="line-232"> ProcedureMember mem = new ProcedureMember(controller, pool2, subprocFactory);</span> |
| <span class="source-line-no">233</span><span id="line-233"> members.add(new Pair<>(mem, controller));</span> |
| <span class="source-line-no">234</span><span id="line-234"> controller.start(member, mem);</span> |
| <span class="source-line-no">235</span><span id="line-235"> }</span> |
| <span class="source-line-no">236</span><span id="line-236"></span> |
| <span class="source-line-no">237</span><span id="line-237"> // setup mock subprocedures</span> |
| <span class="source-line-no">238</span><span id="line-238"> final List<Subprocedure> cohortTasks = new ArrayList<>();</span> |
| <span class="source-line-no">239</span><span id="line-239"> final int[] elem = new int[1];</span> |
| <span class="source-line-no">240</span><span id="line-240"> for (int i = 0; i < members.size(); i++) {</span> |
| <span class="source-line-no">241</span><span id="line-241"> ForeignExceptionDispatcher cohortMonitor = new ForeignExceptionDispatcher();</span> |
| <span class="source-line-no">242</span><span id="line-242"> final ProcedureMember comms = members.get(i).getFirst();</span> |
| <span class="source-line-no">243</span><span id="line-243"> Subprocedure commit =</span> |
| <span class="source-line-no">244</span><span id="line-244"> Mockito.spy(new SubprocedureImpl(comms, opName, cohortMonitor, WAKE_FREQUENCY, TIMEOUT));</span> |
| <span class="source-line-no">245</span><span id="line-245"> // This nasty bit has one of the impls throw a TimeoutException</span> |
| <span class="source-line-no">246</span><span id="line-246"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">247</span><span id="line-247"> @Override</span> |
| <span class="source-line-no">248</span><span id="line-248"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">249</span><span id="line-249"> int index = elem[0];</span> |
| <span class="source-line-no">250</span><span id="line-250"> if (index == memberErrorIndex) {</span> |
| <span class="source-line-no">251</span><span id="line-251"> LOG.debug("Sending error to coordinator");</span> |
| <span class="source-line-no">252</span><span id="line-252"> ForeignException remoteCause =</span> |
| <span class="source-line-no">253</span><span id="line-253"> new ForeignException("TIMER", new TimeoutException("subprocTimeout", 1, 2, 0));</span> |
| <span class="source-line-no">254</span><span id="line-254"> Subprocedure r = ((Subprocedure) invocation.getMock());</span> |
| <span class="source-line-no">255</span><span id="line-255"> LOG.error("Remote commit failure, not propagating error:" + remoteCause);</span> |
| <span class="source-line-no">256</span><span id="line-256"> comms.receiveAbortProcedure(r.getName(), remoteCause);</span> |
| <span class="source-line-no">257</span><span id="line-257"> assertTrue(r.isComplete());</span> |
| <span class="source-line-no">258</span><span id="line-258"> // don't complete the error phase until the coordinator has gotten the error</span> |
| <span class="source-line-no">259</span><span id="line-259"> // notification (which ensures that we never progress past prepare)</span> |
| <span class="source-line-no">260</span><span id="line-260"> try {</span> |
| <span class="source-line-no">261</span><span id="line-261"> Procedure.waitForLatch(coordinatorReceivedErrorLatch,</span> |
| <span class="source-line-no">262</span><span id="line-262"> new ForeignExceptionDispatcher(), WAKE_FREQUENCY, "coordinator received error");</span> |
| <span class="source-line-no">263</span><span id="line-263"> } catch (InterruptedException e) {</span> |
| <span class="source-line-no">264</span><span id="line-264"> LOG.debug("Wait for latch interrupted, done:"</span> |
| <span class="source-line-no">265</span><span id="line-265"> + (coordinatorReceivedErrorLatch.getCount() == 0));</span> |
| <span class="source-line-no">266</span><span id="line-266"> // reset the interrupt status on the thread</span> |
| <span class="source-line-no">267</span><span id="line-267"> Thread.currentThread().interrupt();</span> |
| <span class="source-line-no">268</span><span id="line-268"> }</span> |
| <span class="source-line-no">269</span><span id="line-269"> }</span> |
| <span class="source-line-no">270</span><span id="line-270"> elem[0] = ++index;</span> |
| <span class="source-line-no">271</span><span id="line-271"> return null;</span> |
| <span class="source-line-no">272</span><span id="line-272"> }</span> |
| <span class="source-line-no">273</span><span id="line-273"> }).when(commit).acquireBarrier();</span> |
| <span class="source-line-no">274</span><span id="line-274"> cohortTasks.add(commit);</span> |
| <span class="source-line-no">275</span><span id="line-275"> }</span> |
| <span class="source-line-no">276</span><span id="line-276"></span> |
| <span class="source-line-no">277</span><span id="line-277"> // pass out a task per member</span> |
| <span class="source-line-no">278</span><span id="line-278"> final AtomicInteger taskIndex = new AtomicInteger();</span> |
| <span class="source-line-no">279</span><span id="line-279"> Mockito.when(subprocFactory.buildSubprocedure(Mockito.eq(opName),</span> |
| <span class="source-line-no">280</span><span id="line-280"> (byte[]) Mockito.argThat(new ArrayEquals(data)))).thenAnswer(new Answer<Subprocedure>() {</span> |
| <span class="source-line-no">281</span><span id="line-281"> @Override</span> |
| <span class="source-line-no">282</span><span id="line-282"> public Subprocedure answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">283</span><span id="line-283"> int index = taskIndex.getAndIncrement();</span> |
| <span class="source-line-no">284</span><span id="line-284"> Subprocedure commit = cohortTasks.get(index);</span> |
| <span class="source-line-no">285</span><span id="line-285"> return commit;</span> |
| <span class="source-line-no">286</span><span id="line-286"> }</span> |
| <span class="source-line-no">287</span><span id="line-287"> });</span> |
| <span class="source-line-no">288</span><span id="line-288"></span> |
| <span class="source-line-no">289</span><span id="line-289"> // setup spying on the coordinator</span> |
| <span class="source-line-no">290</span><span id="line-290"> ForeignExceptionDispatcher coordinatorTaskErrorMonitor =</span> |
| <span class="source-line-no">291</span><span id="line-291"> Mockito.spy(new ForeignExceptionDispatcher());</span> |
| <span class="source-line-no">292</span><span id="line-292"> Procedure coordinatorTask = Mockito.spy(new Procedure(coordinator, coordinatorTaskErrorMonitor,</span> |
| <span class="source-line-no">293</span><span id="line-293"> WAKE_FREQUENCY, TIMEOUT, opName, data, expected));</span> |
| <span class="source-line-no">294</span><span id="line-294"> when(coordinator.createProcedure(any(), eq(opName), eq(data), anyList()))</span> |
| <span class="source-line-no">295</span><span id="line-295"> .thenReturn(coordinatorTask);</span> |
| <span class="source-line-no">296</span><span id="line-296"> // count down the error latch when we get the remote error</span> |
| <span class="source-line-no">297</span><span id="line-297"> Mockito.doAnswer(new Answer<Void>() {</span> |
| <span class="source-line-no">298</span><span id="line-298"> @Override</span> |
| <span class="source-line-no">299</span><span id="line-299"> public Void answer(InvocationOnMock invocation) throws Throwable {</span> |
| <span class="source-line-no">300</span><span id="line-300"> // pass on the error to the master</span> |
| <span class="source-line-no">301</span><span id="line-301"> invocation.callRealMethod();</span> |
| <span class="source-line-no">302</span><span id="line-302"> // then count down the got error latch</span> |
| <span class="source-line-no">303</span><span id="line-303"> coordinatorReceivedErrorLatch.countDown();</span> |
| <span class="source-line-no">304</span><span id="line-304"> return null;</span> |
| <span class="source-line-no">305</span><span id="line-305"> }</span> |
| <span class="source-line-no">306</span><span id="line-306"> }).when(coordinatorTask).receive(Mockito.any());</span> |
| <span class="source-line-no">307</span><span id="line-307"></span> |
| <span class="source-line-no">308</span><span id="line-308"> // ----------------------------</span> |
| <span class="source-line-no">309</span><span id="line-309"> // start running the operation</span> |
| <span class="source-line-no">310</span><span id="line-310"> // ----------------------------</span> |
| <span class="source-line-no">311</span><span id="line-311"></span> |
| <span class="source-line-no">312</span><span id="line-312"> Procedure task =</span> |
| <span class="source-line-no">313</span><span id="line-313"> coordinator.startProcedure(coordinatorTaskErrorMonitor, opName, data, expected);</span> |
| <span class="source-line-no">314</span><span id="line-314"> assertEquals("Didn't mock coordinator task", coordinatorTask, task);</span> |
| <span class="source-line-no">315</span><span id="line-315"></span> |
| <span class="source-line-no">316</span><span id="line-316"> // wait for the task to complete</span> |
| <span class="source-line-no">317</span><span id="line-317"> try {</span> |
| <span class="source-line-no">318</span><span id="line-318"> task.waitForCompleted();</span> |
| <span class="source-line-no">319</span><span id="line-319"> } catch (ForeignException fe) {</span> |
| <span class="source-line-no">320</span><span id="line-320"> // this may get caught or may not</span> |
| <span class="source-line-no">321</span><span id="line-321"> }</span> |
| <span class="source-line-no">322</span><span id="line-322"></span> |
| <span class="source-line-no">323</span><span id="line-323"> // -------------</span> |
| <span class="source-line-no">324</span><span id="line-324"> // verification</span> |
| <span class="source-line-no">325</span><span id="line-325"> // -------------</span> |
| <span class="source-line-no">326</span><span id="line-326"></span> |
| <span class="source-line-no">327</span><span id="line-327"> // always expect prepared, never committed, and possible to have cleanup and finish (racy since</span> |
| <span class="source-line-no">328</span><span id="line-328"> // error case)</span> |
| <span class="source-line-no">329</span><span id="line-329"> waitAndVerifyProc(coordinatorTask, once, never(), once, atMost(1), true);</span> |
| <span class="source-line-no">330</span><span id="line-330"> verifyCohortSuccessful(expected, subprocFactory, cohortTasks, once, never(), once, once, true);</span> |
| <span class="source-line-no">331</span><span id="line-331"></span> |
| <span class="source-line-no">332</span><span id="line-332"> // close all the open things</span> |
| <span class="source-line-no">333</span><span id="line-333"> closeAll(coordinator, coordinatorController, members);</span> |
| <span class="source-line-no">334</span><span id="line-334"> }</span> |
| <span class="source-line-no">335</span><span id="line-335"></span> |
| <span class="source-line-no">336</span><span id="line-336"> /**</span> |
| <span class="source-line-no">337</span><span id="line-337"> * Wait for the coordinator task to complete, and verify all the mocks</span> |
| <span class="source-line-no">338</span><span id="line-338"> * @param proc the {@link Procedure} to execute</span> |
| <span class="source-line-no">339</span><span id="line-339"> * @param prepare the mock prepare</span> |
| <span class="source-line-no">340</span><span id="line-340"> * @param commit the mock commit</span> |
| <span class="source-line-no">341</span><span id="line-341"> * @param cleanup the mock cleanup</span> |
| <span class="source-line-no">342</span><span id="line-342"> * @param finish the mock finish</span> |
| <span class="source-line-no">343</span><span id="line-343"> * @param opHasError the operation error state</span> |
| <span class="source-line-no">344</span><span id="line-344"> * @throws Exception on unexpected failure</span> |
| <span class="source-line-no">345</span><span id="line-345"> */</span> |
| <span class="source-line-no">346</span><span id="line-346"> private void waitAndVerifyProc(Procedure proc, VerificationMode prepare, VerificationMode commit,</span> |
| <span class="source-line-no">347</span><span id="line-347"> VerificationMode cleanup, VerificationMode finish, boolean opHasError) throws Exception {</span> |
| <span class="source-line-no">348</span><span id="line-348"> boolean caughtError = false;</span> |
| <span class="source-line-no">349</span><span id="line-349"> try {</span> |
| <span class="source-line-no">350</span><span id="line-350"> proc.waitForCompleted();</span> |
| <span class="source-line-no">351</span><span id="line-351"> } catch (ForeignException fe) {</span> |
| <span class="source-line-no">352</span><span id="line-352"> caughtError = true;</span> |
| <span class="source-line-no">353</span><span id="line-353"> }</span> |
| <span class="source-line-no">354</span><span id="line-354"> // make sure that the task called all the expected phases</span> |
| <span class="source-line-no">355</span><span id="line-355"> Mockito.verify(proc, prepare).sendGlobalBarrierStart();</span> |
| <span class="source-line-no">356</span><span id="line-356"> Mockito.verify(proc, commit).sendGlobalBarrierReached();</span> |
| <span class="source-line-no">357</span><span id="line-357"> Mockito.verify(proc, finish).sendGlobalBarrierComplete();</span> |
| <span class="source-line-no">358</span><span id="line-358"> assertEquals("Operation error state was unexpected", opHasError,</span> |
| <span class="source-line-no">359</span><span id="line-359"> proc.getErrorMonitor().hasException());</span> |
| <span class="source-line-no">360</span><span id="line-360"> assertEquals("Operation error state was unexpected", opHasError, caughtError);</span> |
| <span class="source-line-no">361</span><span id="line-361"></span> |
| <span class="source-line-no">362</span><span id="line-362"> }</span> |
| <span class="source-line-no">363</span><span id="line-363"></span> |
| <span class="source-line-no">364</span><span id="line-364"> /**</span> |
| <span class="source-line-no">365</span><span id="line-365"> * Wait for the coordinator task to complete, and verify all the mocks</span> |
| <span class="source-line-no">366</span><span id="line-366"> * @param op the {@link Subprocedure} to use</span> |
| <span class="source-line-no">367</span><span id="line-367"> * @param prepare the mock prepare</span> |
| <span class="source-line-no">368</span><span id="line-368"> * @param commit the mock commit</span> |
| <span class="source-line-no">369</span><span id="line-369"> * @param cleanup the mock cleanup</span> |
| <span class="source-line-no">370</span><span id="line-370"> * @param finish the mock finish</span> |
| <span class="source-line-no">371</span><span id="line-371"> * @param opHasError the operation error state</span> |
| <span class="source-line-no">372</span><span id="line-372"> * @throws Exception on unexpected failure</span> |
| <span class="source-line-no">373</span><span id="line-373"> */</span> |
| <span class="source-line-no">374</span><span id="line-374"> private void waitAndVerifySubproc(Subprocedure op, VerificationMode prepare,</span> |
| <span class="source-line-no">375</span><span id="line-375"> VerificationMode commit, VerificationMode cleanup, VerificationMode finish, boolean opHasError)</span> |
| <span class="source-line-no">376</span><span id="line-376"> throws Exception {</span> |
| <span class="source-line-no">377</span><span id="line-377"> boolean caughtError = false;</span> |
| <span class="source-line-no">378</span><span id="line-378"> try {</span> |
| <span class="source-line-no">379</span><span id="line-379"> op.waitForLocallyCompleted();</span> |
| <span class="source-line-no">380</span><span id="line-380"> } catch (ForeignException fe) {</span> |
| <span class="source-line-no">381</span><span id="line-381"> caughtError = true;</span> |
| <span class="source-line-no">382</span><span id="line-382"> }</span> |
| <span class="source-line-no">383</span><span id="line-383"> // make sure that the task called all the expected phases</span> |
| <span class="source-line-no">384</span><span id="line-384"> Mockito.verify(op, prepare).acquireBarrier();</span> |
| <span class="source-line-no">385</span><span id="line-385"> Mockito.verify(op, commit).insideBarrier();</span> |
| <span class="source-line-no">386</span><span id="line-386"> // We cannot guarantee that cleanup has run so we don't check it.</span> |
| <span class="source-line-no">387</span><span id="line-387"></span> |
| <span class="source-line-no">388</span><span id="line-388"> assertEquals("Operation error state was unexpected", opHasError,</span> |
| <span class="source-line-no">389</span><span id="line-389"> op.getErrorCheckable().hasException());</span> |
| <span class="source-line-no">390</span><span id="line-390"> assertEquals("Operation error state was unexpected", opHasError, caughtError);</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"></span> |
| <span class="source-line-no">394</span><span id="line-394"> private void verifyCohortSuccessful(List<String> cohortNames, SubprocedureFactory subprocFactory,</span> |
| <span class="source-line-no">395</span><span id="line-395"> Iterable<Subprocedure> cohortTasks, VerificationMode prepare, VerificationMode commit,</span> |
| <span class="source-line-no">396</span><span id="line-396"> VerificationMode cleanup, VerificationMode finish, boolean opHasError) throws Exception {</span> |
| <span class="source-line-no">397</span><span id="line-397"></span> |
| <span class="source-line-no">398</span><span id="line-398"> // make sure we build the correct number of cohort members</span> |
| <span class="source-line-no">399</span><span id="line-399"> Mockito.verify(subprocFactory, Mockito.times(cohortNames.size()))</span> |
| <span class="source-line-no">400</span><span id="line-400"> .buildSubprocedure(Mockito.eq(opName), (byte[]) Mockito.argThat(new ArrayEquals(data)));</span> |
| <span class="source-line-no">401</span><span id="line-401"> // verify that we ran each of the operations cleanly</span> |
| <span class="source-line-no">402</span><span id="line-402"> int j = 0;</span> |
| <span class="source-line-no">403</span><span id="line-403"> for (Subprocedure op : cohortTasks) {</span> |
| <span class="source-line-no">404</span><span id="line-404"> LOG.debug("Checking mock:" + (j++));</span> |
| <span class="source-line-no">405</span><span id="line-405"> waitAndVerifySubproc(op, prepare, commit, cleanup, finish, opHasError);</span> |
| <span class="source-line-no">406</span><span id="line-406"> }</span> |
| <span class="source-line-no">407</span><span id="line-407"> }</span> |
| <span class="source-line-no">408</span><span id="line-408"></span> |
| <span class="source-line-no">409</span><span id="line-409"> private void closeAll(ProcedureCoordinator coordinator,</span> |
| <span class="source-line-no">410</span><span id="line-410"> ZKProcedureCoordinator coordinatorController,</span> |
| <span class="source-line-no">411</span><span id="line-411"> List<Pair<ProcedureMember, ZKProcedureMemberRpcs>> cohort) throws IOException {</span> |
| <span class="source-line-no">412</span><span id="line-412"> // make sure we close all the resources</span> |
| <span class="source-line-no">413</span><span id="line-413"> for (Pair<ProcedureMember, ZKProcedureMemberRpcs> member : cohort) {</span> |
| <span class="source-line-no">414</span><span id="line-414"> member.getFirst().close();</span> |
| <span class="source-line-no">415</span><span id="line-415"> member.getSecond().close();</span> |
| <span class="source-line-no">416</span><span id="line-416"> }</span> |
| <span class="source-line-no">417</span><span id="line-417"> coordinator.close();</span> |
| <span class="source-line-no">418</span><span id="line-418"> coordinatorController.close();</span> |
| <span class="source-line-no">419</span><span id="line-419"> }</span> |
| <span class="source-line-no">420</span><span id="line-420">}</span> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| </pre> |
| </div> |
| </main> |
| </body> |
| </html> |
