You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@omid.apache.org by fp...@apache.org on 2016/06/17 21:21:06 UTC
[31/51] [partial] incubator-omid-site git commit: [wagon-git] commit
to branch asf-site 6/17/16 2:20 PM
http://git-wip-us.apache.org/repos/asf/incubator-omid-site/blob/3a40a657/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
----------------------------------------------------------------------
diff --git a/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html b/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
index 8ba282a..4ee07b1 100644
--- a/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
+++ b/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
@@ -8,227 +8,507 @@
<body>
<div id="overview"><a href="../../../../../../target/site/testapidocs/org/apache/omid/tso/TestPersistenceProcessor.html">View Javadoc</a></div><pre>
-<a class="jxr_linenumber" name="1" href="#1">1</a> <em class="jxr_javadoccomment">/**</em>
-<a class="jxr_linenumber" name="2" href="#2">2</a> <em class="jxr_javadoccomment"> * Licensed to the Apache Software Foundation (ASF) under one</em>
-<a class="jxr_linenumber" name="3" href="#3">3</a> <em class="jxr_javadoccomment"> * or more contributor license agreements. See the NOTICE file</em>
-<a class="jxr_linenumber" name="4" href="#4">4</a> <em class="jxr_javadoccomment"> * distributed with this work for additional information</em>
-<a class="jxr_linenumber" name="5" href="#5">5</a> <em class="jxr_javadoccomment"> * regarding copyright ownership. The ASF licenses this file</em>
-<a class="jxr_linenumber" name="6" href="#6">6</a> <em class="jxr_javadoccomment"> * to you under the Apache License, Version 2.0 (the</em>
-<a class="jxr_linenumber" name="7" href="#7">7</a> <em class="jxr_javadoccomment"> * "License"); you may not use this file except in compliance</em>
-<a class="jxr_linenumber" name="8" href="#8">8</a> <em class="jxr_javadoccomment"> * with the License. You may obtain a copy of the License at</em>
-<a class="jxr_linenumber" name="9" href="#9">9</a> <em class="jxr_javadoccomment"> *</em>
-<a class="jxr_linenumber" name="10" href="#10">10</a> <em class="jxr_javadoccomment"> * <a href="http://www.apache.org/licenses/LICENSE-2.0" target="alexandria_uri">http://www.apache.org/licenses/LICENSE-2.0</a></em>
-<a class="jxr_linenumber" name="11" href="#11">11</a> <em class="jxr_javadoccomment"> *</em>
-<a class="jxr_linenumber" name="12" href="#12">12</a> <em class="jxr_javadoccomment"> * Unless required by applicable law or agreed to in writing, software</em>
-<a class="jxr_linenumber" name="13" href="#13">13</a> <em class="jxr_javadoccomment"> * distributed under the License is distributed on an "AS IS" BASIS,</em>
-<a class="jxr_linenumber" name="14" href="#14">14</a> <em class="jxr_javadoccomment"> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.</em>
-<a class="jxr_linenumber" name="15" href="#15">15</a> <em class="jxr_javadoccomment"> * See the License for the specific language governing permissions and</em>
-<a class="jxr_linenumber" name="16" href="#16">16</a> <em class="jxr_javadoccomment"> * limitations under the License.</em>
-<a class="jxr_linenumber" name="17" href="#17">17</a> <em class="jxr_javadoccomment"> */</em>
+<a class="jxr_linenumber" name="1" href="#1">1</a> <em class="jxr_comment">/*</em>
+<a class="jxr_linenumber" name="2" href="#2">2</a> <em class="jxr_comment"> * Licensed to the Apache Software Foundation (ASF) under one</em>
+<a class="jxr_linenumber" name="3" href="#3">3</a> <em class="jxr_comment"> * or more contributor license agreements. See the NOTICE file</em>
+<a class="jxr_linenumber" name="4" href="#4">4</a> <em class="jxr_comment"> * distributed with this work for additional information</em>
+<a class="jxr_linenumber" name="5" href="#5">5</a> <em class="jxr_comment"> * regarding copyright ownership. The ASF licenses this file</em>
+<a class="jxr_linenumber" name="6" href="#6">6</a> <em class="jxr_comment"> * to you under the Apache License, Version 2.0 (the</em>
+<a class="jxr_linenumber" name="7" href="#7">7</a> <em class="jxr_comment"> * "License"); you may not use this file except in compliance</em>
+<a class="jxr_linenumber" name="8" href="#8">8</a> <em class="jxr_comment"> * with the License. You may obtain a copy of the License at</em>
+<a class="jxr_linenumber" name="9" href="#9">9</a> <em class="jxr_comment"> *</em>
+<a class="jxr_linenumber" name="10" href="#10">10</a> <em class="jxr_comment"> * <a href="http://www.apache.org/licenses/LICENSE-2.0" target="alexandria_uri">http://www.apache.org/licenses/LICENSE-2.0</a></em>
+<a class="jxr_linenumber" name="11" href="#11">11</a> <em class="jxr_comment"> *</em>
+<a class="jxr_linenumber" name="12" href="#12">12</a> <em class="jxr_comment"> * Unless required by applicable law or agreed to in writing, software</em>
+<a class="jxr_linenumber" name="13" href="#13">13</a> <em class="jxr_comment"> * distributed under the License is distributed on an "AS IS" BASIS,</em>
+<a class="jxr_linenumber" name="14" href="#14">14</a> <em class="jxr_comment"> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.</em>
+<a class="jxr_linenumber" name="15" href="#15">15</a> <em class="jxr_comment"> * See the License for the specific language governing permissions and</em>
+<a class="jxr_linenumber" name="16" href="#16">16</a> <em class="jxr_comment"> * limitations under the License.</em>
+<a class="jxr_linenumber" name="17" href="#17">17</a> <em class="jxr_comment"> */</em>
<a class="jxr_linenumber" name="18" href="#18">18</a> <strong class="jxr_keyword">package</strong> org.apache.omid.tso;
<a class="jxr_linenumber" name="19" href="#19">19</a>
-<a class="jxr_linenumber" name="20" href="#20">20</a> <strong class="jxr_keyword">import</strong> org.apache.omid.committable.CommitTable;
-<a class="jxr_linenumber" name="21" href="#21">21</a> <strong class="jxr_keyword">import</strong> org.apache.omid.metrics.MetricsRegistry;
-<a class="jxr_linenumber" name="22" href="#22">22</a> <strong class="jxr_keyword">import</strong> org.apache.omid.metrics.NullMetricsProvider;
-<a class="jxr_linenumber" name="23" href="#23">23</a> <strong class="jxr_keyword">import</strong> org.apache.omid.tso.PersistenceProcessorImpl.Batch;
-<a class="jxr_linenumber" name="24" href="#24">24</a> <strong class="jxr_keyword">import</strong> org.mockito.Mock;
-<a class="jxr_linenumber" name="25" href="#25">25</a> <strong class="jxr_keyword">import</strong> org.mockito.Mockito;
-<a class="jxr_linenumber" name="26" href="#26">26</a> <strong class="jxr_keyword">import</strong> org.mockito.MockitoAnnotations;
-<a class="jxr_linenumber" name="27" href="#27">27</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.AfterMethod;
-<a class="jxr_linenumber" name="28" href="#28">28</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.BeforeMethod;
-<a class="jxr_linenumber" name="29" href="#29">29</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.Test;
-<a class="jxr_linenumber" name="30" href="#30">30</a>
-<a class="jxr_linenumber" name="31" href="#31">31</a> <strong class="jxr_keyword">import</strong> java.io.IOException;
-<a class="jxr_linenumber" name="32" href="#32">32</a>
-<a class="jxr_linenumber" name="33" href="#33">33</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.any;
-<a class="jxr_linenumber" name="34" href="#34">34</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyLong;
-<a class="jxr_linenumber" name="35" href="#35">35</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyString;
-<a class="jxr_linenumber" name="36" href="#36">36</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.eq;
-<a class="jxr_linenumber" name="37" href="#37">37</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.doReturn;
-<a class="jxr_linenumber" name="38" href="#38">38</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.doThrow;
-<a class="jxr_linenumber" name="39" href="#39">39</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.mock;
-<a class="jxr_linenumber" name="40" href="#40">40</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.reset;
-<a class="jxr_linenumber" name="41" href="#41">41</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.spy;
-<a class="jxr_linenumber" name="42" href="#42">42</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.timeout;
-<a class="jxr_linenumber" name="43" href="#43">43</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.verify;
-<a class="jxr_linenumber" name="44" href="#44">44</a>
-<a class="jxr_linenumber" name="45" href="#45">45</a> @SuppressWarnings({<span class="jxr_string">"UnusedDeclaration"</span>})
-<a class="jxr_linenumber" name="46" href="#46">46</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../org/apache/omid/tso/TestPersistenceProcessor.html">TestPersistenceProcessor</a> {
-<a class="jxr_linenumber" name="47" href="#47">47</a>
-<a class="jxr_linenumber" name="48" href="#48">48</a> @Mock
-<a class="jxr_linenumber" name="49" href="#49">49</a> <strong class="jxr_keyword">private</strong> Batch batch;
-<a class="jxr_linenumber" name="50" href="#50">50</a> @Mock
-<a class="jxr_linenumber" name="51" href="#51">51</a> <strong class="jxr_keyword">private</strong> CommitTable.Writer mockWriter;
-<a class="jxr_linenumber" name="52" href="#52">52</a> @Mock
-<a class="jxr_linenumber" name="53" href="#53">53</a> <strong class="jxr_keyword">private</strong> CommitTable.Client mockClient;
-<a class="jxr_linenumber" name="54" href="#54">54</a> @Mock
-<a class="jxr_linenumber" name="55" href="#55">55</a> <strong class="jxr_keyword">private</strong> RetryProcessor retryProcessor;
-<a class="jxr_linenumber" name="56" href="#56">56</a> @Mock
-<a class="jxr_linenumber" name="57" href="#57">57</a> <strong class="jxr_keyword">private</strong> ReplyProcessor replyProcessor;
+<a class="jxr_linenumber" name="20" href="#20">20</a> <strong class="jxr_keyword">import</strong> org.apache.commons.pool2.ObjectPool;
+<a class="jxr_linenumber" name="21" href="#21">21</a> <strong class="jxr_keyword">import</strong> org.apache.omid.committable.CommitTable;
+<a class="jxr_linenumber" name="22" href="#22">22</a> <strong class="jxr_keyword">import</strong> org.apache.omid.metrics.MetricsRegistry;
+<a class="jxr_linenumber" name="23" href="#23">23</a> <strong class="jxr_keyword">import</strong> org.apache.omid.metrics.NullMetricsProvider;
+<a class="jxr_linenumber" name="24" href="#24">24</a> <strong class="jxr_keyword">import</strong> org.jboss.netty.channel.Channel;
+<a class="jxr_linenumber" name="25" href="#25">25</a> <strong class="jxr_keyword">import</strong> org.mockito.ArgumentCaptor;
+<a class="jxr_linenumber" name="26" href="#26">26</a> <strong class="jxr_keyword">import</strong> org.mockito.Mock;
+<a class="jxr_linenumber" name="27" href="#27">27</a> <strong class="jxr_keyword">import</strong> org.mockito.Mockito;
+<a class="jxr_linenumber" name="28" href="#28">28</a> <strong class="jxr_keyword">import</strong> org.mockito.MockitoAnnotations;
+<a class="jxr_linenumber" name="29" href="#29">29</a> <strong class="jxr_keyword">import</strong> org.slf4j.Logger;
+<a class="jxr_linenumber" name="30" href="#30">30</a> <strong class="jxr_keyword">import</strong> org.slf4j.LoggerFactory;
+<a class="jxr_linenumber" name="31" href="#31">31</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.AfterMethod;
+<a class="jxr_linenumber" name="32" href="#32">32</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.BeforeMethod;
+<a class="jxr_linenumber" name="33" href="#33">33</a> <strong class="jxr_keyword">import</strong> org.testng.annotations.Test;
+<a class="jxr_linenumber" name="34" href="#34">34</a>
+<a class="jxr_linenumber" name="35" href="#35">35</a> <strong class="jxr_keyword">import</strong> java.io.IOException;
+<a class="jxr_linenumber" name="36" href="#36">36</a>
+<a class="jxr_linenumber" name="37" href="#37">37</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.any;
+<a class="jxr_linenumber" name="38" href="#38">38</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyLong;
+<a class="jxr_linenumber" name="39" href="#39">39</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyString;
+<a class="jxr_linenumber" name="40" href="#40">40</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.doReturn;
+<a class="jxr_linenumber" name="41" href="#41">41</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.doThrow;
+<a class="jxr_linenumber" name="42" href="#42">42</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.mock;
+<a class="jxr_linenumber" name="43" href="#43">43</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.spy;
+<a class="jxr_linenumber" name="44" href="#44">44</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.timeout;
+<a class="jxr_linenumber" name="45" href="#45">45</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.times;
+<a class="jxr_linenumber" name="46" href="#46">46</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.verify;
+<a class="jxr_linenumber" name="47" href="#47">47</a> <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.testng.Assert.assertEquals;
+<a class="jxr_linenumber" name="48" href="#48">48</a>
+<a class="jxr_linenumber" name="49" href="#49">49</a> <em class="jxr_comment">// TODO Refactor: Make visible currentBatch in PersistenceProcessorImpl to add proper verifications</em>
+<a class="jxr_linenumber" name="50" href="#50">50</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../org/apache/omid/tso/TestPersistenceProcessor.html">TestPersistenceProcessor</a> {
+<a class="jxr_linenumber" name="51" href="#51">51</a>
+<a class="jxr_linenumber" name="52" href="#52">52</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> Logger LOG = LoggerFactory.getLogger(TestPersistenceProcessor.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="53" href="#53">53</a>
+<a class="jxr_linenumber" name="54" href="#54">54</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">long</strong> ANY_LWM = 1234L;
+<a class="jxr_linenumber" name="55" href="#55">55</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> ANY_ST = 0;
+<a class="jxr_linenumber" name="56" href="#56">56</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> ANY_CT = 1;
+<a class="jxr_linenumber" name="57" href="#57">57</a>
<a class="jxr_linenumber" name="58" href="#58">58</a> @Mock
-<a class="jxr_linenumber" name="59" href="#59">59</a> <strong class="jxr_keyword">private</strong> Panicker panicker;
-<a class="jxr_linenumber" name="60" href="#60">60</a>
-<a class="jxr_linenumber" name="61" href="#61">61</a> <strong class="jxr_keyword">private</strong> MetricsRegistry metrics;
-<a class="jxr_linenumber" name="62" href="#62">62</a> <strong class="jxr_keyword">private</strong> CommitTable commitTable;
-<a class="jxr_linenumber" name="63" href="#63">63</a>
-<a class="jxr_linenumber" name="64" href="#64">64</a> @BeforeMethod(alwaysRun = <strong class="jxr_keyword">true</strong>, timeOut = 30_000)
-<a class="jxr_linenumber" name="65" href="#65">65</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> initMocksAndComponents() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="59" href="#59">59</a> <strong class="jxr_keyword">private</strong> CommitTable.Writer mockWriter;
+<a class="jxr_linenumber" name="60" href="#60">60</a> @Mock
+<a class="jxr_linenumber" name="61" href="#61">61</a> <strong class="jxr_keyword">private</strong> CommitTable.Client mockClient;
+<a class="jxr_linenumber" name="62" href="#62">62</a> @Mock
+<a class="jxr_linenumber" name="63" href="#63">63</a> <strong class="jxr_keyword">private</strong> RetryProcessor retryProcessor;
+<a class="jxr_linenumber" name="64" href="#64">64</a> @Mock
+<a class="jxr_linenumber" name="65" href="#65">65</a> <strong class="jxr_keyword">private</strong> Panicker panicker;
<a class="jxr_linenumber" name="66" href="#66">66</a>
-<a class="jxr_linenumber" name="67" href="#67">67</a> MockitoAnnotations.initMocks(<strong class="jxr_keyword">this</strong>);
-<a class="jxr_linenumber" name="68" href="#68">68</a>
-<a class="jxr_linenumber" name="69" href="#69">69</a> <em class="jxr_comment">// Configure mock writer to flush successfully</em>
-<a class="jxr_linenumber" name="70" href="#70">70</a> doThrow(<strong class="jxr_keyword">new</strong> IOException(<span class="jxr_string">"Unable to write"</span>)).when(mockWriter).flush();
-<a class="jxr_linenumber" name="71" href="#71">71</a>
-<a class="jxr_linenumber" name="72" href="#72">72</a> <em class="jxr_comment">// Configure null metrics provider</em>
-<a class="jxr_linenumber" name="73" href="#73">73</a> metrics = <strong class="jxr_keyword">new</strong> NullMetricsProvider();
+<a class="jxr_linenumber" name="67" href="#67">67</a> <strong class="jxr_keyword">private</strong> MetricsRegistry metrics;
+<a class="jxr_linenumber" name="68" href="#68">68</a> <strong class="jxr_keyword">private</strong> CommitTable commitTable;
+<a class="jxr_linenumber" name="69" href="#69">69</a>
+<a class="jxr_linenumber" name="70" href="#70">70</a> @BeforeMethod(alwaysRun = <strong class="jxr_keyword">true</strong>, timeOut = 30_000)
+<a class="jxr_linenumber" name="71" href="#71">71</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> initMocksAndComponents() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="72" href="#72">72</a>
+<a class="jxr_linenumber" name="73" href="#73">73</a> MockitoAnnotations.initMocks(<strong class="jxr_keyword">this</strong>);
<a class="jxr_linenumber" name="74" href="#74">74</a>
-<a class="jxr_linenumber" name="75" href="#75">75</a> <em class="jxr_comment">// Configure commit table to return the mocked writer and client</em>
-<a class="jxr_linenumber" name="76" href="#76">76</a> commitTable = <strong class="jxr_keyword">new</strong> CommitTable() {
-<a class="jxr_linenumber" name="77" href="#77">77</a> @Override
-<a class="jxr_linenumber" name="78" href="#78">78</a> <strong class="jxr_keyword">public</strong> Writer getWriter() {
-<a class="jxr_linenumber" name="79" href="#79">79</a> <strong class="jxr_keyword">return</strong> mockWriter;
-<a class="jxr_linenumber" name="80" href="#80">80</a> }
-<a class="jxr_linenumber" name="81" href="#81">81</a>
-<a class="jxr_linenumber" name="82" href="#82">82</a> @Override
-<a class="jxr_linenumber" name="83" href="#83">83</a> <strong class="jxr_keyword">public</strong> Client getClient() {
-<a class="jxr_linenumber" name="84" href="#84">84</a> <strong class="jxr_keyword">return</strong> mockClient;
-<a class="jxr_linenumber" name="85" href="#85">85</a> }
-<a class="jxr_linenumber" name="86" href="#86">86</a> };
-<a class="jxr_linenumber" name="87" href="#87">87</a> }
-<a class="jxr_linenumber" name="88" href="#88">88</a>
-<a class="jxr_linenumber" name="89" href="#89">89</a> @AfterMethod
-<a class="jxr_linenumber" name="90" href="#90">90</a> <strong class="jxr_keyword">void</strong> afterMethod() {
-<a class="jxr_linenumber" name="91" href="#91">91</a> Mockito.reset(mockWriter);
-<a class="jxr_linenumber" name="92" href="#92">92</a> }
-<a class="jxr_linenumber" name="93" href="#93">93</a>
-<a class="jxr_linenumber" name="94" href="#94">94</a> @Test
-<a class="jxr_linenumber" name="95" href="#95">95</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithNonHALeaseManager() <strong class="jxr_keyword">throws</strong> Exception {
-<a class="jxr_linenumber" name="96" href="#96">96</a>
-<a class="jxr_linenumber" name="97" href="#97">97</a> <em class="jxr_comment">// Init a non-HA lease manager</em>
-<a class="jxr_linenumber" name="98" href="#98">98</a> VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="99" href="#99">99</a> mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="75" href="#75">75</a> <em class="jxr_comment">// Configure null metrics provider</em>
+<a class="jxr_linenumber" name="76" href="#76">76</a> metrics = <strong class="jxr_keyword">new</strong> NullMetricsProvider();
+<a class="jxr_linenumber" name="77" href="#77">77</a>
+<a class="jxr_linenumber" name="78" href="#78">78</a> <em class="jxr_comment">// Configure commit table to return the mocked writer and client</em>
+<a class="jxr_linenumber" name="79" href="#79">79</a> commitTable = <strong class="jxr_keyword">new</strong> CommitTable() {
+<a class="jxr_linenumber" name="80" href="#80">80</a> @Override
+<a class="jxr_linenumber" name="81" href="#81">81</a> <strong class="jxr_keyword">public</strong> Writer getWriter() {
+<a class="jxr_linenumber" name="82" href="#82">82</a> <strong class="jxr_keyword">return</strong> mockWriter;
+<a class="jxr_linenumber" name="83" href="#83">83</a> }
+<a class="jxr_linenumber" name="84" href="#84">84</a>
+<a class="jxr_linenumber" name="85" href="#85">85</a> @Override
+<a class="jxr_linenumber" name="86" href="#86">86</a> <strong class="jxr_keyword">public</strong> Client getClient() {
+<a class="jxr_linenumber" name="87" href="#87">87</a> <strong class="jxr_keyword">return</strong> mockClient;
+<a class="jxr_linenumber" name="88" href="#88">88</a> }
+<a class="jxr_linenumber" name="89" href="#89">89</a> };
+<a class="jxr_linenumber" name="90" href="#90">90</a>
+<a class="jxr_linenumber" name="91" href="#91">91</a> }
+<a class="jxr_linenumber" name="92" href="#92">92</a>
+<a class="jxr_linenumber" name="93" href="#93">93</a> @AfterMethod
+<a class="jxr_linenumber" name="94" href="#94">94</a> <strong class="jxr_keyword">void</strong> afterMethod() {
+<a class="jxr_linenumber" name="95" href="#95">95</a> Mockito.reset(mockWriter);
+<a class="jxr_linenumber" name="96" href="#96">96</a> }
+<a class="jxr_linenumber" name="97" href="#97">97</a>
+<a class="jxr_linenumber" name="98" href="#98">98</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="99" href="#99">99</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testLowWatermarkIsPersisted() <strong class="jxr_keyword">throws</strong> Exception {
<a class="jxr_linenumber" name="100" href="#100">100</a>
-<a class="jxr_linenumber" name="101" href="#101">101</a> TSOServerConfig tsoServerConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
-<a class="jxr_linenumber" name="102" href="#102">102</a> tsoServerConfig.setBatchPersistTimeoutInMs(100);
-<a class="jxr_linenumber" name="103" href="#103">103</a> <em class="jxr_comment">// Component under test</em>
-<a class="jxr_linenumber" name="104" href="#104">104</a> PersistenceProcessor proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoServerConfig,
-<a class="jxr_linenumber" name="105" href="#105">105</a> metrics,
-<a class="jxr_linenumber" name="106" href="#106">106</a> <span class="jxr_string">"localhost:1234"</span>,
-<a class="jxr_linenumber" name="107" href="#107">107</a> batch,
-<a class="jxr_linenumber" name="108" href="#108">108</a> leaseManager,
-<a class="jxr_linenumber" name="109" href="#109">109</a> commitTable,
-<a class="jxr_linenumber" name="110" href="#110">110</a> replyProcessor,
-<a class="jxr_linenumber" name="111" href="#111">111</a> retryProcessor,
-<a class="jxr_linenumber" name="112" href="#112">112</a> panicker);
+<a class="jxr_linenumber" name="101" href="#101">101</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="102" href="#102">102</a>
+<a class="jxr_linenumber" name="103" href="#103">103</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="104" href="#104">104</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="105" href="#105">105</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="106" href="#106">106</a> <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="107" href="#107">107</a> mock(LeaseManager.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="108" href="#108">108</a> commitTable,
+<a class="jxr_linenumber" name="109" href="#109">109</a> mock(ReplyProcessor.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="110" href="#110">110</a> retryProcessor,
+<a class="jxr_linenumber" name="111" href="#111">111</a> panicker);
+<a class="jxr_linenumber" name="112" href="#112">112</a> }
<a class="jxr_linenumber" name="113" href="#113">113</a>
-<a class="jxr_linenumber" name="114" href="#114">114</a> <em class="jxr_comment">// The non-ha lease manager always return true for</em>
-<a class="jxr_linenumber" name="115" href="#115">115</a> <em class="jxr_comment">// stillInLeasePeriod(), so verify the batch sends replies as master</em>
-<a class="jxr_linenumber" name="116" href="#116">116</a> MonitoringContext monCtx = <strong class="jxr_keyword">new</strong> MonitoringContext(metrics);
-<a class="jxr_linenumber" name="117" href="#117">117</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="118" href="#118">118</a> verify(leaseManager, timeout(1000).times(2)).stillInLeasePeriod();
-<a class="jxr_linenumber" name="119" href="#119">119</a> verify(batch, timeout(1000).times(2)).sendRepliesAndReset(any(ReplyProcessor.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="120" href="#120">120</a> any(RetryProcessor.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="121" href="#121">121</a> eq(<strong class="jxr_keyword">true</strong>));
-<a class="jxr_linenumber" name="122" href="#122">122</a> }
-<a class="jxr_linenumber" name="123" href="#123">123</a>
-<a class="jxr_linenumber" name="124" href="#124">124</a> @Test
-<a class="jxr_linenumber" name="125" href="#125">125</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithHALeaseManager() <strong class="jxr_keyword">throws</strong> Exception {
-<a class="jxr_linenumber" name="126" href="#126">126</a>
-<a class="jxr_linenumber" name="127" href="#127">127</a> <em class="jxr_comment">// Init a HA lease manager</em>
-<a class="jxr_linenumber" name="128" href="#128">128</a> LeaseManager leaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="114" href="#114">114</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="115" href="#115">115</a> PersistenceProcessorImpl persistenceProcessor =
+<a class="jxr_linenumber" name="116" href="#116">116</a> <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig,
+<a class="jxr_linenumber" name="117" href="#117">117</a> commitTable,
+<a class="jxr_linenumber" name="118" href="#118">118</a> mock(ObjectPool.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="119" href="#119">119</a> panicker,
+<a class="jxr_linenumber" name="120" href="#120">120</a> handlers,
+<a class="jxr_linenumber" name="121" href="#121">121</a> metrics);
+<a class="jxr_linenumber" name="122" href="#122">122</a>
+<a class="jxr_linenumber" name="123" href="#123">123</a> persistenceProcessor.persistLowWatermark(ANY_LWM).get();
+<a class="jxr_linenumber" name="124" href="#124">124</a>
+<a class="jxr_linenumber" name="125" href="#125">125</a> ArgumentCaptor<Long> lwmCapture = ArgumentCaptor.forClass(Long.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="126" href="#126">126</a> CommitTable.Writer lwmWriter = commitTable.getWriter();
+<a class="jxr_linenumber" name="127" href="#127">127</a> verify(lwmWriter, timeout(100).times(1)).updateLowWatermark(lwmCapture.capture());
+<a class="jxr_linenumber" name="128" href="#128">128</a> assertEquals(lwmCapture.getValue().longValue(), ANY_LWM);
<a class="jxr_linenumber" name="129" href="#129">129</a>
-<a class="jxr_linenumber" name="130" href="#130">130</a> TSOServerConfig tsoServerConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
-<a class="jxr_linenumber" name="131" href="#131">131</a> tsoServerConfig.setBatchPersistTimeoutInMs(100);
-<a class="jxr_linenumber" name="132" href="#132">132</a> <em class="jxr_comment">// Component under test</em>
-<a class="jxr_linenumber" name="133" href="#133">133</a> PersistenceProcessor proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoServerConfig,
-<a class="jxr_linenumber" name="134" href="#134">134</a> metrics,
-<a class="jxr_linenumber" name="135" href="#135">135</a> <span class="jxr_string">"localhost:1234"</span>,
-<a class="jxr_linenumber" name="136" href="#136">136</a> batch,
-<a class="jxr_linenumber" name="137" href="#137">137</a> leaseManager,
-<a class="jxr_linenumber" name="138" href="#138">138</a> commitTable,
-<a class="jxr_linenumber" name="139" href="#139">139</a> replyProcessor,
-<a class="jxr_linenumber" name="140" href="#140">140</a> retryProcessor,
-<a class="jxr_linenumber" name="141" href="#141">141</a> panicker);
-<a class="jxr_linenumber" name="142" href="#142">142</a>
-<a class="jxr_linenumber" name="143" href="#143">143</a> <em class="jxr_comment">// Configure the lease manager to always return true for</em>
-<a class="jxr_linenumber" name="144" href="#144">144</a> <em class="jxr_comment">// stillInLeasePeriod, so verify the batch sends replies as master</em>
-<a class="jxr_linenumber" name="145" href="#145">145</a> doReturn(<strong class="jxr_keyword">true</strong>).when(leaseManager).stillInLeasePeriod();
-<a class="jxr_linenumber" name="146" href="#146">146</a> MonitoringContext monCtx = <strong class="jxr_keyword">new</strong> MonitoringContext(metrics);
-<a class="jxr_linenumber" name="147" href="#147">147</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="148" href="#148">148</a> verify(leaseManager, timeout(1000).times(2)).stillInLeasePeriod();
-<a class="jxr_linenumber" name="149" href="#149">149</a> verify(batch).sendRepliesAndReset(any(ReplyProcessor.<strong class="jxr_keyword">class</strong>), any(RetryProcessor.<strong class="jxr_keyword">class</strong>), eq(<strong class="jxr_keyword">true</strong>));
-<a class="jxr_linenumber" name="150" href="#150">150</a>
-<a class="jxr_linenumber" name="151" href="#151">151</a> <em class="jxr_comment">// Configure the lease manager to always return true first and false</em>
-<a class="jxr_linenumber" name="152" href="#152">152</a> <em class="jxr_comment">// later for stillInLeasePeriod, so verify the batch sends replies as</em>
-<a class="jxr_linenumber" name="153" href="#153">153</a> <em class="jxr_comment">// non-master</em>
-<a class="jxr_linenumber" name="154" href="#154">154</a> reset(leaseManager);
-<a class="jxr_linenumber" name="155" href="#155">155</a> reset(batch);
-<a class="jxr_linenumber" name="156" href="#156">156</a> doReturn(<strong class="jxr_keyword">true</strong>).doReturn(false).when(leaseManager).stillInLeasePeriod();
-<a class="jxr_linenumber" name="157" href="#157">157</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="158" href="#158">158</a> verify(leaseManager, timeout(1000).times(2)).stillInLeasePeriod();
-<a class="jxr_linenumber" name="159" href="#159">159</a> verify(batch).sendRepliesAndReset(any(ReplyProcessor.<strong class="jxr_keyword">class</strong>), any(RetryProcessor.<strong class="jxr_keyword">class</strong>), eq(false));
-<a class="jxr_linenumber" name="160" href="#160">160</a>
-<a class="jxr_linenumber" name="161" href="#161">161</a> <em class="jxr_comment">// Configure the lease manager to always return false for</em>
-<a class="jxr_linenumber" name="162" href="#162">162</a> <em class="jxr_comment">// stillInLeasePeriod, so verify the batch sends replies as non-master</em>
-<a class="jxr_linenumber" name="163" href="#163">163</a> reset(leaseManager);
-<a class="jxr_linenumber" name="164" href="#164">164</a> reset(batch);
-<a class="jxr_linenumber" name="165" href="#165">165</a> doReturn(false).when(leaseManager).stillInLeasePeriod();
-<a class="jxr_linenumber" name="166" href="#166">166</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="167" href="#167">167</a> verify(leaseManager, timeout(1000).times(1)).stillInLeasePeriod();
-<a class="jxr_linenumber" name="168" href="#168">168</a> verify(batch).sendRepliesAndReset(any(ReplyProcessor.<strong class="jxr_keyword">class</strong>), any(RetryProcessor.<strong class="jxr_keyword">class</strong>), eq(false));
-<a class="jxr_linenumber" name="169" href="#169">169</a>
-<a class="jxr_linenumber" name="170" href="#170">170</a> }
-<a class="jxr_linenumber" name="171" href="#171">171</a>
-<a class="jxr_linenumber" name="172" href="#172">172</a> @Test
-<a class="jxr_linenumber" name="173" href="#173">173</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitTableExceptionOnCommitPersistenceTakesDownDaemon() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="130" href="#130">130</a> }
+<a class="jxr_linenumber" name="131" href="#131">131</a>
+<a class="jxr_linenumber" name="132" href="#132">132</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="133" href="#133">133</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithSingleCommitTableWriter() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="134" href="#134">134</a>
+<a class="jxr_linenumber" name="135" href="#135">135</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 1;
+<a class="jxr_linenumber" name="136" href="#136">136</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 2;
+<a class="jxr_linenumber" name="137" href="#137">137</a>
+<a class="jxr_linenumber" name="138" href="#138">138</a> <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="139" href="#139">139</a> VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="140" href="#140">140</a> mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="141" href="#141">141</a>
+<a class="jxr_linenumber" name="142" href="#142">142</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="143" href="#143">143</a> tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="144" href="#144">144</a> tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="145" href="#145">145</a>
+<a class="jxr_linenumber" name="146" href="#146">146</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="147" href="#147">147</a>
+<a class="jxr_linenumber" name="148" href="#148">148</a> ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="149" href="#149">149</a>
+<a class="jxr_linenumber" name="150" href="#150">150</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="151" href="#151">151</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="152" href="#152">152</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics, <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="153" href="#153">153</a> leaseManager,
+<a class="jxr_linenumber" name="154" href="#154">154</a> commitTable,
+<a class="jxr_linenumber" name="155" href="#155">155</a> replyProcessor,
+<a class="jxr_linenumber" name="156" href="#156">156</a> retryProcessor,
+<a class="jxr_linenumber" name="157" href="#157">157</a> panicker);
+<a class="jxr_linenumber" name="158" href="#158">158</a> }
+<a class="jxr_linenumber" name="159" href="#159">159</a>
+<a class="jxr_linenumber" name="160" href="#160">160</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="161" href="#161">161</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="162" href="#162">162</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="163" href="#163">163</a>
+<a class="jxr_linenumber" name="164" href="#164">164</a> verify(batchPool, times(1)).borrowObject(); <em class="jxr_comment">// Called during initialization</em>
+<a class="jxr_linenumber" name="165" href="#165">165</a>
+<a class="jxr_linenumber" name="166" href="#166">166</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="167" href="#167">167</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// Flush: batch full</em>
+<a class="jxr_linenumber" name="168" href="#168">168</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="169" href="#169">169</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// Flush: batch full</em>
+<a class="jxr_linenumber" name="170" href="#170">170</a>
+<a class="jxr_linenumber" name="171" href="#171">171</a> verify(batchPool, times(1 + BATCH_SIZE_PER_CT_WRITER)).borrowObject(); <em class="jxr_comment">// 3: 1 in init + 2 when flushing</em>
+<a class="jxr_linenumber" name="172" href="#172">172</a>
+<a class="jxr_linenumber" name="173" href="#173">173</a> }
<a class="jxr_linenumber" name="174" href="#174">174</a>
-<a class="jxr_linenumber" name="175" href="#175">175</a> <em class="jxr_comment">// Init lease management (doesn't matter if HA or not)</em>
-<a class="jxr_linenumber" name="176" href="#176">176</a> LeaseManagement leaseManager = mock(LeaseManagement.<strong class="jxr_keyword">class</strong>);
-<a class="jxr_linenumber" name="177" href="#177">177</a> PersistenceProcessor proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(<strong class="jxr_keyword">new</strong> TSOServerConfig(),
-<a class="jxr_linenumber" name="178" href="#178">178</a> metrics,
-<a class="jxr_linenumber" name="179" href="#179">179</a> <span class="jxr_string">"localhost:1234"</span>,
-<a class="jxr_linenumber" name="180" href="#180">180</a> leaseManager,
-<a class="jxr_linenumber" name="181" href="#181">181</a> commitTable,
-<a class="jxr_linenumber" name="182" href="#182">182</a> mock(ReplyProcessor.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="183" href="#183">183</a> mock(RetryProcessor.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="184" href="#184">184</a> panicker);
-<a class="jxr_linenumber" name="185" href="#185">185</a> MonitoringContext monCtx = <strong class="jxr_keyword">new</strong> MonitoringContext(metrics);
-<a class="jxr_linenumber" name="186" href="#186">186</a>
-<a class="jxr_linenumber" name="187" href="#187">187</a> <em class="jxr_comment">// Configure lease manager to work normally</em>
-<a class="jxr_linenumber" name="188" href="#188">188</a> doReturn(<strong class="jxr_keyword">true</strong>).when(leaseManager).stillInLeasePeriod();
-<a class="jxr_linenumber" name="189" href="#189">189</a>
-<a class="jxr_linenumber" name="190" href="#190">190</a> <em class="jxr_comment">// Configure commit table writer to explode when flushing changes to DB</em>
-<a class="jxr_linenumber" name="191" href="#191">191</a> doThrow(<strong class="jxr_keyword">new</strong> IOException(<span class="jxr_string">"Unable to write@TestPersistenceProcessor2"</span>)).when(mockWriter).flush();
+<a class="jxr_linenumber" name="175" href="#175">175</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="176" href="#176">176</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithMultipleCommitTableWriters() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="177" href="#177">177</a>
+<a class="jxr_linenumber" name="178" href="#178">178</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 2;
+<a class="jxr_linenumber" name="179" href="#179">179</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 2;
+<a class="jxr_linenumber" name="180" href="#180">180</a>
+<a class="jxr_linenumber" name="181" href="#181">181</a> <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="182" href="#182">182</a> VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="183" href="#183">183</a> mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="184" href="#184">184</a>
+<a class="jxr_linenumber" name="185" href="#185">185</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="186" href="#186">186</a> tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="187" href="#187">187</a> tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="188" href="#188">188</a>
+<a class="jxr_linenumber" name="189" href="#189">189</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="190" href="#190">190</a>
+<a class="jxr_linenumber" name="191" href="#191">191</a> ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(metrics, panicker, batchPool);
<a class="jxr_linenumber" name="192" href="#192">192</a>
-<a class="jxr_linenumber" name="193" href="#193">193</a> <em class="jxr_comment">// Check the panic is extended!</em>
-<a class="jxr_linenumber" name="194" href="#194">194</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="195" href="#195">195</a> verify(panicker, timeout(1000).atLeastOnce()).panic(anyString(), any(Throwable.<strong class="jxr_keyword">class</strong>));
-<a class="jxr_linenumber" name="196" href="#196">196</a>
-<a class="jxr_linenumber" name="197" href="#197">197</a> }
-<a class="jxr_linenumber" name="198" href="#198">198</a>
-<a class="jxr_linenumber" name="199" href="#199">199</a> @Test
-<a class="jxr_linenumber" name="200" href="#200">200</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testRuntimeExceptionOnCommitPersistenceTakesDownDaemon() <strong class="jxr_keyword">throws</strong> Exception {
-<a class="jxr_linenumber" name="201" href="#201">201</a>
-<a class="jxr_linenumber" name="202" href="#202">202</a> PersistenceProcessor proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(<strong class="jxr_keyword">new</strong> TSOServerConfig(),
-<a class="jxr_linenumber" name="203" href="#203">203</a> metrics,
-<a class="jxr_linenumber" name="204" href="#204">204</a> <span class="jxr_string">"localhost:1234"</span>,
-<a class="jxr_linenumber" name="205" href="#205">205</a> mock(LeaseManagement.<strong class="jxr_keyword">class</strong>),
-<a class="jxr_linenumber" name="206" href="#206">206</a> commitTable,
-<a class="jxr_linenumber" name="207" href="#207">207</a> replyProcessor,
-<a class="jxr_linenumber" name="208" href="#208">208</a> retryProcessor,
-<a class="jxr_linenumber" name="209" href="#209">209</a> panicker);
-<a class="jxr_linenumber" name="210" href="#210">210</a>
-<a class="jxr_linenumber" name="211" href="#211">211</a> <em class="jxr_comment">// Configure writer to explode with a runtime exception</em>
-<a class="jxr_linenumber" name="212" href="#212">212</a> doThrow(<strong class="jxr_keyword">new</strong> RuntimeException(<span class="jxr_string">"Kaboom!"</span>)).when(mockWriter).addCommittedTransaction(anyLong(), anyLong());
-<a class="jxr_linenumber" name="213" href="#213">213</a> MonitoringContext monCtx = <strong class="jxr_keyword">new</strong> MonitoringContext(metrics);
-<a class="jxr_linenumber" name="214" href="#214">214</a>
-<a class="jxr_linenumber" name="215" href="#215">215</a> <em class="jxr_comment">// Check the panic is extended!</em>
-<a class="jxr_linenumber" name="216" href="#216">216</a> proc.persistCommit(1, 2, <strong class="jxr_keyword">null</strong>, monCtx);
-<a class="jxr_linenumber" name="217" href="#217">217</a> verify(panicker, timeout(1000).atLeastOnce()).panic(anyString(), any(Throwable.<strong class="jxr_keyword">class</strong>));
-<a class="jxr_linenumber" name="218" href="#218">218</a>
-<a class="jxr_linenumber" name="219" href="#219">219</a> }
-<a class="jxr_linenumber" name="220" href="#220">220</a>
-<a class="jxr_linenumber" name="221" href="#221">221</a> }
+<a class="jxr_linenumber" name="193" href="#193">193</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="194" href="#194">194</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="195" href="#195">195</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="196" href="#196">196</a> <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="197" href="#197">197</a> leaseManager,
+<a class="jxr_linenumber" name="198" href="#198">198</a> commitTable,
+<a class="jxr_linenumber" name="199" href="#199">199</a> replyProcessor,
+<a class="jxr_linenumber" name="200" href="#200">200</a> retryProcessor,
+<a class="jxr_linenumber" name="201" href="#201">201</a> panicker);
+<a class="jxr_linenumber" name="202" href="#202">202</a> }
+<a class="jxr_linenumber" name="203" href="#203">203</a>
+<a class="jxr_linenumber" name="204" href="#204">204</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="205" href="#205">205</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="206" href="#206">206</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="207" href="#207">207</a>
+<a class="jxr_linenumber" name="208" href="#208">208</a> verify(batchPool, times(1)).borrowObject(); <em class="jxr_comment">// Called during initialization</em>
+<a class="jxr_linenumber" name="209" href="#209">209</a>
+<a class="jxr_linenumber" name="210" href="#210">210</a> <em class="jxr_comment">// Fill 1st handler Batches completely</em>
+<a class="jxr_linenumber" name="211" href="#211">211</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="212" href="#212">212</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// 1st batch full</em>
+<a class="jxr_linenumber" name="213" href="#213">213</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="214" href="#214">214</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="215" href="#215">215</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// 2nd batch full</em>
+<a class="jxr_linenumber" name="216" href="#216">216</a> verify(batchPool, times(3)).borrowObject();
+<a class="jxr_linenumber" name="217" href="#217">217</a>
+<a class="jxr_linenumber" name="218" href="#218">218</a> <em class="jxr_comment">// Test empty flush does not trigger response in getting a new currentBatch</em>
+<a class="jxr_linenumber" name="219" href="#219">219</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="220" href="#220">220</a> verify(batchPool, times(3)).borrowObject();
+<a class="jxr_linenumber" name="221" href="#221">221</a>
+<a class="jxr_linenumber" name="222" href="#222">222</a> <em class="jxr_comment">// Fill 2nd handler Batches completely</em>
+<a class="jxr_linenumber" name="223" href="#223">223</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="224" href="#224">224</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// 1st batch full</em>
+<a class="jxr_linenumber" name="225" href="#225">225</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="226" href="#226">226</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// 2nd batch full</em>
+<a class="jxr_linenumber" name="227" href="#227">227</a> verify(batchPool, times(1 + (NUM_CT_WRITERS * BATCH_SIZE_PER_CT_WRITER))).borrowObject();
+<a class="jxr_linenumber" name="228" href="#228">228</a>
+<a class="jxr_linenumber" name="229" href="#229">229</a> <em class="jxr_comment">// Start filling a new currentBatch and flush it immediately</em>
+<a class="jxr_linenumber" name="230" href="#230">230</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>)); <em class="jxr_comment">// Batch not full</em>
+<a class="jxr_linenumber" name="231" href="#231">231</a> verify(batchPool, times(5)).borrowObject();
+<a class="jxr_linenumber" name="232" href="#232">232</a> proc.triggerCurrentBatchFlush(); <em class="jxr_comment">// Flushing should provoke invocation of a new batch</em>
+<a class="jxr_linenumber" name="233" href="#233">233</a> verify(batchPool, times(6)).borrowObject();
+<a class="jxr_linenumber" name="234" href="#234">234</a>
+<a class="jxr_linenumber" name="235" href="#235">235</a> <em class="jxr_comment">// Test empty flush does not trigger response</em>
+<a class="jxr_linenumber" name="236" href="#236">236</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="237" href="#237">237</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="238" href="#238">238</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="239" href="#239">239</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="240" href="#240">240</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="241" href="#241">241</a> verify(batchPool, times(6)).borrowObject();
+<a class="jxr_linenumber" name="242" href="#242">242</a>
+<a class="jxr_linenumber" name="243" href="#243">243</a> }
+<a class="jxr_linenumber" name="244" href="#244">244</a>
+<a class="jxr_linenumber" name="245" href="#245">245</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="246" href="#246">246</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithNonHALeaseManager() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="247" href="#247">247</a>
+<a class="jxr_linenumber" name="248" href="#248">248</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 1;
+<a class="jxr_linenumber" name="249" href="#249">249</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 1;
+<a class="jxr_linenumber" name="250" href="#250">250</a>
+<a class="jxr_linenumber" name="251" href="#251">251</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="252" href="#252">252</a> tsoConfig.setBatchSizePerCTWriter(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="253" href="#253">253</a> tsoConfig.setNumConcurrentCTWriters(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="254" href="#254">254</a> tsoConfig.setBatchPersistTimeoutInMs(100);
+<a class="jxr_linenumber" name="255" href="#255">255</a>
+<a class="jxr_linenumber" name="256" href="#256">256</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="257" href="#257">257</a>
+<a class="jxr_linenumber" name="258" href="#258">258</a> ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="259" href="#259">259</a>
+<a class="jxr_linenumber" name="260" href="#260">260</a> <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="261" href="#261">261</a> VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="262" href="#262">262</a> mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="263" href="#263">263</a>
+<a class="jxr_linenumber" name="264" href="#264">264</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="265" href="#265">265</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="266" href="#266">266</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="267" href="#267">267</a> <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="268" href="#268">268</a> leaseManager,
+<a class="jxr_linenumber" name="269" href="#269">269</a> commitTable,
+<a class="jxr_linenumber" name="270" href="#270">270</a> replyProcessor,
+<a class="jxr_linenumber" name="271" href="#271">271</a> retryProcessor,
+<a class="jxr_linenumber" name="272" href="#272">272</a> panicker);
+<a class="jxr_linenumber" name="273" href="#273">273</a> }
+<a class="jxr_linenumber" name="274" href="#274">274</a>
+<a class="jxr_linenumber" name="275" href="#275">275</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="276" href="#276">276</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="277" href="#277">277</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="278" href="#278">278</a>
+<a class="jxr_linenumber" name="279" href="#279">279</a> <em class="jxr_comment">// The non-ha lease manager always return true for</em>
+<a class="jxr_linenumber" name="280" href="#280">280</a> <em class="jxr_comment">// stillInLeasePeriod(), so verify the currentBatch sends replies as master</em>
+<a class="jxr_linenumber" name="281" href="#281">281</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="282" href="#282">282</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="283" href="#283">283</a> verify(leaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="284" href="#284">284</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="285" href="#285">285</a>
+<a class="jxr_linenumber" name="286" href="#286">286</a> }
+<a class="jxr_linenumber" name="287" href="#287">287</a>
+<a class="jxr_linenumber" name="288" href="#288">288</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="289" href="#289">289</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithHALeaseManagerAndMinimumCommitTableWriters() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="290" href="#290">290</a>
+<a class="jxr_linenumber" name="291" href="#291">291</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_PERSIST_HANDLERS = 2; <em class="jxr_comment">// Minimum commit table writers is 2</em>
+<a class="jxr_linenumber" name="292" href="#292">292</a>
+<a class="jxr_linenumber" name="293" href="#293">293</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="294" href="#294">294</a> tsoConfig.setNumConcurrentCTWriters(NUM_PERSIST_HANDLERS);
+<a class="jxr_linenumber" name="295" href="#295">295</a>
+<a class="jxr_linenumber" name="296" href="#296">296</a> testPersistenceWithHALeaseManagerPreservingLease(tsoConfig);
+<a class="jxr_linenumber" name="297" href="#297">297</a> testPersistenceWithHALeaseManagerFailingToPreserveLease1(tsoConfig);
+<a class="jxr_linenumber" name="298" href="#298">298</a> testPersistenceWithHALeaseManagerFailingToPreserveLease2(tsoConfig);
+<a class="jxr_linenumber" name="299" href="#299">299</a> testPersistenceWithHALeaseManagerFailingToPreserveLease3(tsoConfig);
+<a class="jxr_linenumber" name="300" href="#300">300</a>
+<a class="jxr_linenumber" name="301" href="#301">301</a> }
+<a class="jxr_linenumber" name="302" href="#302">302</a>
+<a class="jxr_linenumber" name="303" href="#303">303</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="304" href="#304">304</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitPersistenceWithHALeaseManagerAndMultipleCommitTableWriters() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="305" href="#305">305</a>
+<a class="jxr_linenumber" name="306" href="#306">306</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 4;
+<a class="jxr_linenumber" name="307" href="#307">307</a> <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 4;
+<a class="jxr_linenumber" name="308" href="#308">308</a>
+<a class="jxr_linenumber" name="309" href="#309">309</a> TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="310" href="#310">310</a> tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="311" href="#311">311</a> tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="312" href="#312">312</a> tsoConfig.setBatchPersistTimeoutInMs(100);
+<a class="jxr_linenumber" name="313" href="#313">313</a>
+<a class="jxr_linenumber" name="314" href="#314">314</a> testPersistenceWithHALeaseManagerPreservingLease(tsoConfig);
+<a class="jxr_linenumber" name="315" href="#315">315</a> testPersistenceWithHALeaseManagerFailingToPreserveLease1(tsoConfig);
+<a class="jxr_linenumber" name="316" href="#316">316</a> testPersistenceWithHALeaseManagerFailingToPreserveLease2(tsoConfig);
+<a class="jxr_linenumber" name="317" href="#317">317</a> testPersistenceWithHALeaseManagerFailingToPreserveLease3(tsoConfig);
+<a class="jxr_linenumber" name="318" href="#318">318</a>
+<a class="jxr_linenumber" name="319" href="#319">319</a> }
+<a class="jxr_linenumber" name="320" href="#320">320</a>
+<a class="jxr_linenumber" name="321" href="#321">321</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> testPersistenceWithHALeaseManagerPreservingLease(TSOServerConfig tsoConfig) <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="322" href="#322">322</a>
+<a class="jxr_linenumber" name="323" href="#323">323</a> <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="324" href="#324">324</a> LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="325" href="#325">325</a>
+<a class="jxr_linenumber" name="326" href="#326">326</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="327" href="#327">327</a>
+<a class="jxr_linenumber" name="328" href="#328">328</a> PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="329" href="#329">329</a>
+<a class="jxr_linenumber" name="330" href="#330">330</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="331" href="#331">331</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="332" href="#332">332</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="333" href="#333">333</a>
+<a class="jxr_linenumber" name="334" href="#334">334</a> <em class="jxr_comment">// Test: Configure the lease manager to return true always</em>
+<a class="jxr_linenumber" name="335" href="#335">335</a> doReturn(<strong class="jxr_keyword">true</strong>).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="336" href="#336">336</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="337" href="#337">337</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="338" href="#338">338</a> verify(simulatedHALeaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="339" href="#339">339</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="340" href="#340">340</a> }
+<a class="jxr_linenumber" name="341" href="#341">341</a>
+<a class="jxr_linenumber" name="342" href="#342">342</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> testPersistenceWithHALeaseManagerFailingToPreserveLease1(TSOServerConfig tsoConfig) <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="343" href="#343">343</a>
+<a class="jxr_linenumber" name="344" href="#344">344</a> <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="345" href="#345">345</a> LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="346" href="#346">346</a>
+<a class="jxr_linenumber" name="347" href="#347">347</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="348" href="#348">348</a>
+<a class="jxr_linenumber" name="349" href="#349">349</a> PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="350" href="#350">350</a>
+<a class="jxr_linenumber" name="351" href="#351">351</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="352" href="#352">352</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="353" href="#353">353</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="354" href="#354">354</a>
+<a class="jxr_linenumber" name="355" href="#355">355</a> <em class="jxr_comment">// Test: Configure the lease manager to return true first and false later for stillInLeasePeriod</em>
+<a class="jxr_linenumber" name="356" href="#356">356</a> doReturn(<strong class="jxr_keyword">true</strong>).doReturn(false).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="357" href="#357">357</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="358" href="#358">358</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="359" href="#359">359</a> verify(simulatedHALeaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="360" href="#360">360</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="361" href="#361">361</a> }
+<a class="jxr_linenumber" name="362" href="#362">362</a>
+<a class="jxr_linenumber" name="363" href="#363">363</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> testPersistenceWithHALeaseManagerFailingToPreserveLease2(TSOServerConfig tsoConfig) <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="364" href="#364">364</a>
+<a class="jxr_linenumber" name="365" href="#365">365</a> <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="366" href="#366">366</a> LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="367" href="#367">367</a>
+<a class="jxr_linenumber" name="368" href="#368">368</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="369" href="#369">369</a>
+<a class="jxr_linenumber" name="370" href="#370">370</a> PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="371" href="#371">371</a>
+<a class="jxr_linenumber" name="372" href="#372">372</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="373" href="#373">373</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="374" href="#374">374</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="375" href="#375">375</a>
+<a class="jxr_linenumber" name="376" href="#376">376</a> <em class="jxr_comment">// Test: Configure the lease manager to return false for stillInLeasePeriod</em>
+<a class="jxr_linenumber" name="377" href="#377">377</a> doReturn(false).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="378" href="#378">378</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="379" href="#379">379</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="380" href="#380">380</a> verify(simulatedHALeaseManager, timeout(1000).times(1)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="381" href="#381">381</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="382" href="#382">382</a> }
+<a class="jxr_linenumber" name="383" href="#383">383</a>
+<a class="jxr_linenumber" name="384" href="#384">384</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> testPersistenceWithHALeaseManagerFailingToPreserveLease3(TSOServerConfig tsoConfig) <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="385" href="#385">385</a>
+<a class="jxr_linenumber" name="386" href="#386">386</a> <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="387" href="#387">387</a> LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="388" href="#388">388</a>
+<a class="jxr_linenumber" name="389" href="#389">389</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="390" href="#390">390</a>
+<a class="jxr_linenumber" name="391" href="#391">391</a> PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="392" href="#392">392</a>
+<a class="jxr_linenumber" name="393" href="#393">393</a> <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="394" href="#394">394</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, commitTable, batchPool,
+<a class="jxr_linenumber" name="395" href="#395">395</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="396" href="#396">396</a>
+<a class="jxr_linenumber" name="397" href="#397">397</a> <em class="jxr_comment">// Test: Configure the lease manager to return true first and false later for stillInLeasePeriod and raise</em>
+<a class="jxr_linenumber" name="398" href="#398">398</a> <em class="jxr_comment">// an exception when flush</em>
+<a class="jxr_linenumber" name="399" href="#399">399</a> <em class="jxr_comment">// Configure mock writer to flush unsuccessfully</em>
+<a class="jxr_linenumber" name="400" href="#400">400</a> doThrow(<strong class="jxr_keyword">new</strong> IOException(<span class="jxr_string">"Unable to write"</span>)).when(mockWriter).flush();
+<a class="jxr_linenumber" name="401" href="#401">401</a> doReturn(<strong class="jxr_keyword">true</strong>).doReturn(false).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="402" href="#402">402</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), mock(MonitoringContext.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="403" href="#403">403</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="404" href="#404">404</a> verify(simulatedHALeaseManager, timeout(1000).times(1)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="405" href="#405">405</a> verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="406" href="#406">406</a>
+<a class="jxr_linenumber" name="407" href="#407">407</a> }
+<a class="jxr_linenumber" name="408" href="#408">408</a>
+<a class="jxr_linenumber" name="409" href="#409">409</a> <strong class="jxr_keyword">private</strong> PersistenceProcessorHandler[] configureHandlers(TSOServerConfig tsoConfig,
+<a class="jxr_linenumber" name="410" href="#410">410</a> LeaseManager leaseManager,
+<a class="jxr_linenumber" name="411" href="#411">411</a> ObjectPool<Batch> batchPool)
+<a class="jxr_linenumber" name="412" href="#412">412</a> <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="413" href="#413">413</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="414" href="#414">414</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="415" href="#415">415</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="416" href="#416">416</a> <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="417" href="#417">417</a> leaseManager,
+<a class="jxr_linenumber" name="418" href="#418">418</a> commitTable,
+<a class="jxr_linenumber" name="419" href="#419">419</a> <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(metrics, panicker, batchPool),
+<a class="jxr_linenumber" name="420" href="#420">420</a> retryProcessor,
+<a class="jxr_linenumber" name="421" href="#421">421</a> <strong class="jxr_keyword">new</strong> RuntimeExceptionPanicker());
+<a class="jxr_linenumber" name="422" href="#422">422</a> }
+<a class="jxr_linenumber" name="423" href="#423">423</a> <strong class="jxr_keyword">return</strong> handlers;
+<a class="jxr_linenumber" name="424" href="#424">424</a> }
+<a class="jxr_linenumber" name="425" href="#425">425</a>
+<a class="jxr_linenumber" name="426" href="#426">426</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="427" href="#427">427</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> testCommitTableExceptionOnCommitPersistenceTakesDownDaemon() <strong class="jxr_keyword">throws</strong> Exception {
+<a class="jxr_linenumber" name="428" href="#428">428</a>
+<a class="jxr_linenumber" name="429" href="#429">429</a> <em class="jxr_comment">// Init lease management (doesn't matter if HA or not)</em>
+<a class="jxr_linenumber" name="430" href="#430">430</a> LeaseManagement leaseManager = mock(LeaseManagement.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="431" href="#431">431</a>
+<a class="jxr_linenumber" name="432" href="#432">432</a> TSOServerConfig config = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="433" href="#433">433</a>
+<a class="jxr_linenumber" name="434" href="#434">434</a> ObjectPool<Batch> batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(config).getBatchPool());
+<a class="jxr_linenumber" name="435" href="#435">435</a>
+<a class="jxr_linenumber" name="436" href="#436">436</a> ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="437" href="#437">437</a>
+<a class="jxr_linenumber" name="438" href="#438">438</a> PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[config.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="439" href="#439">439</a> <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i < config.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="440" href="#440">440</a> handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="441" href="#441">441</a> <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="442" href="#442">442</a> leaseManager,
+<a class="jxr_linenumber" name="443" href="#443">443</a> commitTable,
+<a class="jxr_linenumber" name="444" href="#444">444</a> replyProcessor,
+<a class="jxr_linenumber" name="445" href="#445">445</a> mock(RetryProcessor.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="446" href="#446">446</a> panicker);
+<a class="jxr_linenumber" name="447" href="#447">447</a> }
+<a class="jxr_linenumber" name="448" href="#448">448</a>
+<a class="jxr_linenumber" name="449" href="#449">449</a> PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(config, commitTable, batchPool,
+<a class="jxr_linenumber" name="450" href="#450">450</a> panicker, handlers, metrics);
+<a class="jxr_linenumber" name="451" href="#451">451</a>
+<a class="jxr_linenumber" name="452" href="#452">452</a> MonitoringContext monCtx = <strong class="jxr_keyword">new</strong> MonitoringContext(metrics);
+<a class="jxr_linenumber" name="453" href="#453">453</a>
+<a class="jxr_linenumber" name="454" href="#454">454</a> <em class="jxr_comment">// Configure lease manager to work normally</em>
+<a class="jxr_linenumber" name="455" href="#455">455</a> doReturn(<strong class="jxr_keyword">true</strong>).when(leaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="456" href="#456">456</a>
+<a class="jxr_linenumber" name="457" href="#457">457</a> <em class="jxr_comment">// Configure commit table writer to explode when flushing changes to DB</em>
+<a class="jxr_linenumber" name="458" href="#458">458</a> doThrow(<strong class="jxr_keyword">new</strong> IOException(<span class="jxr_string">"Unable to write@TestPersistenceProcessor2"</span>)).when(mockWriter).flush();
+<a class="jxr_linenumber" name="459" href="#459">459</a>
+<a class="jxr_linenumber" name="460" href="#460">460</a> <em class="jxr_comment">// Check the panic is extended!</em>
+<a class="jxr_linenumber" name="461" href="#461">461</a> proc.addCommitToBatch(ANY_ST, ANY_CT, mock(Channel.<strong class="jxr_keyword">class</strong>), monCtx);
+<a class="jxr_linenumber" name="462" href="#462">462</a> proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="463" href="#463">463</a> verify(panicker, timeout(1000).atLeastOnce()).panic(anyString(), any(Throwable.<strong class="jxr_keyword">class</strong>));
+<a class="jxr_linenumber" name="464" href="#464">464</a>
+<a class="jxr_linenumber" name="465" href="#465">465</a> }
+<a class="jxr_linenumber" name="466" href="#466">466</a>
+<a class="jxr_linenumber" name="467" href="#467">467</a> @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="468" href="#468">468</a> <strong class="jxr_keyword">pu
<TRUNCATED>