You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@omid.apache.org by oh...@apache.org on 2018/07/23 12:05:08 UTC

[24/34] incubator-omid-site git commit: [wagon-git] commit to branch asf-site 7/23/18 3:04 PM

http://git-wip-us.apache.org/repos/asf/incubator-omid-site/blob/dc01c0be/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 4ee07b1..19e6713 100644
--- a/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
+++ b/xref-test/org/apache/omid/tso/TestPersistenceProcessor.html
@@ -42,473 +42,476 @@
 <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="35" href="#35">35</a>  <strong class="jxr_keyword">import</strong> com.lmax.disruptor.BlockingWaitStrategy;
 <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="37" href="#37">37</a>  <strong class="jxr_keyword">import</strong> java.io.IOException;
+<a class="jxr_linenumber" name="38" href="#38">38</a>  
+<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.any;
+<a class="jxr_linenumber" name="40" href="#40">40</a>  <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyLong;
+<a class="jxr_linenumber" name="41" href="#41">41</a>  <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Matchers.anyString;
+<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.doReturn;
+<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.doThrow;
+<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.mock;
+<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.spy;
+<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.timeout;
+<a class="jxr_linenumber" name="47" href="#47">47</a>  <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.times;
+<a class="jxr_linenumber" name="48" href="#48">48</a>  <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.mockito.Mockito.verify;
+<a class="jxr_linenumber" name="49" href="#49">49</a>  <strong class="jxr_keyword">import</strong> <strong class="jxr_keyword">static</strong> org.testng.Assert.assertEquals;
+<a class="jxr_linenumber" name="50" href="#50">50</a>  
+<a class="jxr_linenumber" name="51" href="#51">51</a>  <em class="jxr_comment">// TODO Refactor: Make visible currentBatch in PersistenceProcessorImpl to add proper verifications</em>
+<a class="jxr_linenumber" name="52" href="#52">52</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="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> CommitTable.Writer mockWriter;
+<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> Logger LOG = LoggerFactory.getLogger(TestPersistenceProcessor.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="55" href="#55">55</a>  
+<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">long</strong> ANY_LWM = 1234L;
+<a class="jxr_linenumber" name="57" href="#57">57</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="58" href="#58">58</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="59" href="#59">59</a>  
 <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="61" href="#61">61</a>      <strong class="jxr_keyword">private</strong> CommitTable.Writer mockWriter;
 <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="63" href="#63">63</a>      <strong class="jxr_keyword">private</strong> CommitTable.Client mockClient;
 <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>      <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="65" href="#65">65</a>      <strong class="jxr_keyword">private</strong> RetryProcessor retryProcessor;
+<a class="jxr_linenumber" name="66" href="#66">66</a>      @Mock
+<a class="jxr_linenumber" name="67" href="#67">67</a>      <strong class="jxr_keyword">private</strong> Panicker panicker;
+<a class="jxr_linenumber" name="68" href="#68">68</a>  
+<a class="jxr_linenumber" name="69" href="#69">69</a>      <strong class="jxr_keyword">private</strong> MetricsRegistry metrics;
+<a class="jxr_linenumber" name="70" href="#70">70</a>      <strong class="jxr_keyword">private</strong> CommitTable commitTable;
+<a class="jxr_linenumber" name="71" href="#71">71</a>  
+<a class="jxr_linenumber" name="72" href="#72">72</a>      @BeforeMethod(alwaysRun = <strong class="jxr_keyword">true</strong>, timeOut = 30_000)
+<a class="jxr_linenumber" name="73" href="#73">73</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="74" href="#74">74</a>  
-<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="75" href="#75">75</a>          MockitoAnnotations.initMocks(<strong class="jxr_keyword">this</strong>);
+<a class="jxr_linenumber" name="76" href="#76">76</a>  
+<a class="jxr_linenumber" name="77" href="#77">77</a>          <em class="jxr_comment">// Configure null metrics provider</em>
+<a class="jxr_linenumber" name="78" href="#78">78</a>          metrics = <strong class="jxr_keyword">new</strong> NullMetricsProvider();
+<a class="jxr_linenumber" name="79" href="#79">79</a>  
+<a class="jxr_linenumber" name="80" href="#80">80</a>          <em class="jxr_comment">// Configure commit table to return the mocked writer and client</em>
+<a class="jxr_linenumber" name="81" href="#81">81</a>          commitTable = <strong class="jxr_keyword">new</strong> CommitTable() {
+<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> Writer getWriter() {
+<a class="jxr_linenumber" name="84" href="#84">84</a>                  <strong class="jxr_keyword">return</strong> mockWriter;
+<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>              @Override
+<a class="jxr_linenumber" name="88" href="#88">88</a>              <strong class="jxr_keyword">public</strong> Client getClient() {
+<a class="jxr_linenumber" name="89" href="#89">89</a>                  <strong class="jxr_keyword">return</strong> mockClient;
+<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 tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="93" href="#93">93</a>      }
+<a class="jxr_linenumber" name="94" href="#94">94</a>  
+<a class="jxr_linenumber" name="95" href="#95">95</a>      @AfterMethod
+<a class="jxr_linenumber" name="96" href="#96">96</a>      <strong class="jxr_keyword">void</strong> afterMethod() {
+<a class="jxr_linenumber" name="97" href="#97">97</a>          Mockito.reset(mockWriter);
+<a class="jxr_linenumber" name="98" href="#98">98</a>      }
+<a class="jxr_linenumber" name="99" href="#99">99</a>  
+<a class="jxr_linenumber" name="100" href="#100">100</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="101" href="#101">101</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="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 &lt; 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">// 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&lt;Long&gt; 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>     }
-<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="103" href="#103">103</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="104" href="#104">104</a> 
+<a class="jxr_linenumber" name="105" href="#105">105</a>         PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="106" href="#106">106</a>         <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i &lt; tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="107" href="#107">107</a>             handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="108" href="#108">108</a>                                                           <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="109" href="#109">109</a>                                                           mock(LeaseManager.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="110" href="#110">110</a>                                                           commitTable,
+<a class="jxr_linenumber" name="111" href="#111">111</a>                                                           mock(ReplyProcessor.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="112" href="#112">112</a>                                                           retryProcessor,
+<a class="jxr_linenumber" name="113" href="#113">113</a>                                                           panicker);
+<a class="jxr_linenumber" name="114" href="#114">114</a>         }
+<a class="jxr_linenumber" name="115" href="#115">115</a> 
+<a class="jxr_linenumber" name="116" href="#116">116</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="117" href="#117">117</a>         PersistenceProcessorImpl persistenceProcessor =
+<a class="jxr_linenumber" name="118" href="#118">118</a>                 <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig,
+<a class="jxr_linenumber" name="119" href="#119">119</a>                                              <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(),
+<a class="jxr_linenumber" name="120" href="#120">120</a>                                              commitTable,
+<a class="jxr_linenumber" name="121" href="#121">121</a>                                              mock(ObjectPool.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="122" href="#122">122</a>                                              panicker,
+<a class="jxr_linenumber" name="123" href="#123">123</a>                                              handlers,
+<a class="jxr_linenumber" name="124" href="#124">124</a>                                              metrics);
+<a class="jxr_linenumber" name="125" href="#125">125</a> 
+<a class="jxr_linenumber" name="126" href="#126">126</a>         persistenceProcessor.persistLowWatermark(ANY_LWM).get();
+<a class="jxr_linenumber" name="127" href="#127">127</a> 
+<a class="jxr_linenumber" name="128" href="#128">128</a>         ArgumentCaptor&lt;Long&gt; lwmCapture = ArgumentCaptor.forClass(Long.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="129" href="#129">129</a>         CommitTable.Writer lwmWriter = commitTable.getWriter();
+<a class="jxr_linenumber" name="130" href="#130">130</a>         verify(lwmWriter, timeout(100).times(1)).updateLowWatermark(lwmCapture.capture());
+<a class="jxr_linenumber" name="131" href="#131">131</a>         assertEquals(lwmCapture.getValue().longValue(), ANY_LWM);
+<a class="jxr_linenumber" name="132" href="#132">132</a> 
+<a class="jxr_linenumber" name="133" href="#133">133</a>     }
 <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="135" href="#135">135</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="136" href="#136">136</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="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&lt;Batch&gt; 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 &lt; 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>     @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="138" href="#138">138</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 1;
+<a class="jxr_linenumber" name="139" href="#139">139</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 2;
+<a class="jxr_linenumber" name="140" href="#140">140</a> 
+<a class="jxr_linenumber" name="141" href="#141">141</a>         <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="142" href="#142">142</a>         VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="143" href="#143">143</a>                                                                  mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="144" href="#144">144</a> 
+<a class="jxr_linenumber" name="145" href="#145">145</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="146" href="#146">146</a>         tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="147" href="#147">147</a>         tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="148" href="#148">148</a> 
+<a class="jxr_linenumber" name="149" href="#149">149</a>         ObjectPool&lt;Batch&gt; batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="150" href="#150">150</a> 
+<a class="jxr_linenumber" name="151" href="#151">151</a>         ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(<strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="152" href="#152">152</a> 
+<a class="jxr_linenumber" name="153" href="#153">153</a>         PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="154" href="#154">154</a>         <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i &lt; tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="155" href="#155">155</a>             handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics, <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="156" href="#156">156</a>                                                           leaseManager,
+<a class="jxr_linenumber" name="157" href="#157">157</a>                                                           commitTable,
+<a class="jxr_linenumber" name="158" href="#158">158</a>                                                           replyProcessor,
+<a class="jxr_linenumber" name="159" href="#159">159</a>                                                           retryProcessor,
+<a class="jxr_linenumber" name="160" href="#160">160</a>                                                           panicker);
+<a class="jxr_linenumber" name="161" href="#161">161</a>         }
+<a class="jxr_linenumber" name="162" href="#162">162</a> 
+<a class="jxr_linenumber" name="163" href="#163">163</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="164" href="#164">164</a>         PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), commitTable, batchPool,
+<a class="jxr_linenumber" name="165" href="#165">165</a>                                                                      panicker, handlers, metrics);
+<a class="jxr_linenumber" name="166" href="#166">166</a> 
+<a class="jxr_linenumber" name="167" href="#167">167</a>         verify(batchPool, times(1)).borrowObject(); <em class="jxr_comment">// Called during initialization</em>
+<a class="jxr_linenumber" name="168" href="#168">168</a> 
+<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>));
+<a class="jxr_linenumber" name="170" href="#170">170</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="171" href="#171">171</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="172" href="#172">172</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="173" href="#173">173</a> 
+<a class="jxr_linenumber" name="174" href="#174">174</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="175" href="#175">175</a> 
+<a class="jxr_linenumber" name="176" href="#176">176</a>     }
 <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="178" href="#178">178</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="179" href="#179">179</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="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&lt;Batch&gt; 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>         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 &lt; 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="181" href="#181">181</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 2;
+<a class="jxr_linenumber" name="182" href="#182">182</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 2;
+<a class="jxr_linenumber" name="183" href="#183">183</a> 
+<a class="jxr_linenumber" name="184" href="#184">184</a>         <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="185" href="#185">185</a>         VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="186" href="#186">186</a>                                                                  mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="187" href="#187">187</a> 
+<a class="jxr_linenumber" name="188" href="#188">188</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="189" href="#189">189</a>         tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="190" href="#190">190</a>         tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="191" href="#191">191</a> 
+<a class="jxr_linenumber" name="192" href="#192">192</a>         ObjectPool&lt;Batch&gt; batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="193" href="#193">193</a> 
+<a class="jxr_linenumber" name="194" href="#194">194</a>         ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(<strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="195" href="#195">195</a> 
+<a class="jxr_linenumber" name="196" href="#196">196</a>         PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="197" href="#197">197</a>         <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i &lt; tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="198" href="#198">198</a>             handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="199" href="#199">199</a>                                                           <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="200" href="#200">200</a>                                                           leaseManager,
+<a class="jxr_linenumber" name="201" href="#201">201</a>                                                           commitTable,
+<a class="jxr_linenumber" name="202" href="#202">202</a>                                                           replyProcessor,
+<a class="jxr_linenumber" name="203" href="#203">203</a>                                                           retryProcessor,
+<a class="jxr_linenumber" name="204" href="#204">204</a>                                                           panicker);
+<a class="jxr_linenumber" name="205" href="#205">205</a>         }
+<a class="jxr_linenumber" name="206" href="#206">206</a> 
+<a class="jxr_linenumber" name="207" href="#207">207</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="208" href="#208">208</a>         PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), commitTable, batchPool,
+<a class="jxr_linenumber" name="209" href="#209">209</a>                                                                      panicker, handlers, metrics);
+<a class="jxr_linenumber" name="210" href="#210">210</a> 
+<a class="jxr_linenumber" name="211" href="#211">211</a>         verify(batchPool, times(1)).borrowObject(); <em class="jxr_comment">// Called during initialization</em>
+<a class="jxr_linenumber" name="212" href="#212">212</a> 
+<a class="jxr_linenumber" name="213" href="#213">213</a>         <em class="jxr_comment">// Fill 1st handler Batches completely</em>
 <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="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">// 1st batch full</em>
+<a class="jxr_linenumber" name="216" href="#216">216</a>         verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="217" href="#217">217</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="218" href="#218">218</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="219" href="#219">219</a>         verify(batchPool, times(3)).borrowObject();
+<a class="jxr_linenumber" name="220" href="#220">220</a> 
+<a class="jxr_linenumber" name="221" href="#221">221</a>         <em class="jxr_comment">// Test empty flush does not trigger response in getting a new currentBatch</em>
+<a class="jxr_linenumber" name="222" href="#222">222</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="223" href="#223">223</a>         verify(batchPool, times(3)).borrowObject();
+<a class="jxr_linenumber" name="224" href="#224">224</a> 
+<a class="jxr_linenumber" name="225" href="#225">225</a>         <em class="jxr_comment">// Fill 2nd handler Batches completely</em>
+<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>));
+<a class="jxr_linenumber" name="227" href="#227">227</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="228" href="#228">228</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="229" href="#229">229</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="230" href="#230">230</a>         verify(batchPool, times(1 + (NUM_CT_WRITERS * BATCH_SIZE_PER_CT_WRITER))).borrowObject();
+<a class="jxr_linenumber" name="231" href="#231">231</a> 
+<a class="jxr_linenumber" name="232" href="#232">232</a>         <em class="jxr_comment">// Start filling a new currentBatch and flush it immediately</em>
+<a class="jxr_linenumber" name="233" href="#233">233</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="234" href="#234">234</a>         verify(batchPool, times(5)).borrowObject();
+<a class="jxr_linenumber" name="235" href="#235">235</a>         proc.triggerCurrentBatchFlush(); <em class="jxr_comment">// Flushing should provoke invocation of a new batch</em>
+<a class="jxr_linenumber" name="236" href="#236">236</a>         verify(batchPool, times(6)).borrowObject();
+<a class="jxr_linenumber" name="237" href="#237">237</a> 
+<a class="jxr_linenumber" name="238" href="#238">238</a>         <em class="jxr_comment">// Test empty flush does not trigger response</em>
 <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="241" href="#241">241</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="242" href="#242">242</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="243" href="#243">243</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="244" href="#244">244</a>         verify(batchPool, times(6)).borrowObject();
+<a class="jxr_linenumber" name="245" href="#245">245</a> 
+<a class="jxr_linenumber" name="246" href="#246">246</a>     }
 <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="248" href="#248">248</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="249" href="#249">249</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="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&lt;Batch&gt; 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 &lt; 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="251" href="#251">251</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 1;
+<a class="jxr_linenumber" name="252" href="#252">252</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 1;
+<a class="jxr_linenumber" name="253" href="#253">253</a> 
+<a class="jxr_linenumber" name="254" href="#254">254</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="255" href="#255">255</a>         tsoConfig.setBatchSizePerCTWriter(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="256" href="#256">256</a>         tsoConfig.setNumConcurrentCTWriters(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="257" href="#257">257</a>         tsoConfig.setBatchPersistTimeoutInMs(100);
+<a class="jxr_linenumber" name="258" href="#258">258</a> 
+<a class="jxr_linenumber" name="259" href="#259">259</a>         ObjectPool&lt;Batch&gt; batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="260" href="#260">260</a> 
+<a class="jxr_linenumber" name="261" href="#261">261</a>         ReplyProcessor replyProcessor = <strong class="jxr_keyword">new</strong> ReplyProcessorImpl(<strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), metrics, panicker, batchPool);
+<a class="jxr_linenumber" name="262" href="#262">262</a> 
+<a class="jxr_linenumber" name="263" href="#263">263</a>         <em class="jxr_comment">// Init a non-HA lease manager</em>
+<a class="jxr_linenumber" name="264" href="#264">264</a>         VoidLeaseManager leaseManager = spy(<strong class="jxr_keyword">new</strong> VoidLeaseManager(mock(TSOChannelHandler.<strong class="jxr_keyword">class</strong>),
+<a class="jxr_linenumber" name="265" href="#265">265</a>                 mock(TSOStateManager.<strong class="jxr_keyword">class</strong>)));
+<a class="jxr_linenumber" name="266" href="#266">266</a> 
+<a class="jxr_linenumber" name="267" href="#267">267</a>         PersistenceProcessorHandler[] handlers = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler[tsoConfig.getNumConcurrentCTWriters()];
+<a class="jxr_linenumber" name="268" href="#268">268</a>         <strong class="jxr_keyword">for</strong> (<strong class="jxr_keyword">int</strong> i = 0; i &lt; tsoConfig.getNumConcurrentCTWriters(); i++) {
+<a class="jxr_linenumber" name="269" href="#269">269</a>             handlers[i] = <strong class="jxr_keyword">new</strong> PersistenceProcessorHandler(metrics,
+<a class="jxr_linenumber" name="270" href="#270">270</a>                                                           <span class="jxr_string">"localhost:1234"</span>,
+<a class="jxr_linenumber" name="271" href="#271">271</a>                                                           leaseManager,
+<a class="jxr_linenumber" name="272" href="#272">272</a>                                                           commitTable,
+<a class="jxr_linenumber" name="273" href="#273">273</a>                                                           replyProcessor,
+<a class="jxr_linenumber" name="274" href="#274">274</a>                                                           retryProcessor,
+<a class="jxr_linenumber" name="275" href="#275">275</a>                                                           panicker);
+<a class="jxr_linenumber" name="276" href="#276">276</a>         }
+<a class="jxr_linenumber" name="277" href="#277">277</a> 
+<a class="jxr_linenumber" name="278" href="#278">278</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="279" href="#279">279</a>         PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), commitTable, batchPool,
+<a class="jxr_linenumber" name="280" href="#280">280</a>                                                                      panicker, handlers, metrics);
+<a class="jxr_linenumber" name="281" href="#281">281</a> 
+<a class="jxr_linenumber" name="282" href="#282">282</a>         <em class="jxr_comment">// The non-ha lease manager always return true for</em>
+<a class="jxr_linenumber" name="283" href="#283">283</a>         <em class="jxr_comment">// stillInLeasePeriod(), so verify the currentBatch sends replies as master</em>
+<a class="jxr_linenumber" name="284" href="#284">284</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="285" href="#285">285</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="286" href="#286">286</a>         verify(leaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="287" href="#287">287</a>         verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="288" href="#288">288</a> 
+<a class="jxr_linenumber" name="289" href="#289">289</a>     }
 <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="291" href="#291">291</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="292" href="#292">292</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="293" href="#293">293</a> 
+<a class="jxr_linenumber" name="294" href="#294">294</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="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="296" href="#296">296</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="297" href="#297">297</a>         tsoConfig.setNumConcurrentCTWriters(NUM_PERSIST_HANDLERS);
+<a class="jxr_linenumber" name="298" href="#298">298</a> 
+<a class="jxr_linenumber" name="299" href="#299">299</a>         testPersistenceWithHALeaseManagerPreservingLease(tsoConfig);
+<a class="jxr_linenumber" name="300" href="#300">300</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease1(tsoConfig);
+<a class="jxr_linenumber" name="301" href="#301">301</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease2(tsoConfig);
+<a class="jxr_linenumber" name="302" href="#302">302</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease3(tsoConfig);
+<a class="jxr_linenumber" name="303" href="#303">303</a> 
+<a class="jxr_linenumber" name="304" href="#304">304</a>     }
 <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="306" href="#306">306</a>     @Test(timeOut = 30_000)
+<a class="jxr_linenumber" name="307" href="#307">307</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="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="309" href="#309">309</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> NUM_CT_WRITERS = 4;
+<a class="jxr_linenumber" name="310" href="#310">310</a>         <strong class="jxr_keyword">final</strong> <strong class="jxr_keyword">int</strong> BATCH_SIZE_PER_CT_WRITER = 4;
+<a class="jxr_linenumber" name="311" href="#311">311</a> 
+<a class="jxr_linenumber" name="312" href="#312">312</a>         TSOServerConfig tsoConfig = <strong class="jxr_keyword">new</strong> TSOServerConfig();
+<a class="jxr_linenumber" name="313" href="#313">313</a>         tsoConfig.setNumConcurrentCTWriters(NUM_CT_WRITERS);
+<a class="jxr_linenumber" name="314" href="#314">314</a>         tsoConfig.setBatchSizePerCTWriter(BATCH_SIZE_PER_CT_WRITER);
+<a class="jxr_linenumber" name="315" href="#315">315</a>         tsoConfig.setBatchPersistTimeoutInMs(100);
+<a class="jxr_linenumber" name="316" href="#316">316</a> 
+<a class="jxr_linenumber" name="317" href="#317">317</a>         testPersistenceWithHALeaseManagerPreservingLease(tsoConfig);
+<a class="jxr_linenumber" name="318" href="#318">318</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease1(tsoConfig);
+<a class="jxr_linenumber" name="319" href="#319">319</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease2(tsoConfig);
+<a class="jxr_linenumber" name="320" href="#320">320</a>         testPersistenceWithHALeaseManagerFailingToPreserveLease3(tsoConfig);
+<a class="jxr_linenumber" name="321" href="#321">321</a> 
+<a class="jxr_linenumber" name="322" href="#322">322</a>     }
+<a class="jxr_linenumber" name="323" href="#323">323</a> 
+<a class="jxr_linenumber" name="324" href="#324">324</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="325" href="#325">325</a> 
-<a class="jxr_linenumber" name="326" href="#326">326</a>         ObjectPool&lt;Batch&gt; 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="326" href="#326">326</a>         <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="327" href="#327">327</a>         LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="328" href="#328">328</a> 
+<a class="jxr_linenumber" name="329" href="#329">329</a>         ObjectPool&lt;Batch&gt; batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="330" href="#330">330</a> 
+<a class="jxr_linenumber" name="331" href="#331">331</a>         PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="332" href="#332">332</a> 
+<a class="jxr_linenumber" name="333" href="#333">333</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="334" href="#334">334</a>         PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), commitTable, batchPool,
+<a class="jxr_linenumber" name="335" href="#335">335</a>                                                                      panicker, handlers, metrics);
+<a class="jxr_linenumber" name="336" href="#336">336</a> 
+<a class="jxr_linenumber" name="337" href="#337">337</a>         <em class="jxr_comment">// Test: Configure the lease manager to return true always</em>
+<a class="jxr_linenumber" name="338" href="#338">338</a>         doReturn(<strong class="jxr_keyword">true</strong>).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="339" href="#339">339</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="340" href="#340">340</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="341" href="#341">341</a>         verify(simulatedHALeaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="342" href="#342">342</a>         verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="343" href="#343">343</a>     }
+<a class="jxr_linenumber" name="344" href="#344">344</a> 
+<a class="jxr_linenumber" name="345" href="#345">345</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="346" href="#346">346</a> 
-<a class="jxr_linenumber" name="347" href="#347">347</a>         ObjectPool&lt;Batch&gt; 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="347" href="#347">347</a>         <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="348" href="#348">348</a>         LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="349" href="#349">349</a> 
+<a class="jxr_linenumber" name="350" href="#350">350</a>         ObjectPool&lt;Batch&gt; batchPool = spy(<strong class="jxr_keyword">new</strong> BatchPoolModule(tsoConfig).getBatchPool());
+<a class="jxr_linenumber" name="351" href="#351">351</a> 
+<a class="jxr_linenumber" name="352" href="#352">352</a>         PersistenceProcessorHandler[] handlers = configureHandlers (tsoConfig, simulatedHALeaseManager, batchPool);
+<a class="jxr_linenumber" name="353" href="#353">353</a> 
+<a class="jxr_linenumber" name="354" href="#354">354</a>         <em class="jxr_comment">// Component under test</em>
+<a class="jxr_linenumber" name="355" href="#355">355</a>         PersistenceProcessorImpl proc = <strong class="jxr_keyword">new</strong> PersistenceProcessorImpl(tsoConfig, <strong class="jxr_keyword">new</strong> BlockingWaitStrategy(), commitTable, batchPool,
+<a class="jxr_linenumber" name="356" href="#356">356</a>                                                                      panicker, handlers, metrics);
+<a class="jxr_linenumber" name="357" href="#357">357</a> 
+<a class="jxr_linenumber" name="358" href="#358">358</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="359" href="#359">359</a>         doReturn(<strong class="jxr_keyword">true</strong>).doReturn(false).when(simulatedHALeaseManager).stillInLeasePeriod();
+<a class="jxr_linenumber" name="360" href="#360">360</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="361" href="#361">361</a>         proc.triggerCurrentBatchFlush();
+<a class="jxr_linenumber" name="362" href="#362">362</a>         verify(simulatedHALeaseManager, timeout(1000).times(2)).stillInLeasePeriod();
+<a class="jxr_linenumber" name="363" href="#363">363</a>         verify(batchPool, times(2)).borrowObject();
+<a class="jxr_linenumber" name="364" href="#364">364</a>     }
+<a class="jxr_linenumber" name="365" href="#365">365</a> 
+<a class="jxr_linenumber" name="366" href="#366">366</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="367" href="#367">367</a> 
-<a class="jxr_linenumber" name="368" href="#368">368</a>         ObjectPool&lt;Batch&gt; 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="368" href="#368">368</a>         <em class="jxr_comment">// Init a HA lease manager</em>
+<a class="jxr_linenumber" name="369" href="#369">369</a>         LeaseManager simulatedHALeaseManager = mock(LeaseManager.<strong class="jxr_keyword">class</strong>);
+<a class="jxr_linenumber" name="370" href="#370">370</a> 
+<a class="jxr_linenumber" name="371" href="#371">371</a>         ObjectPool&lt;Batch&gt; ba

<TRUNCATED>