hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bus...@apache.org
Subject [09/51] [partial] hbase git commit: Published site at e73a9594c218ed969a2f5b0b356d7b8d0e1474c0.
Date Thu, 26 Nov 2015 04:29:34 GMT
http://git-wip-us.apache.org/repos/asf/hbase/blob/a986dfe6/testdevapidocs/src-html/org/apache/hadoop/hbase/regionserver/TestHRegion.PutThread.html
----------------------------------------------------------------------
diff --git a/testdevapidocs/src-html/org/apache/hadoop/hbase/regionserver/TestHRegion.PutThread.html b/testdevapidocs/src-html/org/apache/hadoop/hbase/regionserver/TestHRegion.PutThread.html
index 5632675..07ddcb9 100644
--- a/testdevapidocs/src-html/org/apache/hadoop/hbase/regionserver/TestHRegion.PutThread.html
+++ b/testdevapidocs/src-html/org/apache/hadoop/hbase/regionserver/TestHRegion.PutThread.html
@@ -64,6486 +64,6554 @@
 <span class="sourceLineNo">056</span>import java.util.NavigableMap;<a name="line.56"></a>
 <span class="sourceLineNo">057</span>import java.util.TreeMap;<a name="line.57"></a>
 <span class="sourceLineNo">058</span>import java.util.UUID;<a name="line.58"></a>
-<span class="sourceLineNo">059</span>import java.util.concurrent.CountDownLatch;<a name="line.59"></a>
-<span class="sourceLineNo">060</span>import java.util.concurrent.atomic.AtomicBoolean;<a name="line.60"></a>
-<span class="sourceLineNo">061</span>import java.util.concurrent.atomic.AtomicInteger;<a name="line.61"></a>
-<span class="sourceLineNo">062</span>import java.util.concurrent.atomic.AtomicReference;<a name="line.62"></a>
-<span class="sourceLineNo">063</span><a name="line.63"></a>
-<span class="sourceLineNo">064</span>import org.apache.commons.lang.RandomStringUtils;<a name="line.64"></a>
-<span class="sourceLineNo">065</span>import org.apache.commons.logging.Log;<a name="line.65"></a>
-<span class="sourceLineNo">066</span>import org.apache.commons.logging.LogFactory;<a name="line.66"></a>
-<span class="sourceLineNo">067</span>import org.apache.hadoop.conf.Configuration;<a name="line.67"></a>
-<span class="sourceLineNo">068</span>import org.apache.hadoop.fs.FSDataOutputStream;<a name="line.68"></a>
-<span class="sourceLineNo">069</span>import org.apache.hadoop.fs.FileStatus;<a name="line.69"></a>
-<span class="sourceLineNo">070</span>import org.apache.hadoop.fs.FileSystem;<a name="line.70"></a>
-<span class="sourceLineNo">071</span>import org.apache.hadoop.fs.Path;<a name="line.71"></a>
-<span class="sourceLineNo">072</span>import org.apache.hadoop.hbase.Cell;<a name="line.72"></a>
-<span class="sourceLineNo">073</span>import org.apache.hadoop.hbase.CellUtil;<a name="line.73"></a>
-<span class="sourceLineNo">074</span>import org.apache.hadoop.hbase.CompatibilitySingletonFactory;<a name="line.74"></a>
-<span class="sourceLineNo">075</span>import org.apache.hadoop.hbase.DroppedSnapshotException;<a name="line.75"></a>
-<span class="sourceLineNo">076</span>import org.apache.hadoop.hbase.HBaseConfiguration;<a name="line.76"></a>
-<span class="sourceLineNo">077</span>import org.apache.hadoop.hbase.HBaseTestCase;<a name="line.77"></a>
-<span class="sourceLineNo">078</span>import org.apache.hadoop.hbase.HBaseTestingUtility;<a name="line.78"></a>
-<span class="sourceLineNo">079</span>import org.apache.hadoop.hbase.HColumnDescriptor;<a name="line.79"></a>
-<span class="sourceLineNo">080</span>import org.apache.hadoop.hbase.HConstants;<a name="line.80"></a>
-<span class="sourceLineNo">081</span>import org.apache.hadoop.hbase.HConstants.OperationStatusCode;<a name="line.81"></a>
-<span class="sourceLineNo">082</span>import org.apache.hadoop.hbase.HDFSBlocksDistribution;<a name="line.82"></a>
-<span class="sourceLineNo">083</span>import org.apache.hadoop.hbase.HRegionInfo;<a name="line.83"></a>
-<span class="sourceLineNo">084</span>import org.apache.hadoop.hbase.HTableDescriptor;<a name="line.84"></a>
-<span class="sourceLineNo">085</span>import org.apache.hadoop.hbase.KeyValue;<a name="line.85"></a>
-<span class="sourceLineNo">086</span>import org.apache.hadoop.hbase.MiniHBaseCluster;<a name="line.86"></a>
-<span class="sourceLineNo">087</span>import org.apache.hadoop.hbase.MultithreadedTestUtil;<a name="line.87"></a>
-<span class="sourceLineNo">088</span>import org.apache.hadoop.hbase.MultithreadedTestUtil.RepeatingTestThread;<a name="line.88"></a>
-<span class="sourceLineNo">089</span>import org.apache.hadoop.hbase.MultithreadedTestUtil.TestThread;<a name="line.89"></a>
-<span class="sourceLineNo">090</span>import org.apache.hadoop.hbase.NotServingRegionException;<a name="line.90"></a>
-<span class="sourceLineNo">091</span>import org.apache.hadoop.hbase.RegionTooBusyException;<a name="line.91"></a>
-<span class="sourceLineNo">092</span>import org.apache.hadoop.hbase.ServerName;<a name="line.92"></a>
-<span class="sourceLineNo">093</span>import org.apache.hadoop.hbase.TableName;<a name="line.93"></a>
-<span class="sourceLineNo">094</span>import org.apache.hadoop.hbase.Tag;<a name="line.94"></a>
-<span class="sourceLineNo">095</span>import org.apache.hadoop.hbase.TagType;<a name="line.95"></a>
-<span class="sourceLineNo">096</span>import org.apache.hadoop.hbase.Waiter;<a name="line.96"></a>
-<span class="sourceLineNo">097</span>import org.apache.hadoop.hbase.client.Append;<a name="line.97"></a>
-<span class="sourceLineNo">098</span>import org.apache.hadoop.hbase.client.Delete;<a name="line.98"></a>
-<span class="sourceLineNo">099</span>import org.apache.hadoop.hbase.client.Durability;<a name="line.99"></a>
-<span class="sourceLineNo">100</span>import org.apache.hadoop.hbase.client.Get;<a name="line.100"></a>
-<span class="sourceLineNo">101</span>import org.apache.hadoop.hbase.client.Increment;<a name="line.101"></a>
-<span class="sourceLineNo">102</span>import org.apache.hadoop.hbase.client.Put;<a name="line.102"></a>
-<span class="sourceLineNo">103</span>import org.apache.hadoop.hbase.client.Result;<a name="line.103"></a>
-<span class="sourceLineNo">104</span>import org.apache.hadoop.hbase.client.RowMutations;<a name="line.104"></a>
-<span class="sourceLineNo">105</span>import org.apache.hadoop.hbase.client.Scan;<a name="line.105"></a>
-<span class="sourceLineNo">106</span>import org.apache.hadoop.hbase.client.Table;<a name="line.106"></a>
-<span class="sourceLineNo">107</span>import org.apache.hadoop.hbase.exceptions.FailedSanityCheckException;<a name="line.107"></a>
-<span class="sourceLineNo">108</span>import org.apache.hadoop.hbase.filter.BinaryComparator;<a name="line.108"></a>
-<span class="sourceLineNo">109</span>import org.apache.hadoop.hbase.filter.ColumnCountGetFilter;<a name="line.109"></a>
-<span class="sourceLineNo">110</span>import org.apache.hadoop.hbase.filter.CompareFilter.CompareOp;<a name="line.110"></a>
-<span class="sourceLineNo">111</span>import org.apache.hadoop.hbase.filter.Filter;<a name="line.111"></a>
-<span class="sourceLineNo">112</span>import org.apache.hadoop.hbase.filter.FilterBase;<a name="line.112"></a>
-<span class="sourceLineNo">113</span>import org.apache.hadoop.hbase.filter.FilterList;<a name="line.113"></a>
-<span class="sourceLineNo">114</span>import org.apache.hadoop.hbase.filter.NullComparator;<a name="line.114"></a>
-<span class="sourceLineNo">115</span>import org.apache.hadoop.hbase.filter.PrefixFilter;<a name="line.115"></a>
-<span class="sourceLineNo">116</span>import org.apache.hadoop.hbase.filter.SingleColumnValueExcludeFilter;<a name="line.116"></a>
-<span class="sourceLineNo">117</span>import org.apache.hadoop.hbase.filter.SingleColumnValueFilter;<a name="line.117"></a>
-<span class="sourceLineNo">118</span>import org.apache.hadoop.hbase.io.hfile.HFile;<a name="line.118"></a>
-<span class="sourceLineNo">119</span>import org.apache.hadoop.hbase.monitoring.MonitoredRPCHandler;<a name="line.119"></a>
-<span class="sourceLineNo">120</span>import org.apache.hadoop.hbase.monitoring.MonitoredTask;<a name="line.120"></a>
-<span class="sourceLineNo">121</span>import org.apache.hadoop.hbase.monitoring.TaskMonitor;<a name="line.121"></a>
-<span class="sourceLineNo">122</span>import org.apache.hadoop.hbase.protobuf.ProtobufUtil;<a name="line.122"></a>
-<span class="sourceLineNo">123</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.CompactionDescriptor;<a name="line.123"></a>
-<span class="sourceLineNo">124</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor;<a name="line.124"></a>
-<span class="sourceLineNo">125</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor.FlushAction;<a name="line.125"></a>
-<span class="sourceLineNo">126</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor.StoreFlushDescriptor;<a name="line.126"></a>
-<span class="sourceLineNo">127</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.RegionEventDescriptor;<a name="line.127"></a>
-<span class="sourceLineNo">128</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.StoreDescriptor;<a name="line.128"></a>
-<span class="sourceLineNo">129</span>import org.apache.hadoop.hbase.regionserver.HRegion.RegionScannerImpl;<a name="line.129"></a>
-<span class="sourceLineNo">130</span>import org.apache.hadoop.hbase.regionserver.Region.RowLock;<a name="line.130"></a>
-<span class="sourceLineNo">131</span>import org.apache.hadoop.hbase.regionserver.TestStore.FaultyFileSystem;<a name="line.131"></a>
-<span class="sourceLineNo">132</span>import org.apache.hadoop.hbase.regionserver.handler.FinishRegionRecoveringHandler;<a name="line.132"></a>
-<span class="sourceLineNo">133</span>import org.apache.hadoop.hbase.regionserver.wal.FSHLog;<a name="line.133"></a>
-<span class="sourceLineNo">134</span>import org.apache.hadoop.hbase.regionserver.wal.HLogKey;<a name="line.134"></a>
-<span class="sourceLineNo">135</span>import org.apache.hadoop.hbase.regionserver.wal.MetricsWAL;<a name="line.135"></a>
-<span class="sourceLineNo">136</span>import org.apache.hadoop.hbase.regionserver.wal.MetricsWALSource;<a name="line.136"></a>
-<span class="sourceLineNo">137</span>import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener;<a name="line.137"></a>
-<span class="sourceLineNo">138</span>import org.apache.hadoop.hbase.regionserver.wal.WALEdit;<a name="line.138"></a>
-<span class="sourceLineNo">139</span>import org.apache.hadoop.hbase.regionserver.wal.WALUtil;<a name="line.139"></a>
-<span class="sourceLineNo">140</span>import org.apache.hadoop.hbase.security.User;<a name="line.140"></a>
-<span class="sourceLineNo">141</span>import org.apache.hadoop.hbase.test.MetricsAssertHelper;<a name="line.141"></a>
-<span class="sourceLineNo">142</span>import org.apache.hadoop.hbase.testclassification.LargeTests;<a name="line.142"></a>
-<span class="sourceLineNo">143</span>import org.apache.hadoop.hbase.testclassification.VerySlowRegionServerTests;<a name="line.143"></a>
-<span class="sourceLineNo">144</span>import org.apache.hadoop.hbase.util.Bytes;<a name="line.144"></a>
-<span class="sourceLineNo">145</span>import org.apache.hadoop.hbase.util.EnvironmentEdgeManager;<a name="line.145"></a>
-<span class="sourceLineNo">146</span>import org.apache.hadoop.hbase.util.EnvironmentEdgeManagerTestHelper;<a name="line.146"></a>
-<span class="sourceLineNo">147</span>import org.apache.hadoop.hbase.util.FSUtils;<a name="line.147"></a>
-<span class="sourceLineNo">148</span>import org.apache.hadoop.hbase.util.IncrementingEnvironmentEdge;<a name="line.148"></a>
-<span class="sourceLineNo">149</span>import org.apache.hadoop.hbase.util.ManualEnvironmentEdge;<a name="line.149"></a>
-<span class="sourceLineNo">150</span>import org.apache.hadoop.hbase.util.PairOfSameType;<a name="line.150"></a>
-<span class="sourceLineNo">151</span>import org.apache.hadoop.hbase.util.Threads;<a name="line.151"></a>
-<span class="sourceLineNo">152</span>import org.apache.hadoop.hbase.wal.DefaultWALProvider;<a name="line.152"></a>
-<span class="sourceLineNo">153</span>import org.apache.hadoop.hbase.wal.FaultyFSLog;<a name="line.153"></a>
-<span class="sourceLineNo">154</span>import org.apache.hadoop.hbase.wal.WAL;<a name="line.154"></a>
-<span class="sourceLineNo">155</span>import org.apache.hadoop.hbase.wal.WALFactory;<a name="line.155"></a>
-<span class="sourceLineNo">156</span>import org.apache.hadoop.hbase.wal.WALKey;<a name="line.156"></a>
-<span class="sourceLineNo">157</span>import org.apache.hadoop.hbase.wal.WALProvider;<a name="line.157"></a>
-<span class="sourceLineNo">158</span>import org.apache.hadoop.hbase.wal.WALProvider.Writer;<a name="line.158"></a>
-<span class="sourceLineNo">159</span>import org.apache.hadoop.hbase.wal.WALSplitter;<a name="line.159"></a>
-<span class="sourceLineNo">160</span>import org.junit.After;<a name="line.160"></a>
-<span class="sourceLineNo">161</span>import org.junit.Assert;<a name="line.161"></a>
-<span class="sourceLineNo">162</span>import org.junit.Before;<a name="line.162"></a>
-<span class="sourceLineNo">163</span>import org.junit.Rule;<a name="line.163"></a>
-<span class="sourceLineNo">164</span>import org.junit.Test;<a name="line.164"></a>
-<span class="sourceLineNo">165</span>import org.junit.experimental.categories.Category;<a name="line.165"></a>
-<span class="sourceLineNo">166</span>import org.junit.rules.TestName;<a name="line.166"></a>
-<span class="sourceLineNo">167</span>import org.mockito.ArgumentCaptor;<a name="line.167"></a>
-<span class="sourceLineNo">168</span>import org.mockito.ArgumentMatcher;<a name="line.168"></a>
-<span class="sourceLineNo">169</span>import org.mockito.Mockito;<a name="line.169"></a>
-<span class="sourceLineNo">170</span>import org.mockito.invocation.InvocationOnMock;<a name="line.170"></a>
-<span class="sourceLineNo">171</span>import org.mockito.stubbing.Answer;<a name="line.171"></a>
-<span class="sourceLineNo">172</span><a name="line.172"></a>
-<span class="sourceLineNo">173</span>import com.google.common.collect.ImmutableList;<a name="line.173"></a>
-<span class="sourceLineNo">174</span>import com.google.common.collect.Lists;<a name="line.174"></a>
-<span class="sourceLineNo">175</span>import com.google.common.collect.Maps;<a name="line.175"></a>
-<span class="sourceLineNo">176</span>import com.google.protobuf.ByteString;<a name="line.176"></a>
-<span class="sourceLineNo">177</span><a name="line.177"></a>
-<span class="sourceLineNo">178</span>/**<a name="line.178"></a>
-<span class="sourceLineNo">179</span> * Basic stand-alone testing of HRegion.  No clusters!<a name="line.179"></a>
-<span class="sourceLineNo">180</span> *<a name="line.180"></a>
-<span class="sourceLineNo">181</span> * A lot of the meta information for an HRegion now lives inside other HRegions<a name="line.181"></a>
-<span class="sourceLineNo">182</span> * or in the HBaseMaster, so only basic testing is possible.<a name="line.182"></a>
-<span class="sourceLineNo">183</span> */<a name="line.183"></a>
-<span class="sourceLineNo">184</span>@Category({VerySlowRegionServerTests.class, LargeTests.class})<a name="line.184"></a>
-<span class="sourceLineNo">185</span>@SuppressWarnings("deprecation")<a name="line.185"></a>
-<span class="sourceLineNo">186</span>public class TestHRegion {<a name="line.186"></a>
-<span class="sourceLineNo">187</span>  // Do not spin up clusters in here. If you need to spin up a cluster, do it<a name="line.187"></a>
-<span class="sourceLineNo">188</span>  // over in TestHRegionOnCluster.<a name="line.188"></a>
-<span class="sourceLineNo">189</span>  private static final Log LOG = LogFactory.getLog(TestHRegion.class);<a name="line.189"></a>
-<span class="sourceLineNo">190</span>  @Rule public TestName name = new TestName();<a name="line.190"></a>
-<span class="sourceLineNo">191</span><a name="line.191"></a>
-<span class="sourceLineNo">192</span>  private static final String COLUMN_FAMILY = "MyCF";<a name="line.192"></a>
-<span class="sourceLineNo">193</span>  private static final byte [] COLUMN_FAMILY_BYTES = Bytes.toBytes(COLUMN_FAMILY);<a name="line.193"></a>
-<span class="sourceLineNo">194</span><a name="line.194"></a>
-<span class="sourceLineNo">195</span>  HRegion region = null;<a name="line.195"></a>
-<span class="sourceLineNo">196</span>  // Do not run unit tests in parallel (? Why not?  It don't work?  Why not?  St.Ack)<a name="line.196"></a>
-<span class="sourceLineNo">197</span>  private static HBaseTestingUtility TEST_UTIL;<a name="line.197"></a>
-<span class="sourceLineNo">198</span>  public static Configuration CONF ;<a name="line.198"></a>
-<span class="sourceLineNo">199</span>  private String dir;<a name="line.199"></a>
-<span class="sourceLineNo">200</span>  private static FileSystem FILESYSTEM;<a name="line.200"></a>
-<span class="sourceLineNo">201</span>  private final int MAX_VERSIONS = 2;<a name="line.201"></a>
-<span class="sourceLineNo">202</span><a name="line.202"></a>
-<span class="sourceLineNo">203</span>  // Test names<a name="line.203"></a>
-<span class="sourceLineNo">204</span>  protected TableName tableName;<a name="line.204"></a>
-<span class="sourceLineNo">205</span>  protected String method;<a name="line.205"></a>
-<span class="sourceLineNo">206</span>  protected final byte[] qual1 = Bytes.toBytes("qual1");<a name="line.206"></a>
-<span class="sourceLineNo">207</span>  protected final byte[] qual2 = Bytes.toBytes("qual2");<a name="line.207"></a>
-<span class="sourceLineNo">208</span>  protected final byte[] qual3 = Bytes.toBytes("qual3");<a name="line.208"></a>
-<span class="sourceLineNo">209</span>  protected final byte[] value1 = Bytes.toBytes("value1");<a name="line.209"></a>
-<span class="sourceLineNo">210</span>  protected final byte[] value2 = Bytes.toBytes("value2");<a name="line.210"></a>
-<span class="sourceLineNo">211</span>  protected final byte[] row = Bytes.toBytes("rowA");<a name="line.211"></a>
-<span class="sourceLineNo">212</span>  protected final byte[] row2 = Bytes.toBytes("rowB");<a name="line.212"></a>
-<span class="sourceLineNo">213</span><a name="line.213"></a>
-<span class="sourceLineNo">214</span>  protected final MetricsAssertHelper metricsAssertHelper = CompatibilitySingletonFactory<a name="line.214"></a>
-<span class="sourceLineNo">215</span>      .getInstance(MetricsAssertHelper.class);<a name="line.215"></a>
-<span class="sourceLineNo">216</span><a name="line.216"></a>
-<span class="sourceLineNo">217</span>  @Before<a name="line.217"></a>
-<span class="sourceLineNo">218</span>  public void setup() throws IOException {<a name="line.218"></a>
-<span class="sourceLineNo">219</span>    TEST_UTIL = HBaseTestingUtility.createLocalHTU();<a name="line.219"></a>
-<span class="sourceLineNo">220</span>    FILESYSTEM = TEST_UTIL.getTestFileSystem();<a name="line.220"></a>
-<span class="sourceLineNo">221</span>    CONF = TEST_UTIL.getConfiguration();<a name="line.221"></a>
-<span class="sourceLineNo">222</span>    dir = TEST_UTIL.getDataTestDir("TestHRegion").toString();<a name="line.222"></a>
-<span class="sourceLineNo">223</span>    method = name.getMethodName();<a name="line.223"></a>
-<span class="sourceLineNo">224</span>    tableName = TableName.valueOf(name.getMethodName());<a name="line.224"></a>
-<span class="sourceLineNo">225</span>  }<a name="line.225"></a>
-<span class="sourceLineNo">226</span><a name="line.226"></a>
-<span class="sourceLineNo">227</span>  @After<a name="line.227"></a>
-<span class="sourceLineNo">228</span>  public void tearDown() throws Exception {<a name="line.228"></a>
-<span class="sourceLineNo">229</span>    EnvironmentEdgeManagerTestHelper.reset();<a name="line.229"></a>
-<span class="sourceLineNo">230</span>    LOG.info("Cleaning test directory: " + TEST_UTIL.getDataTestDir());<a name="line.230"></a>
-<span class="sourceLineNo">231</span>    TEST_UTIL.cleanupTestDir();<a name="line.231"></a>
-<span class="sourceLineNo">232</span>  }<a name="line.232"></a>
-<span class="sourceLineNo">233</span><a name="line.233"></a>
-<span class="sourceLineNo">234</span>  String getName() {<a name="line.234"></a>
-<span class="sourceLineNo">235</span>    return name.getMethodName();<a name="line.235"></a>
-<span class="sourceLineNo">236</span>  }<a name="line.236"></a>
-<span class="sourceLineNo">237</span><a name="line.237"></a>
-<span class="sourceLineNo">238</span>  /**<a name="line.238"></a>
-<span class="sourceLineNo">239</span>   * Test that I can use the max flushed sequence id after the close.<a name="line.239"></a>
-<span class="sourceLineNo">240</span>   * @throws IOException<a name="line.240"></a>
-<span class="sourceLineNo">241</span>   */<a name="line.241"></a>
-<span class="sourceLineNo">242</span>  @Test (timeout = 100000)<a name="line.242"></a>
-<span class="sourceLineNo">243</span>  public void testSequenceId() throws IOException {<a name="line.243"></a>
-<span class="sourceLineNo">244</span>    HRegion region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.244"></a>
-<span class="sourceLineNo">245</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.245"></a>
-<span class="sourceLineNo">246</span>    // Weird. This returns 0 if no store files or no edits. Afraid to change it.<a name="line.246"></a>
-<span class="sourceLineNo">247</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.247"></a>
-<span class="sourceLineNo">248</span>    region.close();<a name="line.248"></a>
-<span class="sourceLineNo">249</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.249"></a>
-<span class="sourceLineNo">250</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.250"></a>
-<span class="sourceLineNo">251</span>    // Open region again.<a name="line.251"></a>
-<span class="sourceLineNo">252</span>    region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.252"></a>
-<span class="sourceLineNo">253</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.253"></a>
-<span class="sourceLineNo">254</span>    // Make a random put against our cf.<a name="line.254"></a>
-<span class="sourceLineNo">255</span>    Put put = new Put(value);<a name="line.255"></a>
-<span class="sourceLineNo">256</span>    put.addColumn(COLUMN_FAMILY_BYTES, null, value);<a name="line.256"></a>
-<span class="sourceLineNo">257</span>    region.put(put);<a name="line.257"></a>
-<span class="sourceLineNo">258</span>    // No flush yet so init numbers should still be in place.<a name="line.258"></a>
-<span class="sourceLineNo">259</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.259"></a>
-<span class="sourceLineNo">260</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.260"></a>
-<span class="sourceLineNo">261</span>    region.flush(true);<a name="line.261"></a>
-<span class="sourceLineNo">262</span>    long max = region.getMaxFlushedSeqId();<a name="line.262"></a>
-<span class="sourceLineNo">263</span>    region.close();<a name="line.263"></a>
-<span class="sourceLineNo">264</span>    assertEquals(max, region.getMaxFlushedSeqId());<a name="line.264"></a>
-<span class="sourceLineNo">265</span>  }<a name="line.265"></a>
-<span class="sourceLineNo">266</span><a name="line.266"></a>
-<span class="sourceLineNo">267</span>  /**<a name="line.267"></a>
-<span class="sourceLineNo">268</span>   * Test for Bug 2 of HBASE-10466.<a name="line.268"></a>
-<span class="sourceLineNo">269</span>   * "Bug 2: Conditions for the first flush of region close (so-called pre-flush) If memstoreSize<a name="line.269"></a>
-<span class="sourceLineNo">270</span>   * is smaller than a certain value, or when region close starts a flush is ongoing, the first<a name="line.270"></a>
-<span class="sourceLineNo">271</span>   * flush is skipped and only the second flush takes place. However, two flushes are required in<a name="line.271"></a>
-<span class="sourceLineNo">272</span>   * case previous flush fails and leaves some data in snapshot. The bug could cause loss of data<a name="line.272"></a>
-<span class="sourceLineNo">273</span>   * in current memstore. The fix is removing all conditions except abort check so we ensure 2<a name="line.273"></a>
-<span class="sourceLineNo">274</span>   * flushes for region close."<a name="line.274"></a>
-<span class="sourceLineNo">275</span>   * @throws IOException<a name="line.275"></a>
-<span class="sourceLineNo">276</span>   */<a name="line.276"></a>
-<span class="sourceLineNo">277</span>  @Test (timeout=60000)<a name="line.277"></a>
-<span class="sourceLineNo">278</span>  public void testCloseCarryingSnapshot() throws IOException {<a name="line.278"></a>
-<span class="sourceLineNo">279</span>    HRegion region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.279"></a>
-<span class="sourceLineNo">280</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.280"></a>
-<span class="sourceLineNo">281</span>    // Get some random bytes.<a name="line.281"></a>
-<span class="sourceLineNo">282</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.282"></a>
-<span class="sourceLineNo">283</span>    // Make a random put against our cf.<a name="line.283"></a>
-<span class="sourceLineNo">284</span>    Put put = new Put(value);<a name="line.284"></a>
-<span class="sourceLineNo">285</span>    put.addColumn(COLUMN_FAMILY_BYTES, null, value);<a name="line.285"></a>
-<span class="sourceLineNo">286</span>    // First put something in current memstore, which will be in snapshot after flusher.prepare()<a name="line.286"></a>
-<span class="sourceLineNo">287</span>    region.put(put);<a name="line.287"></a>
-<span class="sourceLineNo">288</span>    StoreFlushContext storeFlushCtx = store.createFlushContext(12345);<a name="line.288"></a>
-<span class="sourceLineNo">289</span>    storeFlushCtx.prepare();<a name="line.289"></a>
-<span class="sourceLineNo">290</span>    // Second put something in current memstore<a name="line.290"></a>
-<span class="sourceLineNo">291</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.291"></a>
-<span class="sourceLineNo">292</span>    region.put(put);<a name="line.292"></a>
-<span class="sourceLineNo">293</span>    // Close with something in memstore and something in the snapshot.  Make sure all is cleared.<a name="line.293"></a>
-<span class="sourceLineNo">294</span>    region.close();<a name="line.294"></a>
-<span class="sourceLineNo">295</span>    assertEquals(0, region.getMemstoreSize());<a name="line.295"></a>
-<span class="sourceLineNo">296</span>    HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.296"></a>
-<span class="sourceLineNo">297</span>  }<a name="line.297"></a>
-<span class="sourceLineNo">298</span><a name="line.298"></a>
-<span class="sourceLineNo">299</span><a name="line.299"></a>
-<span class="sourceLineNo">300</span><a name="line.300"></a>
-<span class="sourceLineNo">301</span>  /*<a name="line.301"></a>
-<span class="sourceLineNo">302</span>   * This test is for verifying memstore snapshot size is correctly updated in case of rollback<a name="line.302"></a>
-<span class="sourceLineNo">303</span>   * See HBASE-10845<a name="line.303"></a>
-<span class="sourceLineNo">304</span>   */<a name="line.304"></a>
-<span class="sourceLineNo">305</span>  @Test (timeout=60000)<a name="line.305"></a>
-<span class="sourceLineNo">306</span>  public void testMemstoreSnapshotSize() throws IOException {<a name="line.306"></a>
-<span class="sourceLineNo">307</span>    class MyFaultyFSLog extends FaultyFSLog {<a name="line.307"></a>
-<span class="sourceLineNo">308</span>      StoreFlushContext storeFlushCtx;<a name="line.308"></a>
-<span class="sourceLineNo">309</span>      public MyFaultyFSLog(FileSystem fs, Path rootDir, String logName, Configuration conf)<a name="line.309"></a>
-<span class="sourceLineNo">310</span>          throws IOException {<a name="line.310"></a>
-<span class="sourceLineNo">311</span>        super(fs, rootDir, logName, conf);<a name="line.311"></a>
-<span class="sourceLineNo">312</span>      }<a name="line.312"></a>
-<span class="sourceLineNo">313</span><a name="line.313"></a>
-<span class="sourceLineNo">314</span>      void setStoreFlushCtx(StoreFlushContext storeFlushCtx) {<a name="line.314"></a>
-<span class="sourceLineNo">315</span>        this.storeFlushCtx = storeFlushCtx;<a name="line.315"></a>
-<span class="sourceLineNo">316</span>      }<a name="line.316"></a>
-<span class="sourceLineNo">317</span><a name="line.317"></a>
-<span class="sourceLineNo">318</span>      @Override<a name="line.318"></a>
-<span class="sourceLineNo">319</span>      public void sync(long txid) throws IOException {<a name="line.319"></a>
-<span class="sourceLineNo">320</span>        storeFlushCtx.prepare();<a name="line.320"></a>
-<span class="sourceLineNo">321</span>        super.sync(txid);<a name="line.321"></a>
+<span class="sourceLineNo">059</span>import java.util.concurrent.Callable;<a name="line.59"></a>
+<span class="sourceLineNo">060</span>import java.util.concurrent.CountDownLatch;<a name="line.60"></a>
+<span class="sourceLineNo">061</span>import java.util.concurrent.ExecutorService;<a name="line.61"></a>
+<span class="sourceLineNo">062</span>import java.util.concurrent.Executors;<a name="line.62"></a>
+<span class="sourceLineNo">063</span>import java.util.concurrent.Future;<a name="line.63"></a>
+<span class="sourceLineNo">064</span>import java.util.concurrent.TimeUnit;<a name="line.64"></a>
+<span class="sourceLineNo">065</span>import java.util.concurrent.atomic.AtomicBoolean;<a name="line.65"></a>
+<span class="sourceLineNo">066</span>import java.util.concurrent.atomic.AtomicInteger;<a name="line.66"></a>
+<span class="sourceLineNo">067</span>import java.util.concurrent.atomic.AtomicReference;<a name="line.67"></a>
+<span class="sourceLineNo">068</span><a name="line.68"></a>
+<span class="sourceLineNo">069</span>import org.apache.commons.lang.RandomStringUtils;<a name="line.69"></a>
+<span class="sourceLineNo">070</span>import org.apache.commons.logging.Log;<a name="line.70"></a>
+<span class="sourceLineNo">071</span>import org.apache.commons.logging.LogFactory;<a name="line.71"></a>
+<span class="sourceLineNo">072</span>import org.apache.hadoop.conf.Configuration;<a name="line.72"></a>
+<span class="sourceLineNo">073</span>import org.apache.hadoop.fs.FSDataOutputStream;<a name="line.73"></a>
+<span class="sourceLineNo">074</span>import org.apache.hadoop.fs.FileStatus;<a name="line.74"></a>
+<span class="sourceLineNo">075</span>import org.apache.hadoop.fs.FileSystem;<a name="line.75"></a>
+<span class="sourceLineNo">076</span>import org.apache.hadoop.fs.Path;<a name="line.76"></a>
+<span class="sourceLineNo">077</span>import org.apache.hadoop.hbase.Cell;<a name="line.77"></a>
+<span class="sourceLineNo">078</span>import org.apache.hadoop.hbase.CellUtil;<a name="line.78"></a>
+<span class="sourceLineNo">079</span>import org.apache.hadoop.hbase.CompatibilitySingletonFactory;<a name="line.79"></a>
+<span class="sourceLineNo">080</span>import org.apache.hadoop.hbase.DroppedSnapshotException;<a name="line.80"></a>
+<span class="sourceLineNo">081</span>import org.apache.hadoop.hbase.HBaseConfiguration;<a name="line.81"></a>
+<span class="sourceLineNo">082</span>import org.apache.hadoop.hbase.HBaseTestCase;<a name="line.82"></a>
+<span class="sourceLineNo">083</span>import org.apache.hadoop.hbase.HBaseTestingUtility;<a name="line.83"></a>
+<span class="sourceLineNo">084</span>import org.apache.hadoop.hbase.HColumnDescriptor;<a name="line.84"></a>
+<span class="sourceLineNo">085</span>import org.apache.hadoop.hbase.HConstants;<a name="line.85"></a>
+<span class="sourceLineNo">086</span>import org.apache.hadoop.hbase.HConstants.OperationStatusCode;<a name="line.86"></a>
+<span class="sourceLineNo">087</span>import org.apache.hadoop.hbase.HDFSBlocksDistribution;<a name="line.87"></a>
+<span class="sourceLineNo">088</span>import org.apache.hadoop.hbase.HRegionInfo;<a name="line.88"></a>
+<span class="sourceLineNo">089</span>import org.apache.hadoop.hbase.HTableDescriptor;<a name="line.89"></a>
+<span class="sourceLineNo">090</span>import org.apache.hadoop.hbase.KeyValue;<a name="line.90"></a>
+<span class="sourceLineNo">091</span>import org.apache.hadoop.hbase.MiniHBaseCluster;<a name="line.91"></a>
+<span class="sourceLineNo">092</span>import org.apache.hadoop.hbase.MultithreadedTestUtil;<a name="line.92"></a>
+<span class="sourceLineNo">093</span>import org.apache.hadoop.hbase.MultithreadedTestUtil.RepeatingTestThread;<a name="line.93"></a>
+<span class="sourceLineNo">094</span>import org.apache.hadoop.hbase.MultithreadedTestUtil.TestThread;<a name="line.94"></a>
+<span class="sourceLineNo">095</span>import org.apache.hadoop.hbase.NotServingRegionException;<a name="line.95"></a>
+<span class="sourceLineNo">096</span>import org.apache.hadoop.hbase.RegionTooBusyException;<a name="line.96"></a>
+<span class="sourceLineNo">097</span>import org.apache.hadoop.hbase.ServerName;<a name="line.97"></a>
+<span class="sourceLineNo">098</span>import org.apache.hadoop.hbase.TableName;<a name="line.98"></a>
+<span class="sourceLineNo">099</span>import org.apache.hadoop.hbase.Tag;<a name="line.99"></a>
+<span class="sourceLineNo">100</span>import org.apache.hadoop.hbase.TagType;<a name="line.100"></a>
+<span class="sourceLineNo">101</span>import org.apache.hadoop.hbase.Waiter;<a name="line.101"></a>
+<span class="sourceLineNo">102</span>import org.apache.hadoop.hbase.client.Append;<a name="line.102"></a>
+<span class="sourceLineNo">103</span>import org.apache.hadoop.hbase.client.Delete;<a name="line.103"></a>
+<span class="sourceLineNo">104</span>import org.apache.hadoop.hbase.client.Durability;<a name="line.104"></a>
+<span class="sourceLineNo">105</span>import org.apache.hadoop.hbase.client.Get;<a name="line.105"></a>
+<span class="sourceLineNo">106</span>import org.apache.hadoop.hbase.client.Increment;<a name="line.106"></a>
+<span class="sourceLineNo">107</span>import org.apache.hadoop.hbase.client.Mutation;<a name="line.107"></a>
+<span class="sourceLineNo">108</span>import org.apache.hadoop.hbase.client.Put;<a name="line.108"></a>
+<span class="sourceLineNo">109</span>import org.apache.hadoop.hbase.client.Result;<a name="line.109"></a>
+<span class="sourceLineNo">110</span>import org.apache.hadoop.hbase.client.RowMutations;<a name="line.110"></a>
+<span class="sourceLineNo">111</span>import org.apache.hadoop.hbase.client.Scan;<a name="line.111"></a>
+<span class="sourceLineNo">112</span>import org.apache.hadoop.hbase.client.Table;<a name="line.112"></a>
+<span class="sourceLineNo">113</span>import org.apache.hadoop.hbase.exceptions.FailedSanityCheckException;<a name="line.113"></a>
+<span class="sourceLineNo">114</span>import org.apache.hadoop.hbase.filter.BinaryComparator;<a name="line.114"></a>
+<span class="sourceLineNo">115</span>import org.apache.hadoop.hbase.filter.ColumnCountGetFilter;<a name="line.115"></a>
+<span class="sourceLineNo">116</span>import org.apache.hadoop.hbase.filter.CompareFilter.CompareOp;<a name="line.116"></a>
+<span class="sourceLineNo">117</span>import org.apache.hadoop.hbase.filter.Filter;<a name="line.117"></a>
+<span class="sourceLineNo">118</span>import org.apache.hadoop.hbase.filter.FilterBase;<a name="line.118"></a>
+<span class="sourceLineNo">119</span>import org.apache.hadoop.hbase.filter.FilterList;<a name="line.119"></a>
+<span class="sourceLineNo">120</span>import org.apache.hadoop.hbase.filter.NullComparator;<a name="line.120"></a>
+<span class="sourceLineNo">121</span>import org.apache.hadoop.hbase.filter.PrefixFilter;<a name="line.121"></a>
+<span class="sourceLineNo">122</span>import org.apache.hadoop.hbase.filter.SingleColumnValueExcludeFilter;<a name="line.122"></a>
+<span class="sourceLineNo">123</span>import org.apache.hadoop.hbase.filter.SingleColumnValueFilter;<a name="line.123"></a>
+<span class="sourceLineNo">124</span>import org.apache.hadoop.hbase.io.hfile.HFile;<a name="line.124"></a>
+<span class="sourceLineNo">125</span>import org.apache.hadoop.hbase.monitoring.MonitoredRPCHandler;<a name="line.125"></a>
+<span class="sourceLineNo">126</span>import org.apache.hadoop.hbase.monitoring.MonitoredTask;<a name="line.126"></a>
+<span class="sourceLineNo">127</span>import org.apache.hadoop.hbase.monitoring.TaskMonitor;<a name="line.127"></a>
+<span class="sourceLineNo">128</span>import org.apache.hadoop.hbase.protobuf.ProtobufUtil;<a name="line.128"></a>
+<span class="sourceLineNo">129</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.CompactionDescriptor;<a name="line.129"></a>
+<span class="sourceLineNo">130</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor;<a name="line.130"></a>
+<span class="sourceLineNo">131</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor.FlushAction;<a name="line.131"></a>
+<span class="sourceLineNo">132</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.FlushDescriptor.StoreFlushDescriptor;<a name="line.132"></a>
+<span class="sourceLineNo">133</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.RegionEventDescriptor;<a name="line.133"></a>
+<span class="sourceLineNo">134</span>import org.apache.hadoop.hbase.protobuf.generated.WALProtos.StoreDescriptor;<a name="line.134"></a>
+<span class="sourceLineNo">135</span>import org.apache.hadoop.hbase.regionserver.HRegion.RegionScannerImpl;<a name="line.135"></a>
+<span class="sourceLineNo">136</span>import org.apache.hadoop.hbase.regionserver.Region.RowLock;<a name="line.136"></a>
+<span class="sourceLineNo">137</span>import org.apache.hadoop.hbase.regionserver.TestStore.FaultyFileSystem;<a name="line.137"></a>
+<span class="sourceLineNo">138</span>import org.apache.hadoop.hbase.regionserver.handler.FinishRegionRecoveringHandler;<a name="line.138"></a>
+<span class="sourceLineNo">139</span>import org.apache.hadoop.hbase.regionserver.wal.FSHLog;<a name="line.139"></a>
+<span class="sourceLineNo">140</span>import org.apache.hadoop.hbase.regionserver.wal.HLogKey;<a name="line.140"></a>
+<span class="sourceLineNo">141</span>import org.apache.hadoop.hbase.regionserver.wal.MetricsWAL;<a name="line.141"></a>
+<span class="sourceLineNo">142</span>import org.apache.hadoop.hbase.regionserver.wal.MetricsWALSource;<a name="line.142"></a>
+<span class="sourceLineNo">143</span>import org.apache.hadoop.hbase.regionserver.wal.WALActionsListener;<a name="line.143"></a>
+<span class="sourceLineNo">144</span>import org.apache.hadoop.hbase.regionserver.wal.WALEdit;<a name="line.144"></a>
+<span class="sourceLineNo">145</span>import org.apache.hadoop.hbase.regionserver.wal.WALUtil;<a name="line.145"></a>
+<span class="sourceLineNo">146</span>import org.apache.hadoop.hbase.security.User;<a name="line.146"></a>
+<span class="sourceLineNo">147</span>import org.apache.hadoop.hbase.test.MetricsAssertHelper;<a name="line.147"></a>
+<span class="sourceLineNo">148</span>import org.apache.hadoop.hbase.testclassification.LargeTests;<a name="line.148"></a>
+<span class="sourceLineNo">149</span>import org.apache.hadoop.hbase.testclassification.VerySlowRegionServerTests;<a name="line.149"></a>
+<span class="sourceLineNo">150</span>import org.apache.hadoop.hbase.util.Bytes;<a name="line.150"></a>
+<span class="sourceLineNo">151</span>import org.apache.hadoop.hbase.util.EnvironmentEdgeManager;<a name="line.151"></a>
+<span class="sourceLineNo">152</span>import org.apache.hadoop.hbase.util.EnvironmentEdgeManagerTestHelper;<a name="line.152"></a>
+<span class="sourceLineNo">153</span>import org.apache.hadoop.hbase.util.FSUtils;<a name="line.153"></a>
+<span class="sourceLineNo">154</span>import org.apache.hadoop.hbase.util.IncrementingEnvironmentEdge;<a name="line.154"></a>
+<span class="sourceLineNo">155</span>import org.apache.hadoop.hbase.util.ManualEnvironmentEdge;<a name="line.155"></a>
+<span class="sourceLineNo">156</span>import org.apache.hadoop.hbase.util.PairOfSameType;<a name="line.156"></a>
+<span class="sourceLineNo">157</span>import org.apache.hadoop.hbase.util.Threads;<a name="line.157"></a>
+<span class="sourceLineNo">158</span>import org.apache.hadoop.hbase.wal.DefaultWALProvider;<a name="line.158"></a>
+<span class="sourceLineNo">159</span>import org.apache.hadoop.hbase.wal.FaultyFSLog;<a name="line.159"></a>
+<span class="sourceLineNo">160</span>import org.apache.hadoop.hbase.wal.WAL;<a name="line.160"></a>
+<span class="sourceLineNo">161</span>import org.apache.hadoop.hbase.wal.WALFactory;<a name="line.161"></a>
+<span class="sourceLineNo">162</span>import org.apache.hadoop.hbase.wal.WALKey;<a name="line.162"></a>
+<span class="sourceLineNo">163</span>import org.apache.hadoop.hbase.wal.WALProvider;<a name="line.163"></a>
+<span class="sourceLineNo">164</span>import org.apache.hadoop.hbase.wal.WALProvider.Writer;<a name="line.164"></a>
+<span class="sourceLineNo">165</span>import org.apache.hadoop.hbase.wal.WALSplitter;<a name="line.165"></a>
+<span class="sourceLineNo">166</span>import org.junit.After;<a name="line.166"></a>
+<span class="sourceLineNo">167</span>import org.junit.Assert;<a name="line.167"></a>
+<span class="sourceLineNo">168</span>import org.junit.Before;<a name="line.168"></a>
+<span class="sourceLineNo">169</span>import org.junit.Rule;<a name="line.169"></a>
+<span class="sourceLineNo">170</span>import org.junit.Test;<a name="line.170"></a>
+<span class="sourceLineNo">171</span>import org.junit.experimental.categories.Category;<a name="line.171"></a>
+<span class="sourceLineNo">172</span>import org.junit.rules.TestName;<a name="line.172"></a>
+<span class="sourceLineNo">173</span>import org.mockito.ArgumentCaptor;<a name="line.173"></a>
+<span class="sourceLineNo">174</span>import org.mockito.ArgumentMatcher;<a name="line.174"></a>
+<span class="sourceLineNo">175</span>import org.mockito.Mockito;<a name="line.175"></a>
+<span class="sourceLineNo">176</span>import org.mockito.invocation.InvocationOnMock;<a name="line.176"></a>
+<span class="sourceLineNo">177</span>import org.mockito.stubbing.Answer;<a name="line.177"></a>
+<span class="sourceLineNo">178</span><a name="line.178"></a>
+<span class="sourceLineNo">179</span>import com.google.common.collect.ImmutableList;<a name="line.179"></a>
+<span class="sourceLineNo">180</span>import com.google.common.collect.Lists;<a name="line.180"></a>
+<span class="sourceLineNo">181</span>import com.google.common.collect.Maps;<a name="line.181"></a>
+<span class="sourceLineNo">182</span>import com.google.protobuf.ByteString;<a name="line.182"></a>
+<span class="sourceLineNo">183</span><a name="line.183"></a>
+<span class="sourceLineNo">184</span>/**<a name="line.184"></a>
+<span class="sourceLineNo">185</span> * Basic stand-alone testing of HRegion.  No clusters!<a name="line.185"></a>
+<span class="sourceLineNo">186</span> *<a name="line.186"></a>
+<span class="sourceLineNo">187</span> * A lot of the meta information for an HRegion now lives inside other HRegions<a name="line.187"></a>
+<span class="sourceLineNo">188</span> * or in the HBaseMaster, so only basic testing is possible.<a name="line.188"></a>
+<span class="sourceLineNo">189</span> */<a name="line.189"></a>
+<span class="sourceLineNo">190</span>@Category({VerySlowRegionServerTests.class, LargeTests.class})<a name="line.190"></a>
+<span class="sourceLineNo">191</span>@SuppressWarnings("deprecation")<a name="line.191"></a>
+<span class="sourceLineNo">192</span>public class TestHRegion {<a name="line.192"></a>
+<span class="sourceLineNo">193</span>  // Do not spin up clusters in here. If you need to spin up a cluster, do it<a name="line.193"></a>
+<span class="sourceLineNo">194</span>  // over in TestHRegionOnCluster.<a name="line.194"></a>
+<span class="sourceLineNo">195</span>  private static final Log LOG = LogFactory.getLog(TestHRegion.class);<a name="line.195"></a>
+<span class="sourceLineNo">196</span>  @Rule public TestName name = new TestName();<a name="line.196"></a>
+<span class="sourceLineNo">197</span><a name="line.197"></a>
+<span class="sourceLineNo">198</span>  private static final String COLUMN_FAMILY = "MyCF";<a name="line.198"></a>
+<span class="sourceLineNo">199</span>  private static final byte [] COLUMN_FAMILY_BYTES = Bytes.toBytes(COLUMN_FAMILY);<a name="line.199"></a>
+<span class="sourceLineNo">200</span><a name="line.200"></a>
+<span class="sourceLineNo">201</span>  HRegion region = null;<a name="line.201"></a>
+<span class="sourceLineNo">202</span>  // Do not run unit tests in parallel (? Why not?  It don't work?  Why not?  St.Ack)<a name="line.202"></a>
+<span class="sourceLineNo">203</span>  private static HBaseTestingUtility TEST_UTIL;<a name="line.203"></a>
+<span class="sourceLineNo">204</span>  public static Configuration CONF ;<a name="line.204"></a>
+<span class="sourceLineNo">205</span>  private String dir;<a name="line.205"></a>
+<span class="sourceLineNo">206</span>  private static FileSystem FILESYSTEM;<a name="line.206"></a>
+<span class="sourceLineNo">207</span>  private final int MAX_VERSIONS = 2;<a name="line.207"></a>
+<span class="sourceLineNo">208</span><a name="line.208"></a>
+<span class="sourceLineNo">209</span>  // Test names<a name="line.209"></a>
+<span class="sourceLineNo">210</span>  protected TableName tableName;<a name="line.210"></a>
+<span class="sourceLineNo">211</span>  protected String method;<a name="line.211"></a>
+<span class="sourceLineNo">212</span>  protected final byte[] qual1 = Bytes.toBytes("qual1");<a name="line.212"></a>
+<span class="sourceLineNo">213</span>  protected final byte[] qual2 = Bytes.toBytes("qual2");<a name="line.213"></a>
+<span class="sourceLineNo">214</span>  protected final byte[] qual3 = Bytes.toBytes("qual3");<a name="line.214"></a>
+<span class="sourceLineNo">215</span>  protected final byte[] value1 = Bytes.toBytes("value1");<a name="line.215"></a>
+<span class="sourceLineNo">216</span>  protected final byte[] value2 = Bytes.toBytes("value2");<a name="line.216"></a>
+<span class="sourceLineNo">217</span>  protected final byte[] row = Bytes.toBytes("rowA");<a name="line.217"></a>
+<span class="sourceLineNo">218</span>  protected final byte[] row2 = Bytes.toBytes("rowB");<a name="line.218"></a>
+<span class="sourceLineNo">219</span><a name="line.219"></a>
+<span class="sourceLineNo">220</span>  protected final MetricsAssertHelper metricsAssertHelper = CompatibilitySingletonFactory<a name="line.220"></a>
+<span class="sourceLineNo">221</span>      .getInstance(MetricsAssertHelper.class);<a name="line.221"></a>
+<span class="sourceLineNo">222</span><a name="line.222"></a>
+<span class="sourceLineNo">223</span>  @Before<a name="line.223"></a>
+<span class="sourceLineNo">224</span>  public void setup() throws IOException {<a name="line.224"></a>
+<span class="sourceLineNo">225</span>    TEST_UTIL = HBaseTestingUtility.createLocalHTU();<a name="line.225"></a>
+<span class="sourceLineNo">226</span>    FILESYSTEM = TEST_UTIL.getTestFileSystem();<a name="line.226"></a>
+<span class="sourceLineNo">227</span>    CONF = TEST_UTIL.getConfiguration();<a name="line.227"></a>
+<span class="sourceLineNo">228</span>    dir = TEST_UTIL.getDataTestDir("TestHRegion").toString();<a name="line.228"></a>
+<span class="sourceLineNo">229</span>    method = name.getMethodName();<a name="line.229"></a>
+<span class="sourceLineNo">230</span>    tableName = TableName.valueOf(name.getMethodName());<a name="line.230"></a>
+<span class="sourceLineNo">231</span>  }<a name="line.231"></a>
+<span class="sourceLineNo">232</span><a name="line.232"></a>
+<span class="sourceLineNo">233</span>  @After<a name="line.233"></a>
+<span class="sourceLineNo">234</span>  public void tearDown() throws Exception {<a name="line.234"></a>
+<span class="sourceLineNo">235</span>    EnvironmentEdgeManagerTestHelper.reset();<a name="line.235"></a>
+<span class="sourceLineNo">236</span>    LOG.info("Cleaning test directory: " + TEST_UTIL.getDataTestDir());<a name="line.236"></a>
+<span class="sourceLineNo">237</span>    TEST_UTIL.cleanupTestDir();<a name="line.237"></a>
+<span class="sourceLineNo">238</span>  }<a name="line.238"></a>
+<span class="sourceLineNo">239</span><a name="line.239"></a>
+<span class="sourceLineNo">240</span>  String getName() {<a name="line.240"></a>
+<span class="sourceLineNo">241</span>    return name.getMethodName();<a name="line.241"></a>
+<span class="sourceLineNo">242</span>  }<a name="line.242"></a>
+<span class="sourceLineNo">243</span><a name="line.243"></a>
+<span class="sourceLineNo">244</span>  /**<a name="line.244"></a>
+<span class="sourceLineNo">245</span>   * Test that I can use the max flushed sequence id after the close.<a name="line.245"></a>
+<span class="sourceLineNo">246</span>   * @throws IOException<a name="line.246"></a>
+<span class="sourceLineNo">247</span>   */<a name="line.247"></a>
+<span class="sourceLineNo">248</span>  @Test (timeout = 100000)<a name="line.248"></a>
+<span class="sourceLineNo">249</span>  public void testSequenceId() throws IOException {<a name="line.249"></a>
+<span class="sourceLineNo">250</span>    HRegion region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.250"></a>
+<span class="sourceLineNo">251</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.251"></a>
+<span class="sourceLineNo">252</span>    // Weird. This returns 0 if no store files or no edits. Afraid to change it.<a name="line.252"></a>
+<span class="sourceLineNo">253</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.253"></a>
+<span class="sourceLineNo">254</span>    region.close();<a name="line.254"></a>
+<span class="sourceLineNo">255</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.255"></a>
+<span class="sourceLineNo">256</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.256"></a>
+<span class="sourceLineNo">257</span>    // Open region again.<a name="line.257"></a>
+<span class="sourceLineNo">258</span>    region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.258"></a>
+<span class="sourceLineNo">259</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.259"></a>
+<span class="sourceLineNo">260</span>    // Make a random put against our cf.<a name="line.260"></a>
+<span class="sourceLineNo">261</span>    Put put = new Put(value);<a name="line.261"></a>
+<span class="sourceLineNo">262</span>    put.addColumn(COLUMN_FAMILY_BYTES, null, value);<a name="line.262"></a>
+<span class="sourceLineNo">263</span>    region.put(put);<a name="line.263"></a>
+<span class="sourceLineNo">264</span>    // No flush yet so init numbers should still be in place.<a name="line.264"></a>
+<span class="sourceLineNo">265</span>    assertEquals(HConstants.NO_SEQNUM, region.getMaxFlushedSeqId());<a name="line.265"></a>
+<span class="sourceLineNo">266</span>    assertEquals(0, (long)region.getMaxStoreSeqId().get(COLUMN_FAMILY_BYTES));<a name="line.266"></a>
+<span class="sourceLineNo">267</span>    region.flush(true);<a name="line.267"></a>
+<span class="sourceLineNo">268</span>    long max = region.getMaxFlushedSeqId();<a name="line.268"></a>
+<span class="sourceLineNo">269</span>    region.close();<a name="line.269"></a>
+<span class="sourceLineNo">270</span>    assertEquals(max, region.getMaxFlushedSeqId());<a name="line.270"></a>
+<span class="sourceLineNo">271</span>  }<a name="line.271"></a>
+<span class="sourceLineNo">272</span><a name="line.272"></a>
+<span class="sourceLineNo">273</span>  /**<a name="line.273"></a>
+<span class="sourceLineNo">274</span>   * Test for Bug 2 of HBASE-10466.<a name="line.274"></a>
+<span class="sourceLineNo">275</span>   * "Bug 2: Conditions for the first flush of region close (so-called pre-flush) If memstoreSize<a name="line.275"></a>
+<span class="sourceLineNo">276</span>   * is smaller than a certain value, or when region close starts a flush is ongoing, the first<a name="line.276"></a>
+<span class="sourceLineNo">277</span>   * flush is skipped and only the second flush takes place. However, two flushes are required in<a name="line.277"></a>
+<span class="sourceLineNo">278</span>   * case previous flush fails and leaves some data in snapshot. The bug could cause loss of data<a name="line.278"></a>
+<span class="sourceLineNo">279</span>   * in current memstore. The fix is removing all conditions except abort check so we ensure 2<a name="line.279"></a>
+<span class="sourceLineNo">280</span>   * flushes for region close."<a name="line.280"></a>
+<span class="sourceLineNo">281</span>   * @throws IOException<a name="line.281"></a>
+<span class="sourceLineNo">282</span>   */<a name="line.282"></a>
+<span class="sourceLineNo">283</span>  @Test (timeout=60000)<a name="line.283"></a>
+<span class="sourceLineNo">284</span>  public void testCloseCarryingSnapshot() throws IOException {<a name="line.284"></a>
+<span class="sourceLineNo">285</span>    HRegion region = initHRegion(tableName, name.getMethodName(), CONF, COLUMN_FAMILY_BYTES);<a name="line.285"></a>
+<span class="sourceLineNo">286</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.286"></a>
+<span class="sourceLineNo">287</span>    // Get some random bytes.<a name="line.287"></a>
+<span class="sourceLineNo">288</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.288"></a>
+<span class="sourceLineNo">289</span>    // Make a random put against our cf.<a name="line.289"></a>
+<span class="sourceLineNo">290</span>    Put put = new Put(value);<a name="line.290"></a>
+<span class="sourceLineNo">291</span>    put.addColumn(COLUMN_FAMILY_BYTES, null, value);<a name="line.291"></a>
+<span class="sourceLineNo">292</span>    // First put something in current memstore, which will be in snapshot after flusher.prepare()<a name="line.292"></a>
+<span class="sourceLineNo">293</span>    region.put(put);<a name="line.293"></a>
+<span class="sourceLineNo">294</span>    StoreFlushContext storeFlushCtx = store.createFlushContext(12345);<a name="line.294"></a>
+<span class="sourceLineNo">295</span>    storeFlushCtx.prepare();<a name="line.295"></a>
+<span class="sourceLineNo">296</span>    // Second put something in current memstore<a name="line.296"></a>
+<span class="sourceLineNo">297</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.297"></a>
+<span class="sourceLineNo">298</span>    region.put(put);<a name="line.298"></a>
+<span class="sourceLineNo">299</span>    // Close with something in memstore and something in the snapshot.  Make sure all is cleared.<a name="line.299"></a>
+<span class="sourceLineNo">300</span>    region.close();<a name="line.300"></a>
+<span class="sourceLineNo">301</span>    assertEquals(0, region.getMemstoreSize());<a name="line.301"></a>
+<span class="sourceLineNo">302</span>    HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.302"></a>
+<span class="sourceLineNo">303</span>  }<a name="line.303"></a>
+<span class="sourceLineNo">304</span><a name="line.304"></a>
+<span class="sourceLineNo">305</span><a name="line.305"></a>
+<span class="sourceLineNo">306</span><a name="line.306"></a>
+<span class="sourceLineNo">307</span>  /*<a name="line.307"></a>
+<span class="sourceLineNo">308</span>   * This test is for verifying memstore snapshot size is correctly updated in case of rollback<a name="line.308"></a>
+<span class="sourceLineNo">309</span>   * See HBASE-10845<a name="line.309"></a>
+<span class="sourceLineNo">310</span>   */<a name="line.310"></a>
+<span class="sourceLineNo">311</span>  @Test (timeout=60000)<a name="line.311"></a>
+<span class="sourceLineNo">312</span>  public void testMemstoreSnapshotSize() throws IOException {<a name="line.312"></a>
+<span class="sourceLineNo">313</span>    class MyFaultyFSLog extends FaultyFSLog {<a name="line.313"></a>
+<span class="sourceLineNo">314</span>      StoreFlushContext storeFlushCtx;<a name="line.314"></a>
+<span class="sourceLineNo">315</span>      public MyFaultyFSLog(FileSystem fs, Path rootDir, String logName, Configuration conf)<a name="line.315"></a>
+<span class="sourceLineNo">316</span>          throws IOException {<a name="line.316"></a>
+<span class="sourceLineNo">317</span>        super(fs, rootDir, logName, conf);<a name="line.317"></a>
+<span class="sourceLineNo">318</span>      }<a name="line.318"></a>
+<span class="sourceLineNo">319</span><a name="line.319"></a>
+<span class="sourceLineNo">320</span>      void setStoreFlushCtx(StoreFlushContext storeFlushCtx) {<a name="line.320"></a>
+<span class="sourceLineNo">321</span>        this.storeFlushCtx = storeFlushCtx;<a name="line.321"></a>
 <span class="sourceLineNo">322</span>      }<a name="line.322"></a>
-<span class="sourceLineNo">323</span>    }<a name="line.323"></a>
-<span class="sourceLineNo">324</span><a name="line.324"></a>
-<span class="sourceLineNo">325</span>    FileSystem fs = FileSystem.get(CONF);<a name="line.325"></a>
-<span class="sourceLineNo">326</span>    Path rootDir = new Path(dir + "testMemstoreSnapshotSize");<a name="line.326"></a>
-<span class="sourceLineNo">327</span>    MyFaultyFSLog faultyLog = new MyFaultyFSLog(fs, rootDir, "testMemstoreSnapshotSize", CONF);<a name="line.327"></a>
-<span class="sourceLineNo">328</span>    HRegion region = initHRegion(tableName, null, null, name.getMethodName(),<a name="line.328"></a>
-<span class="sourceLineNo">329</span>        CONF, false, Durability.SYNC_WAL, faultyLog, COLUMN_FAMILY_BYTES);<a name="line.329"></a>
+<span class="sourceLineNo">323</span><a name="line.323"></a>
+<span class="sourceLineNo">324</span>      @Override<a name="line.324"></a>
+<span class="sourceLineNo">325</span>      public void sync(long txid) throws IOException {<a name="line.325"></a>
+<span class="sourceLineNo">326</span>        storeFlushCtx.prepare();<a name="line.326"></a>
+<span class="sourceLineNo">327</span>        super.sync(txid);<a name="line.327"></a>
+<span class="sourceLineNo">328</span>      }<a name="line.328"></a>
+<span class="sourceLineNo">329</span>    }<a name="line.329"></a>
 <span class="sourceLineNo">330</span><a name="line.330"></a>
-<span class="sourceLineNo">331</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.331"></a>
-<span class="sourceLineNo">332</span>    // Get some random bytes.<a name="line.332"></a>
-<span class="sourceLineNo">333</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.333"></a>
-<span class="sourceLineNo">334</span>    faultyLog.setStoreFlushCtx(store.createFlushContext(12345));<a name="line.334"></a>
-<span class="sourceLineNo">335</span><a name="line.335"></a>
-<span class="sourceLineNo">336</span>    Put put = new Put(value);<a name="line.336"></a>
-<span class="sourceLineNo">337</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.337"></a>
-<span class="sourceLineNo">338</span>    faultyLog.setFailureType(FaultyFSLog.FailureType.SYNC);<a name="line.338"></a>
-<span class="sourceLineNo">339</span><a name="line.339"></a>
-<span class="sourceLineNo">340</span>    boolean threwIOE = false;<a name="line.340"></a>
-<span class="sourceLineNo">341</span>    try {<a name="line.341"></a>
-<span class="sourceLineNo">342</span>      region.put(put);<a name="line.342"></a>
-<span class="sourceLineNo">343</span>    } catch (IOException ioe) {<a name="line.343"></a>
-<span class="sourceLineNo">344</span>      threwIOE = true;<a name="line.344"></a>
-<span class="sourceLineNo">345</span>    } finally {<a name="line.345"></a>
-<span class="sourceLineNo">346</span>      assertTrue("The regionserver should have thrown an exception", threwIOE);<a name="line.346"></a>
-<span class="sourceLineNo">347</span>    }<a name="line.347"></a>
-<span class="sourceLineNo">348</span>    long sz = store.getFlushableSize();<a name="line.348"></a>
-<span class="sourceLineNo">349</span>    assertTrue("flushable size should be zero, but it is " + sz, sz == 0);<a name="line.349"></a>
-<span class="sourceLineNo">350</span>    HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.350"></a>
-<span class="sourceLineNo">351</span>  }<a name="line.351"></a>
-<span class="sourceLineNo">352</span><a name="line.352"></a>
-<span class="sourceLineNo">353</span>  /**<a name="line.353"></a>
-<span class="sourceLineNo">354</span>   * Create a WAL outside of the usual helper in<a name="line.354"></a>
-<span class="sourceLineNo">355</span>   * {@link HBaseTestingUtility#createWal(Configuration, Path, HRegionInfo)} because that method<a name="line.355"></a>
-<span class="sourceLineNo">356</span>   * doesn't play nicely with FaultyFileSystem. Call this method before overriding<a name="line.356"></a>
-<span class="sourceLineNo">357</span>   * {@code fs.file.impl}.<a name="line.357"></a>
-<span class="sourceLineNo">358</span>   * @param callingMethod a unique component for the path, probably the name of the test method.<a name="line.358"></a>
-<span class="sourceLineNo">359</span>   */<a name="line.359"></a>
-<span class="sourceLineNo">360</span>  private static WAL createWALCompatibleWithFaultyFileSystem(String callingMethod,<a name="line.360"></a>
-<span class="sourceLineNo">361</span>      Configuration conf, TableName tableName) throws IOException {<a name="line.361"></a>
-<span class="sourceLineNo">362</span>    final Path logDir = TEST_UTIL.getDataTestDirOnTestFS(callingMethod + ".log");<a name="line.362"></a>
-<span class="sourceLineNo">363</span>    final Configuration walConf = new Configuration(conf);<a name="line.363"></a>
-<span class="sourceLineNo">364</span>    FSUtils.setRootDir(walConf, logDir);<a name="line.364"></a>
-<span class="sourceLineNo">365</span>    return (new WALFactory(walConf,<a name="line.365"></a>
-<span class="sourceLineNo">366</span>        Collections.&lt;WALActionsListener&gt;singletonList(new MetricsWAL()), callingMethod))<a name="line.366"></a>
-<span class="sourceLineNo">367</span>        .getWAL(tableName.toBytes(), tableName.getNamespace());<a name="line.367"></a>
-<span class="sourceLineNo">368</span>  }<a name="line.368"></a>
-<span class="sourceLineNo">369</span><a name="line.369"></a>
-<span class="sourceLineNo">370</span>  /**<a name="line.370"></a>
-<span class="sourceLineNo">371</span>   * Test for HBASE-14229: Flushing canceled by coprocessor still leads to memstoreSize set down<a name="line.371"></a>
-<span class="sourceLineNo">372</span>   */<a name="line.372"></a>
-<span class="sourceLineNo">373</span>  @Test<a name="line.373"></a>
-<span class="sourceLineNo">374</span>  public void testMemstoreSizeWithFlushCanceling() throws IOException {<a name="line.374"></a>
-<span class="sourceLineNo">375</span>    FileSystem fs = FileSystem.get(CONF);<a name="line.375"></a>
-<span class="sourceLineNo">376</span>    Path rootDir = new Path(dir + "testMemstoreSizeWithFlushCanceling");<a name="line.376"></a>
-<span class="sourceLineNo">377</span>    FSHLog hLog = new FSHLog(fs, rootDir, "testMemstoreSizeWithFlushCanceling", CONF);<a name="line.377"></a>
-<span class="sourceLineNo">378</span>    HRegion region = initHRegion(tableName, null, null, name.getMethodName(),<a name="line.378"></a>
-<span class="sourceLineNo">379</span>        CONF, false, Durability.SYNC_WAL, hLog, COLUMN_FAMILY_BYTES);<a name="line.379"></a>
-<span class="sourceLineNo">380</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.380"></a>
-<span class="sourceLineNo">381</span>    assertEquals(0, region.getMemstoreSize());<a name="line.381"></a>
-<span class="sourceLineNo">382</span><a name="line.382"></a>
-<span class="sourceLineNo">383</span>    // Put some value and make sure flush could be completed normally<a name="line.383"></a>
-<span class="sourceLineNo">384</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.384"></a>
-<span class="sourceLineNo">385</span>    Put put = new Put(value);<a name="line.385"></a>
-<span class="sourceLineNo">386</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.386"></a>
-<span class="sourceLineNo">387</span>    region.put(put);<a name="line.387"></a>
-<span class="sourceLineNo">388</span>    long onePutSize = region.getMemstoreSize();<a name="line.388"></a>
-<span class="sourceLineNo">389</span>    assertTrue(onePutSize &gt; 0);<a name="line.389"></a>
-<span class="sourceLineNo">390</span>    region.flush(true);<a name="line.390"></a>
-<span class="sourceLineNo">391</span>    assertEquals("memstoreSize should be zero", 0, region.getMemstoreSize());<a name="line.391"></a>
-<span class="sourceLineNo">392</span>    assertEquals("flushable size should be zero", 0, store.getFlushableSize());<a name="line.392"></a>
-<span class="sourceLineNo">393</span><a name="line.393"></a>
-<span class="sourceLineNo">394</span>    // save normalCPHost and replaced by mockedCPHost, which will cancel flush requests<a name="line.394"></a>
-<span class="sourceLineNo">395</span>    RegionCoprocessorHost normalCPHost = region.getCoprocessorHost();<a name="line.395"></a>
-<span class="sourceLineNo">396</span>    RegionCoprocessorHost mockedCPHost = Mockito.mock(RegionCoprocessorHost.class);<a name="line.396"></a>
-<span class="sourceLineNo">397</span>    when(mockedCPHost.preFlush(Mockito.isA(HStore.class), Mockito.isA(InternalScanner.class))).<a name="line.397"></a>
-<span class="sourceLineNo">398</span>      thenReturn(null);<a name="line.398"></a>
-<span class="sourceLineNo">399</span>    region.setCoprocessorHost(mockedCPHost);<a name="line.399"></a>
-<span class="sourceLineNo">400</span>    region.put(put);<a name="line.400"></a>
-<span class="sourceLineNo">401</span>    region.flush(true);<a name="line.401"></a>
-<span class="sourceLineNo">402</span>    assertEquals("memstoreSize should NOT be zero", onePutSize, region.getMemstoreSize());<a name="line.402"></a>
-<span class="sourceLineNo">403</span>    assertEquals("flushable size should NOT be zero", onePutSize, store.getFlushableSize());<a name="line.403"></a>
-<span class="sourceLineNo">404</span><a name="line.404"></a>
-<span class="sourceLineNo">405</span>    // set normalCPHost and flush again, the snapshot will be flushed<a name="line.405"></a>
-<span class="sourceLineNo">406</span>    region.setCoprocessorHost(normalCPHost);<a name="line.406"></a>
+<span class="sourceLineNo">331</span>    FileSystem fs = FileSystem.get(CONF);<a name="line.331"></a>
+<span class="sourceLineNo">332</span>    Path rootDir = new Path(dir + "testMemstoreSnapshotSize");<a name="line.332"></a>
+<span class="sourceLineNo">333</span>    MyFaultyFSLog faultyLog = new MyFaultyFSLog(fs, rootDir, "testMemstoreSnapshotSize", CONF);<a name="line.333"></a>
+<span class="sourceLineNo">334</span>    HRegion region = initHRegion(tableName, null, null, name.getMethodName(),<a name="line.334"></a>
+<span class="sourceLineNo">335</span>        CONF, false, Durability.SYNC_WAL, faultyLog, COLUMN_FAMILY_BYTES);<a name="line.335"></a>
+<span class="sourceLineNo">336</span><a name="line.336"></a>
+<span class="sourceLineNo">337</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.337"></a>
+<span class="sourceLineNo">338</span>    // Get some random bytes.<a name="line.338"></a>
+<span class="sourceLineNo">339</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.339"></a>
+<span class="sourceLineNo">340</span>    faultyLog.setStoreFlushCtx(store.createFlushContext(12345));<a name="line.340"></a>
+<span class="sourceLineNo">341</span><a name="line.341"></a>
+<span class="sourceLineNo">342</span>    Put put = new Put(value);<a name="line.342"></a>
+<span class="sourceLineNo">343</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.343"></a>
+<span class="sourceLineNo">344</span>    faultyLog.setFailureType(FaultyFSLog.FailureType.SYNC);<a name="line.344"></a>
+<span class="sourceLineNo">345</span><a name="line.345"></a>
+<span class="sourceLineNo">346</span>    boolean threwIOE = false;<a name="line.346"></a>
+<span class="sourceLineNo">347</span>    try {<a name="line.347"></a>
+<span class="sourceLineNo">348</span>      region.put(put);<a name="line.348"></a>
+<span class="sourceLineNo">349</span>    } catch (IOException ioe) {<a name="line.349"></a>
+<span class="sourceLineNo">350</span>      threwIOE = true;<a name="line.350"></a>
+<span class="sourceLineNo">351</span>    } finally {<a name="line.351"></a>
+<span class="sourceLineNo">352</span>      assertTrue("The regionserver should have thrown an exception", threwIOE);<a name="line.352"></a>
+<span class="sourceLineNo">353</span>    }<a name="line.353"></a>
+<span class="sourceLineNo">354</span>    long sz = store.getFlushableSize();<a name="line.354"></a>
+<span class="sourceLineNo">355</span>    assertTrue("flushable size should be zero, but it is " + sz, sz == 0);<a name="line.355"></a>
+<span class="sourceLineNo">356</span>    HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.356"></a>
+<span class="sourceLineNo">357</span>  }<a name="line.357"></a>
+<span class="sourceLineNo">358</span><a name="line.358"></a>
+<span class="sourceLineNo">359</span>  /**<a name="line.359"></a>
+<span class="sourceLineNo">360</span>   * Create a WAL outside of the usual helper in<a name="line.360"></a>
+<span class="sourceLineNo">361</span>   * {@link HBaseTestingUtility#createWal(Configuration, Path, HRegionInfo)} because that method<a name="line.361"></a>
+<span class="sourceLineNo">362</span>   * doesn't play nicely with FaultyFileSystem. Call this method before overriding<a name="line.362"></a>
+<span class="sourceLineNo">363</span>   * {@code fs.file.impl}.<a name="line.363"></a>
+<span class="sourceLineNo">364</span>   * @param callingMethod a unique component for the path, probably the name of the test method.<a name="line.364"></a>
+<span class="sourceLineNo">365</span>   */<a name="line.365"></a>
+<span class="sourceLineNo">366</span>  private static WAL createWALCompatibleWithFaultyFileSystem(String callingMethod,<a name="line.366"></a>
+<span class="sourceLineNo">367</span>      Configuration conf, TableName tableName) throws IOException {<a name="line.367"></a>
+<span class="sourceLineNo">368</span>    final Path logDir = TEST_UTIL.getDataTestDirOnTestFS(callingMethod + ".log");<a name="line.368"></a>
+<span class="sourceLineNo">369</span>    final Configuration walConf = new Configuration(conf);<a name="line.369"></a>
+<span class="sourceLineNo">370</span>    FSUtils.setRootDir(walConf, logDir);<a name="line.370"></a>
+<span class="sourceLineNo">371</span>    return (new WALFactory(walConf,<a name="line.371"></a>
+<span class="sourceLineNo">372</span>        Collections.&lt;WALActionsListener&gt;singletonList(new MetricsWAL()), callingMethod))<a name="line.372"></a>
+<span class="sourceLineNo">373</span>        .getWAL(tableName.toBytes(), tableName.getNamespace());<a name="line.373"></a>
+<span class="sourceLineNo">374</span>  }<a name="line.374"></a>
+<span class="sourceLineNo">375</span><a name="line.375"></a>
+<span class="sourceLineNo">376</span>  /**<a name="line.376"></a>
+<span class="sourceLineNo">377</span>   * Test for HBASE-14229: Flushing canceled by coprocessor still leads to memstoreSize set down<a name="line.377"></a>
+<span class="sourceLineNo">378</span>   */<a name="line.378"></a>
+<span class="sourceLineNo">379</span>  @Test<a name="line.379"></a>
+<span class="sourceLineNo">380</span>  public void testMemstoreSizeWithFlushCanceling() throws IOException {<a name="line.380"></a>
+<span class="sourceLineNo">381</span>    FileSystem fs = FileSystem.get(CONF);<a name="line.381"></a>
+<span class="sourceLineNo">382</span>    Path rootDir = new Path(dir + "testMemstoreSizeWithFlushCanceling");<a name="line.382"></a>
+<span class="sourceLineNo">383</span>    FSHLog hLog = new FSHLog(fs, rootDir, "testMemstoreSizeWithFlushCanceling", CONF);<a name="line.383"></a>
+<span class="sourceLineNo">384</span>    HRegion region = initHRegion(tableName, null, null, name.getMethodName(),<a name="line.384"></a>
+<span class="sourceLineNo">385</span>        CONF, false, Durability.SYNC_WAL, hLog, COLUMN_FAMILY_BYTES);<a name="line.385"></a>
+<span class="sourceLineNo">386</span>    Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.386"></a>
+<span class="sourceLineNo">387</span>    assertEquals(0, region.getMemstoreSize());<a name="line.387"></a>
+<span class="sourceLineNo">388</span><a name="line.388"></a>
+<span class="sourceLineNo">389</span>    // Put some value and make sure flush could be completed normally<a name="line.389"></a>
+<span class="sourceLineNo">390</span>    byte [] value = Bytes.toBytes(name.getMethodName());<a name="line.390"></a>
+<span class="sourceLineNo">391</span>    Put put = new Put(value);<a name="line.391"></a>
+<span class="sourceLineNo">392</span>    put.addColumn(COLUMN_FAMILY_BYTES, Bytes.toBytes("abc"), value);<a name="line.392"></a>
+<span class="sourceLineNo">393</span>    region.put(put);<a name="line.393"></a>
+<span class="sourceLineNo">394</span>    long onePutSize = region.getMemstoreSize();<a name="line.394"></a>
+<span class="sourceLineNo">395</span>    assertTrue(onePutSize &gt; 0);<a name="line.395"></a>
+<span class="sourceLineNo">396</span>    region.flush(true);<a name="line.396"></a>
+<span class="sourceLineNo">397</span>    assertEquals("memstoreSize should be zero", 0, region.getMemstoreSize());<a name="line.397"></a>
+<span class="sourceLineNo">398</span>    assertEquals("flushable size should be zero", 0, store.getFlushableSize());<a name="line.398"></a>
+<span class="sourceLineNo">399</span><a name="line.399"></a>
+<span class="sourceLineNo">400</span>    // save normalCPHost and replaced by mockedCPHost, which will cancel flush requests<a name="line.400"></a>
+<span class="sourceLineNo">401</span>    RegionCoprocessorHost normalCPHost = region.getCoprocessorHost();<a name="line.401"></a>
+<span class="sourceLineNo">402</span>    RegionCoprocessorHost mockedCPHost = Mockito.mock(RegionCoprocessorHost.class);<a name="line.402"></a>
+<span class="sourceLineNo">403</span>    when(mockedCPHost.preFlush(Mockito.isA(HStore.class), Mockito.isA(InternalScanner.class))).<a name="line.403"></a>
+<span class="sourceLineNo">404</span>      thenReturn(null);<a name="line.404"></a>
+<span class="sourceLineNo">405</span>    region.setCoprocessorHost(mockedCPHost);<a name="line.405"></a>
+<span class="sourceLineNo">406</span>    region.put(put);<a name="line.406"></a>
 <span class="sourceLineNo">407</span>    region.flush(true);<a name="line.407"></a>
-<span class="sourceLineNo">408</span>    assertEquals("memstoreSize should be zero", 0, region.getMemstoreSize());<a name="line.408"></a>
-<span class="sourceLineNo">409</span>    assertEquals("flushable size should be zero", 0, store.getFlushableSize());<a name="line.409"></a>
-<span class="sourceLineNo">410</span>    HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.410"></a>
-<span class="sourceLineNo">411</span>  }<a name="line.411"></a>
-<span class="sourceLineNo">412</span><a name="line.412"></a>
-<span class="sourceLineNo">413</span>  /**<a name="line.413"></a>
-<span class="sourceLineNo">414</span>   * Test we do not lose data if we fail a flush and then close.<a name="line.414"></a>
-<span class="sourceLineNo">415</span>   * Part of HBase-10466.  Tests the following from the issue description:<a name="line.415"></a>
-<span class="sourceLineNo">416</span>   * "Bug 1: Wrong calculation of HRegion.memstoreSize: When a flush fails, data to be flushed is<a name="line.416"></a>
-<span class="sourceLineNo">417</span>   * kept in each MemStore's snapshot and wait for next flush attempt to continue on it. But when<a name="line.417"></a>
-<span class="sourceLineNo">418</span>   * the next flush succeeds, the counter of total memstore size in HRegion is always deduced by<a name="line.418"></a>
-<span class="sourceLineNo">419</span>   * the sum of current memstore sizes instead of snapshots left from previous failed flush. This<a name="line.419"></a>
-<span class="sourceLineNo">420</span>   * calculation is problematic that almost every time there is failed flush, HRegion.memstoreSize<a name="line.420"></a>
-<span class="sourceLineNo">421</span>   * gets reduced by a wrong value. If region flush could not proceed for a couple cycles, the size<a name="line.421"></a>
-<span class="sourceLineNo">422</span>   * in current memstore could be much larger than the snapshot. It's likely to drift memstoreSize<a name="line.422"></a>
-<span class="sourceLineNo">423</span>   * much smaller than expected. In extreme case, if the error accumulates to even bigger than<a name="line.423"></a>
-<span class="sourceLineNo">424</span>   * HRegion's memstore size limit, any further flush is skipped because flush does not do anything<a name="line.424"></a>
-<span class="sourceLineNo">425</span>   * if memstoreSize is not larger than 0."<a name="line.425"></a>
-<span class="sourceLineNo">426</span>   * @throws Exception<a name="line.426"></a>
-<span class="sourceLineNo">427</span>   */<a name="line.427"></a>
-<span class="sourceLineNo">428</span>  @Test (timeout=60000)<a name="line.428"></a>
-<span class="sourceLineNo">429</span>  public void testFlushSizeAccounting() throws Exception {<a name="line.429"></a>
-<span class="sourceLineNo">430</span>    final Configuration conf = HBaseConfiguration.create(CONF);<a name="line.430"></a>
-<span class="sourceLineNo">431</span>    final String callingMethod = name.getMethodName();<a name="line.431"></a>
-<span class="sourceLineNo">432</span>    final WAL wal = createWALCompatibleWithFaultyFileSystem(callingMethod, conf, tableName);<a name="line.432"></a>
-<span class="sourceLineNo">433</span>    // Only retry once.<a name="line.433"></a>
-<span class="sourceLineNo">434</span>    conf.setInt("hbase.hstore.flush.retries.number", 1);<a name="line.434"></a>
-<span class="sourceLineNo">435</span>    final User user =<a name="line.435"></a>
-<span class="sourceLineNo">436</span>      User.createUserForTesting(conf, this.name.getMethodName(), new String[]{"foo"});<a name="line.436"></a>
-<span class="sourceLineNo">437</span>    // Inject our faulty LocalFileSystem<a name="line.437"></a>
-<span class="sourceLineNo">438</span>    conf.setClass("fs.file.impl", FaultyFileSystem.class, FileSystem.class);<a name="line.438"></a>
-<span class="sourceLineNo">439</span>    user.runAs(new PrivilegedExceptionAction&lt;Object&gt;() {<a name="line.439"></a>
-<span class="sourceLineNo">440</span>      @Override<a name="line.440"></a>
-<span class="sourceLineNo">441</span>      public Object run() throws Exception {<a name="line.441"></a>
-<span class="sourceLineNo">442</span>        // Make sure it worked (above is sensitive to caching details in hadoop core)<a name="line.442"></a>
-<span class="sourceLineNo">443</span>        FileSystem fs = FileSystem.get(conf);<a name="line.443"></a>
-<span class="sourceLineNo">444</span>        Assert.assertEquals(FaultyFileSystem.class, fs.getClass());<a name="line.444"></a>
-<span class="sourceLineNo">445</span>        FaultyFileSystem ffs = (FaultyFileSystem)fs;<a name="line.445"></a>
-<span class="sourceLineNo">446</span>        HRegion region = null;<a name="line.446"></a>
-<span class="sourceLineNo">447</span>        try {<a name="line.447"></a>
-<span class="sourceLineNo">448</span>          // Initialize region<a name="line.448"></a>
-<span class="sourceLineNo">449</span>          region = initHRegion(tableName, null, null, callingMethod, conf, false,<a name="line.449"></a>
-<span class="sourceLineNo">450</span>              Durability.SYNC_WAL, wal, COLUMN_FAMILY_BYTES);<a name="line.450"></a>
-<span class="sourceLineNo">451</span>          long size = region.getMemstoreSize();<a name="line.451"></a>
-<span class="sourceLineNo">452</span>          Assert.assertEquals(0, size);<a name="line.452"></a>
-<span class="sourceLineNo">453</span>          // Put one item into memstore.  Measure the size of one item in memstore.<a name="line.453"></a>
-<span class="sourceLineNo">454</span>          Put p1 = new Put(row);<a name="line.454"></a>
-<span class="sourceLineNo">455</span>          p1.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual1, 1, (byte[]) null));<a name="line.455"></a>
-<span class="sourceLineNo">456</span>          region.put(p1);<a name="line.456"></a>
-<span class="sourceLineNo">457</span>          final long sizeOfOnePut = region.getMemstoreSize();<a name="line.457"></a>
-<span class="sourceLineNo">458</span>          // Fail a flush which means the current memstore will hang out as memstore 'snapshot'.<a name="line.458"></a>
-<span class="sourceLineNo">459</span>          try {<a name="line.459"></a>
-<span class="sourceLineNo">460</span>            LOG.info("Flushing");<a name="line.460"></a>
-<span class="sourceLineNo">461</span>            region.flush(true);<a name="line.461"></a>
-<span class="sourceLineNo">462</span>            Assert.fail("Didn't bubble up IOE!");<a name="line.462"></a>
-<span class="sourceLineNo">463</span>          } catch (DroppedSnapshotException dse) {<a name="line.463"></a>
-<span class="sourceLineNo">464</span>            // What we are expecting<a name="line.464"></a>
-<span class="sourceLineNo">465</span>            region.closing.set(false); // this is needed for the rest of the test to work<a name="line.465"></a>
-<span class="sourceLineNo">466</span>          }<a name="line.466"></a>
-<span class="sourceLineNo">467</span>          // Make it so all writes succeed from here on out<a name="line.467"></a>
-<span class="sourceLineNo">468</span>          ffs.fault.set(false);<a name="line.468"></a>
-<span class="sourceLineNo">469</span>          // Check sizes.  Should still be the one entry.<a name="line.469"></a>
-<span class="sourceLineNo">470</span>          Assert.assertEquals(sizeOfOnePut, region.getMemstoreSize());<a name="line.470"></a>
-<span class="sourceLineNo">471</span>          // Now add two entries so that on this next flush that fails, we can see if we<a name="line.471"></a>
-<span class="sourceLineNo">472</span>          // subtract the right amount, the snapshot size only.<a name="line.472"></a>
-<span class="sourceLineNo">473</span>          Put p2 = new Put(row);<a name="line.473"></a>
-<span class="sourceLineNo">474</span>          p2.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual2, 2, (byte[])null));<a name="line.474"></a>
-<span class="sourceLineNo">475</span>          p2.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual3, 3, (byte[])null));<a name="line.475"></a>
-<span class="sourceLineNo">476</span>          region.put(p2);<a name="line.476"></a>
-<span class="sourceLineNo">477</span>          Assert.assertEquals(sizeOfOnePut * 3, region.getMemstoreSize());<a name="line.477"></a>
-<span class="sourceLineNo">478</span>          // Do a successful flush.  It will clear the snapshot only.  Thats how flushes work.<a name="line.478"></a>
-<span class="sourceLineNo">479</span>          // If already a snapshot, we clear it else we move the memstore to be snapshot and flush<a name="line.479"></a>
-<span class="sourceLineNo">480</span>          // it<a name="line.480"></a>
-<span class="sourceLineNo">481</span>          region.flush(true);<a name="line.481"></a>
-<span class="sourceLineNo">482</span>          // Make sure our memory accounting is right.<a name="line.482"></a>
-<span class="sourceLineNo">483</span>          Assert.assertEquals(sizeOfOnePut * 2, region.getMemstoreSize());<a name="line.483"></a>
-<span class="sourceLineNo">484</span>        } finally {<a name="line.484"></a>
-<span class="sourceLineNo">485</span>          HBaseTestingUtility.closeRegionAndWAL(region);<a name="line.485"></a>
-<span class="sourceLineNo">486</span>        }<a name="line.486"></a>
-<span class="sourceLineNo">487</span>        return null;<a name="line.487"></a>
-<span class="sourceLineNo">488</span>      }<a name="line.488"></a>
-<span class="sourceLineNo">489</span>    });<a name="line.489"></a>
-<span class="sourceLineNo">490</span>    FileSystem.closeAllForUGI(user.getUGI());<a name="line.490"></a>
-<span class="sourceLineNo">491</span>  }<a name="line.491"></a>
-<span class="sourceLineNo">492</span><a name="line.492"></a>
-<span class="sourceLineNo">493</span>  @Test (timeout=60000)<a name="line.493"></a>
-<span class="sourceLineNo">494</span>  public void testCloseWithFailingFlush() throws Exception {<a name="line.494"></a>
-<span class="sourceLineNo">495</span>    final Configuration conf = HBaseConfiguration.create(CONF);<a name="line.495"></a>
-<span class="sourceLineNo">496</span>    final String callingMethod = name.getMethodName();<a name="line.496"></a>
-<span class="sourceLineNo">497</span>    final WAL wal = createWALCompatibleWithFaultyFileSystem(callingMethod, conf, tableName);<a name="line.497"></a>
-<span class="sourceLineNo">498</span>    // Only retry once.<a name="line.498"></a>
-<span class="sourceLineNo">499</span>    conf.setInt("hbase.hstore.flush.retries.number", 1);<a name="line.499"></a>
-<span class="sourceLineNo">500</span>    final User user =<a name="line.500"></a>
-<span class="sourceLineNo">501</span>      User.createUserForTesting(conf, this.name.getMethodName(), new String[]{"foo"});<a name="line.501"></a>
-<span class="sourceLineNo">502</span>    // Inject our faulty LocalFileSystem<a name="line.502"></a>
-<span class="sourceLineNo">503</span>    conf.setClass("fs.file.impl", FaultyFileSystem.class, FileSystem.class);<a name="line.503"></a>
-<span class="sourceLineNo">504</span>    user.runAs(new PrivilegedExceptionAction&lt;Object&gt;() {<a name="line.504"></a>
-<span class="sourceLineNo">505</span>      @Override<a name="line.505"></a>
-<span class="sourceLineNo">506</span>      public Object run() throws Exception {<a name="line.506"></a>
-<span class="sourceLineNo">507</span>        // Make sure it worked (above is sensitive to caching details in hadoop core)<a name="line.507"></a>
-<span class="sourceLineNo">508</span>        FileSystem fs = FileSystem.get(conf);<a name="line.508"></a>
-<span class="sourceLineNo">509</span>        Assert.assertEquals(FaultyFileSystem.class, fs.getClass());<a name="line.509"></a>
-<span class="sourceLineNo">510</span>        FaultyFileSystem ffs = (FaultyFileSystem)fs;<a name="line.510"></a>
-<span class="sourceLineNo">511</span>        HRegion region = null;<a name="line.511"></a>
-<span class="sourceLineNo">512</span>        try {<a name="line.512"></a>
-<span class="sourceLineNo">513</span>          // Initialize region<a name="line.513"></a>
-<span class="sourceLineNo">514</span>          region = initHRegion(tableName, null, null, callingMethod, conf, false,<a name="line.514"></a>
-<span class="sourceLineNo">515</span>              Durability.SYNC_WAL, wal, COLUMN_FAMILY_BYTES);<a name="line.515"></a>
-<span class="sourceLineNo">516</span>          long size = region.getMemstoreSize();<a name="line.516"></a>
-<span class="sourceLineNo">517</span>          Assert.assertEquals(0, size);<a name="line.517"></a>
-<span class="sourceLineNo">518</span>          // Put one item into memstore.  Measure the size of one item in memstore.<a name="line.518"></a>
-<span class="sourceLineNo">519</span>          Put p1 = new Put(row);<a name="line.519"></a>
-<span class="sourceLineNo">520</span>          p1.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual1, 1, (byte[])null));<a name="line.520"></a>
-<span class="sourceLineNo">521</span>          region.put(p1);<a name="line.521"></a>
-<span class="sourceLineNo">522</span>          // Manufacture an outstanding snapshot -- fake a failed flush by doing prepare step only.<a name="line.522"></a>
-<span class="sourceLineNo">523</span>          Store store = region.getStore(COLUMN_FAMILY_BYTES);<a name="line.523"></a>
-<span class="sourceLineNo">524</span>          StoreFlushContext storeFlushCtx = store.createFlushContext(12345);<a name="line.524"></a>
-<span class="sourceLineNo">525</span>          storeFlushCtx.prepare();<a name="line.525"></a>
-<span class="sourceLineNo">526</span>          // Now add two entries to the foreground memstore.<a name="line.526"></a>
-<span class="sourceLineNo">527</span>          Put p2 = new Put(row);<a name="line.527"></a>
-<span class="sourceLineNo">528</span>          p2.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual2, 2, (byte[])null));<a name="line.528"></a>
-<span class="sourceLineNo">529</span>          p2.add(new KeyValue(row, COLUMN_FAMILY_BYTES, qual3, 3, (byte[])null));<a name="line.529"></a>
-<span class="sourceLineNo">530</span>          region.put(p2);<a name="line.530"></a>
-<span class="sourceLineNo">531</span>          // Now try close on top of a failing flush.<a name="line.531"></a>
-<span class="sourceLineNo">532</span>          region.close();<a name="line.532"></a>
-<span class="sourceLineNo">533</span>          fail();<a name="line.533"></a>
-<span class="sourceLineNo">534</span>        } catch (DroppedSnapshotException dse) {<a name="line.534"></a>
-<span class="sourceLineNo">535</span>          // Expected<a name="line.535"></a>
-<span class="sourceLineNo">536</span>          LOG.info("Expe

<TRUNCATED>

Mime
View raw message