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 < 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<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> }
-<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 < 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<Long> 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<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> @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<Batch> 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 < 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<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> 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="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<Batch> 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 < 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<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="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<Batch> 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 < 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<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="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<Batch> 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<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="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<Batch> 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<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="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<Batch> ba
<TRUNCATED>