| <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> |
| <html lang="en"> |
| <head> |
| <title>Source code</title> |
| <link rel="stylesheet" type="text/css" href="../../../../../../stylesheet.css" title="Style"> |
| </head> |
| <body> |
| <div class="sourceContainer"> |
| <pre><span class="sourceLineNo">001</span>/*<a name="line.1"></a> |
| <span class="sourceLineNo">002</span> * Licensed to the Apache Software Foundation (ASF) under one<a name="line.2"></a> |
| <span class="sourceLineNo">003</span> * or more contributor license agreements. See the NOTICE file<a name="line.3"></a> |
| <span class="sourceLineNo">004</span> * distributed with this work for additional information<a name="line.4"></a> |
| <span class="sourceLineNo">005</span> * regarding copyright ownership. The ASF licenses this file<a name="line.5"></a> |
| <span class="sourceLineNo">006</span> * to you under the Apache License, Version 2.0 (the<a name="line.6"></a> |
| <span class="sourceLineNo">007</span> * "License"); you may not use this file except in compliance<a name="line.7"></a> |
| <span class="sourceLineNo">008</span> * with the License. You may obtain a copy of the License at<a name="line.8"></a> |
| <span class="sourceLineNo">009</span> *<a name="line.9"></a> |
| <span class="sourceLineNo">010</span> * http://www.apache.org/licenses/LICENSE-2.0<a name="line.10"></a> |
| <span class="sourceLineNo">011</span> *<a name="line.11"></a> |
| <span class="sourceLineNo">012</span> * Unless required by applicable law or agreed to in writing, software<a name="line.12"></a> |
| <span class="sourceLineNo">013</span> * distributed under the License is distributed on an "AS IS" BASIS,<a name="line.13"></a> |
| <span class="sourceLineNo">014</span> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.<a name="line.14"></a> |
| <span class="sourceLineNo">015</span> * See the License for the specific language governing permissions and<a name="line.15"></a> |
| <span class="sourceLineNo">016</span> * limitations under the License.<a name="line.16"></a> |
| <span class="sourceLineNo">017</span> */<a name="line.17"></a> |
| <span class="sourceLineNo">018</span>package org.apache.hadoop.hbase.regionserver;<a name="line.18"></a> |
| <span class="sourceLineNo">019</span><a name="line.19"></a> |
| <span class="sourceLineNo">020</span>import static org.apache.hadoop.hbase.HBaseTestingUtil.START_KEY;<a name="line.20"></a> |
| <span class="sourceLineNo">021</span>import static org.apache.hadoop.hbase.HBaseTestingUtil.START_KEY_BYTES;<a name="line.21"></a> |
| <span class="sourceLineNo">022</span>import static org.apache.hadoop.hbase.HBaseTestingUtil.fam1;<a name="line.22"></a> |
| <span class="sourceLineNo">023</span>import static org.apache.hadoop.hbase.regionserver.Store.PRIORITY_USER;<a name="line.23"></a> |
| <span class="sourceLineNo">024</span>import static org.apache.hadoop.hbase.regionserver.compactions.CloseChecker.SIZE_LIMIT_KEY;<a name="line.24"></a> |
| <span class="sourceLineNo">025</span>import static org.apache.hadoop.hbase.regionserver.compactions.CloseChecker.TIME_LIMIT_KEY;<a name="line.25"></a> |
| <span class="sourceLineNo">026</span>import static org.junit.Assert.assertEquals;<a name="line.26"></a> |
| <span class="sourceLineNo">027</span>import static org.junit.Assert.assertFalse;<a name="line.27"></a> |
| <span class="sourceLineNo">028</span>import static org.junit.Assert.assertThrows;<a name="line.28"></a> |
| <span class="sourceLineNo">029</span>import static org.junit.Assert.assertTrue;<a name="line.29"></a> |
| <span class="sourceLineNo">030</span>import static org.junit.Assert.fail;<a name="line.30"></a> |
| <span class="sourceLineNo">031</span>import static org.mockito.ArgumentMatchers.any;<a name="line.31"></a> |
| <span class="sourceLineNo">032</span>import static org.mockito.Mockito.doAnswer;<a name="line.32"></a> |
| <span class="sourceLineNo">033</span>import static org.mockito.Mockito.mock;<a name="line.33"></a> |
| <span class="sourceLineNo">034</span>import static org.mockito.Mockito.spy;<a name="line.34"></a> |
| <span class="sourceLineNo">035</span>import static org.mockito.Mockito.when;<a name="line.35"></a> |
| <span class="sourceLineNo">036</span><a name="line.36"></a> |
| <span class="sourceLineNo">037</span>import java.io.IOException;<a name="line.37"></a> |
| <span class="sourceLineNo">038</span>import java.util.ArrayList;<a name="line.38"></a> |
| <span class="sourceLineNo">039</span>import java.util.Collection;<a name="line.39"></a> |
| <span class="sourceLineNo">040</span>import java.util.Collections;<a name="line.40"></a> |
| <span class="sourceLineNo">041</span>import java.util.List;<a name="line.41"></a> |
| <span class="sourceLineNo">042</span>import java.util.Optional;<a name="line.42"></a> |
| <span class="sourceLineNo">043</span>import java.util.concurrent.CountDownLatch;<a name="line.43"></a> |
| <span class="sourceLineNo">044</span>import java.util.concurrent.TimeUnit;<a name="line.44"></a> |
| <span class="sourceLineNo">045</span>import org.apache.hadoop.conf.Configuration;<a name="line.45"></a> |
| <span class="sourceLineNo">046</span>import org.apache.hadoop.fs.FSDataOutputStream;<a name="line.46"></a> |
| <span class="sourceLineNo">047</span>import org.apache.hadoop.fs.FileStatus;<a name="line.47"></a> |
| <span class="sourceLineNo">048</span>import org.apache.hadoop.fs.FileSystem;<a name="line.48"></a> |
| <span class="sourceLineNo">049</span>import org.apache.hadoop.fs.Path;<a name="line.49"></a> |
| <span class="sourceLineNo">050</span>import org.apache.hadoop.hbase.ChoreService;<a name="line.50"></a> |
| <span class="sourceLineNo">051</span>import org.apache.hadoop.hbase.HBaseClassTestRule;<a name="line.51"></a> |
| <span class="sourceLineNo">052</span>import org.apache.hadoop.hbase.HBaseConfiguration;<a name="line.52"></a> |
| <span class="sourceLineNo">053</span>import org.apache.hadoop.hbase.HBaseTestingUtil;<a name="line.53"></a> |
| <span class="sourceLineNo">054</span>import org.apache.hadoop.hbase.HConstants;<a name="line.54"></a> |
| <span class="sourceLineNo">055</span>import org.apache.hadoop.hbase.HTestConst;<a name="line.55"></a> |
| <span class="sourceLineNo">056</span>import org.apache.hadoop.hbase.Waiter;<a name="line.56"></a> |
| <span class="sourceLineNo">057</span>import org.apache.hadoop.hbase.client.ColumnFamilyDescriptor;<a name="line.57"></a> |
| <span class="sourceLineNo">058</span>import org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder;<a name="line.58"></a> |
| <span class="sourceLineNo">059</span>import org.apache.hadoop.hbase.client.Delete;<a name="line.59"></a> |
| <span class="sourceLineNo">060</span>import org.apache.hadoop.hbase.client.Durability;<a name="line.60"></a> |
| <span class="sourceLineNo">061</span>import org.apache.hadoop.hbase.client.Put;<a name="line.61"></a> |
| <span class="sourceLineNo">062</span>import org.apache.hadoop.hbase.client.Table;<a name="line.62"></a> |
| <span class="sourceLineNo">063</span>import org.apache.hadoop.hbase.client.TableDescriptor;<a name="line.63"></a> |
| <span class="sourceLineNo">064</span>import org.apache.hadoop.hbase.client.TableDescriptorBuilder;<a name="line.64"></a> |
| <span class="sourceLineNo">065</span>import org.apache.hadoop.hbase.io.hfile.HFileScanner;<a name="line.65"></a> |
| <span class="sourceLineNo">066</span>import org.apache.hadoop.hbase.regionserver.compactions.CompactionContext;<a name="line.66"></a> |
| <span class="sourceLineNo">067</span>import org.apache.hadoop.hbase.regionserver.compactions.CompactionLifeCycleTracker;<a name="line.67"></a> |
| <span class="sourceLineNo">068</span>import org.apache.hadoop.hbase.regionserver.compactions.CompactionRequestImpl;<a name="line.68"></a> |
| <span class="sourceLineNo">069</span>import org.apache.hadoop.hbase.regionserver.compactions.DefaultCompactor;<a name="line.69"></a> |
| <span class="sourceLineNo">070</span>import org.apache.hadoop.hbase.regionserver.throttle.CompactionThroughputControllerFactory;<a name="line.70"></a> |
| <span class="sourceLineNo">071</span>import org.apache.hadoop.hbase.regionserver.throttle.NoLimitThroughputController;<a name="line.71"></a> |
| <span class="sourceLineNo">072</span>import org.apache.hadoop.hbase.regionserver.throttle.ThroughputController;<a name="line.72"></a> |
| <span class="sourceLineNo">073</span>import org.apache.hadoop.hbase.security.User;<a name="line.73"></a> |
| <span class="sourceLineNo">074</span>import org.apache.hadoop.hbase.testclassification.MediumTests;<a name="line.74"></a> |
| <span class="sourceLineNo">075</span>import org.apache.hadoop.hbase.testclassification.RegionServerTests;<a name="line.75"></a> |
| <span class="sourceLineNo">076</span>import org.apache.hadoop.hbase.util.Bytes;<a name="line.76"></a> |
| <span class="sourceLineNo">077</span>import org.apache.hadoop.hbase.util.EnvironmentEdgeManager;<a name="line.77"></a> |
| <span class="sourceLineNo">078</span>import org.apache.hadoop.hbase.util.Threads;<a name="line.78"></a> |
| <span class="sourceLineNo">079</span>import org.apache.hadoop.hbase.wal.WAL;<a name="line.79"></a> |
| <span class="sourceLineNo">080</span>import org.junit.After;<a name="line.80"></a> |
| <span class="sourceLineNo">081</span>import org.junit.Assume;<a name="line.81"></a> |
| <span class="sourceLineNo">082</span>import org.junit.Before;<a name="line.82"></a> |
| <span class="sourceLineNo">083</span>import org.junit.ClassRule;<a name="line.83"></a> |
| <span class="sourceLineNo">084</span>import org.junit.Rule;<a name="line.84"></a> |
| <span class="sourceLineNo">085</span>import org.junit.Test;<a name="line.85"></a> |
| <span class="sourceLineNo">086</span>import org.junit.experimental.categories.Category;<a name="line.86"></a> |
| <span class="sourceLineNo">087</span>import org.junit.rules.TestName;<a name="line.87"></a> |
| <span class="sourceLineNo">088</span>import org.mockito.Mockito;<a name="line.88"></a> |
| <span class="sourceLineNo">089</span>import org.mockito.invocation.InvocationOnMock;<a name="line.89"></a> |
| <span class="sourceLineNo">090</span>import org.mockito.stubbing.Answer;<a name="line.90"></a> |
| <span class="sourceLineNo">091</span><a name="line.91"></a> |
| <span class="sourceLineNo">092</span>/**<a name="line.92"></a> |
| <span class="sourceLineNo">093</span> * Test compaction framework and common functions<a name="line.93"></a> |
| <span class="sourceLineNo">094</span> */<a name="line.94"></a> |
| <span class="sourceLineNo">095</span>@Category({RegionServerTests.class, MediumTests.class})<a name="line.95"></a> |
| <span class="sourceLineNo">096</span>public class TestCompaction {<a name="line.96"></a> |
| <span class="sourceLineNo">097</span><a name="line.97"></a> |
| <span class="sourceLineNo">098</span> @ClassRule<a name="line.98"></a> |
| <span class="sourceLineNo">099</span> public static final HBaseClassTestRule CLASS_RULE =<a name="line.99"></a> |
| <span class="sourceLineNo">100</span> HBaseClassTestRule.forClass(TestCompaction.class);<a name="line.100"></a> |
| <span class="sourceLineNo">101</span><a name="line.101"></a> |
| <span class="sourceLineNo">102</span> @Rule<a name="line.102"></a> |
| <span class="sourceLineNo">103</span> public TestName name = new TestName();<a name="line.103"></a> |
| <span class="sourceLineNo">104</span> private static final HBaseTestingUtil UTIL = new HBaseTestingUtil();<a name="line.104"></a> |
| <span class="sourceLineNo">105</span> protected Configuration conf = UTIL.getConfiguration();<a name="line.105"></a> |
| <span class="sourceLineNo">106</span><a name="line.106"></a> |
| <span class="sourceLineNo">107</span> private HRegion r = null;<a name="line.107"></a> |
| <span class="sourceLineNo">108</span> private TableDescriptor tableDescriptor = null;<a name="line.108"></a> |
| <span class="sourceLineNo">109</span> private static final byte [] COLUMN_FAMILY = fam1;<a name="line.109"></a> |
| <span class="sourceLineNo">110</span> private final byte [] STARTROW = Bytes.toBytes(START_KEY);<a name="line.110"></a> |
| <span class="sourceLineNo">111</span> private static final byte [] COLUMN_FAMILY_TEXT = COLUMN_FAMILY;<a name="line.111"></a> |
| <span class="sourceLineNo">112</span> private int compactionThreshold;<a name="line.112"></a> |
| <span class="sourceLineNo">113</span> private byte[] secondRowBytes, thirdRowBytes;<a name="line.113"></a> |
| <span class="sourceLineNo">114</span> private static final long MAX_FILES_TO_COMPACT = 10;<a name="line.114"></a> |
| <span class="sourceLineNo">115</span> private final byte[] FAMILY = Bytes.toBytes("cf");<a name="line.115"></a> |
| <span class="sourceLineNo">116</span><a name="line.116"></a> |
| <span class="sourceLineNo">117</span> /** constructor */<a name="line.117"></a> |
| <span class="sourceLineNo">118</span> public TestCompaction() {<a name="line.118"></a> |
| <span class="sourceLineNo">119</span> super();<a name="line.119"></a> |
| <span class="sourceLineNo">120</span><a name="line.120"></a> |
| <span class="sourceLineNo">121</span> // Set cache flush size to 1MB<a name="line.121"></a> |
| <span class="sourceLineNo">122</span> conf.setInt(HConstants.HREGION_MEMSTORE_FLUSH_SIZE, 1024 * 1024);<a name="line.122"></a> |
| <span class="sourceLineNo">123</span> conf.setInt(HConstants.HREGION_MEMSTORE_BLOCK_MULTIPLIER, 100);<a name="line.123"></a> |
| <span class="sourceLineNo">124</span> conf.set(CompactionThroughputControllerFactory.HBASE_THROUGHPUT_CONTROLLER_KEY,<a name="line.124"></a> |
| <span class="sourceLineNo">125</span> NoLimitThroughputController.class.getName());<a name="line.125"></a> |
| <span class="sourceLineNo">126</span> compactionThreshold = conf.getInt("hbase.hstore.compactionThreshold", 3);<a name="line.126"></a> |
| <span class="sourceLineNo">127</span><a name="line.127"></a> |
| <span class="sourceLineNo">128</span> secondRowBytes = START_KEY_BYTES.clone();<a name="line.128"></a> |
| <span class="sourceLineNo">129</span> // Increment the least significant character so we get to next row.<a name="line.129"></a> |
| <span class="sourceLineNo">130</span> secondRowBytes[START_KEY_BYTES.length - 1]++;<a name="line.130"></a> |
| <span class="sourceLineNo">131</span> thirdRowBytes = START_KEY_BYTES.clone();<a name="line.131"></a> |
| <span class="sourceLineNo">132</span> thirdRowBytes[START_KEY_BYTES.length - 1] =<a name="line.132"></a> |
| <span class="sourceLineNo">133</span> (byte) (thirdRowBytes[START_KEY_BYTES.length - 1] + 2);<a name="line.133"></a> |
| <span class="sourceLineNo">134</span> }<a name="line.134"></a> |
| <span class="sourceLineNo">135</span><a name="line.135"></a> |
| <span class="sourceLineNo">136</span> @Before<a name="line.136"></a> |
| <span class="sourceLineNo">137</span> public void setUp() throws Exception {<a name="line.137"></a> |
| <span class="sourceLineNo">138</span> TableDescriptorBuilder builder = UTIL.createModifyableTableDescriptor(name.getMethodName());<a name="line.138"></a> |
| <span class="sourceLineNo">139</span> if (name.getMethodName().equals("testCompactionSeqId")) {<a name="line.139"></a> |
| <span class="sourceLineNo">140</span> UTIL.getConfiguration().set("hbase.hstore.compaction.kv.max", "10");<a name="line.140"></a> |
| <span class="sourceLineNo">141</span> UTIL.getConfiguration().set(DefaultStoreEngine.DEFAULT_COMPACTOR_CLASS_KEY,<a name="line.141"></a> |
| <span class="sourceLineNo">142</span> DummyCompactor.class.getName());<a name="line.142"></a> |
| <span class="sourceLineNo">143</span> ColumnFamilyDescriptor familyDescriptor =<a name="line.143"></a> |
| <span class="sourceLineNo">144</span> ColumnFamilyDescriptorBuilder.newBuilder(FAMILY).setMaxVersions(65536).build();<a name="line.144"></a> |
| <span class="sourceLineNo">145</span> builder.setColumnFamily(familyDescriptor);<a name="line.145"></a> |
| <span class="sourceLineNo">146</span> }<a name="line.146"></a> |
| <span class="sourceLineNo">147</span> this.tableDescriptor = builder.build();<a name="line.147"></a> |
| <span class="sourceLineNo">148</span> this.r = UTIL.createLocalHRegion(tableDescriptor, null, null);<a name="line.148"></a> |
| <span class="sourceLineNo">149</span> }<a name="line.149"></a> |
| <span class="sourceLineNo">150</span><a name="line.150"></a> |
| <span class="sourceLineNo">151</span> @After<a name="line.151"></a> |
| <span class="sourceLineNo">152</span> public void tearDown() throws Exception {<a name="line.152"></a> |
| <span class="sourceLineNo">153</span> WAL wal = r.getWAL();<a name="line.153"></a> |
| <span class="sourceLineNo">154</span> this.r.close();<a name="line.154"></a> |
| <span class="sourceLineNo">155</span> wal.close();<a name="line.155"></a> |
| <span class="sourceLineNo">156</span> }<a name="line.156"></a> |
| <span class="sourceLineNo">157</span><a name="line.157"></a> |
| <span class="sourceLineNo">158</span> /**<a name="line.158"></a> |
| <span class="sourceLineNo">159</span> * Verify that you can stop a long-running compaction<a name="line.159"></a> |
| <span class="sourceLineNo">160</span> * (used during RS shutdown)<a name="line.160"></a> |
| <span class="sourceLineNo">161</span> */<a name="line.161"></a> |
| <span class="sourceLineNo">162</span> @Test<a name="line.162"></a> |
| <span class="sourceLineNo">163</span> public void testInterruptCompactionBySize() throws Exception {<a name="line.163"></a> |
| <span class="sourceLineNo">164</span> assertEquals(0, count());<a name="line.164"></a> |
| <span class="sourceLineNo">165</span><a name="line.165"></a> |
| <span class="sourceLineNo">166</span> // lower the polling interval for this test<a name="line.166"></a> |
| <span class="sourceLineNo">167</span> conf.setInt(SIZE_LIMIT_KEY, 10 * 1000 /* 10 KB */);<a name="line.167"></a> |
| <span class="sourceLineNo">168</span><a name="line.168"></a> |
| <span class="sourceLineNo">169</span> try {<a name="line.169"></a> |
| <span class="sourceLineNo">170</span> // Create a couple store files w/ 15KB (over 10KB interval)<a name="line.170"></a> |
| <span class="sourceLineNo">171</span> int jmax = (int) Math.ceil(15.0/compactionThreshold);<a name="line.171"></a> |
| <span class="sourceLineNo">172</span> byte [] pad = new byte[1000]; // 1 KB chunk<a name="line.172"></a> |
| <span class="sourceLineNo">173</span> for (int i = 0; i < compactionThreshold; i++) {<a name="line.173"></a> |
| <span class="sourceLineNo">174</span> Table loader = new RegionAsTable(r);<a name="line.174"></a> |
| <span class="sourceLineNo">175</span> Put p = new Put(Bytes.add(STARTROW, Bytes.toBytes(i)));<a name="line.175"></a> |
| <span class="sourceLineNo">176</span> p.setDurability(Durability.SKIP_WAL);<a name="line.176"></a> |
| <span class="sourceLineNo">177</span> for (int j = 0; j < jmax; j++) {<a name="line.177"></a> |
| <span class="sourceLineNo">178</span> p.addColumn(COLUMN_FAMILY, Bytes.toBytes(j), pad);<a name="line.178"></a> |
| <span class="sourceLineNo">179</span> }<a name="line.179"></a> |
| <span class="sourceLineNo">180</span> HTestConst.addContent(loader, Bytes.toString(COLUMN_FAMILY));<a name="line.180"></a> |
| <span class="sourceLineNo">181</span> loader.put(p);<a name="line.181"></a> |
| <span class="sourceLineNo">182</span> r.flush(true);<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> HRegion spyR = spy(r);<a name="line.185"></a> |
| <span class="sourceLineNo">186</span> doAnswer(new Answer<Object>() {<a name="line.186"></a> |
| <span class="sourceLineNo">187</span> @Override<a name="line.187"></a> |
| <span class="sourceLineNo">188</span> public Object answer(InvocationOnMock invocation) throws Throwable {<a name="line.188"></a> |
| <span class="sourceLineNo">189</span> r.writestate.writesEnabled = false;<a name="line.189"></a> |
| <span class="sourceLineNo">190</span> return invocation.callRealMethod();<a name="line.190"></a> |
| <span class="sourceLineNo">191</span> }<a name="line.191"></a> |
| <span class="sourceLineNo">192</span> }).when(spyR).doRegionCompactionPrep();<a name="line.192"></a> |
| <span class="sourceLineNo">193</span><a name="line.193"></a> |
| <span class="sourceLineNo">194</span> // force a minor compaction, but not before requesting a stop<a name="line.194"></a> |
| <span class="sourceLineNo">195</span> spyR.compactStores();<a name="line.195"></a> |
| <span class="sourceLineNo">196</span><a name="line.196"></a> |
| <span class="sourceLineNo">197</span> // ensure that the compaction stopped, all old files are intact,<a name="line.197"></a> |
| <span class="sourceLineNo">198</span> HStore s = r.getStore(COLUMN_FAMILY);<a name="line.198"></a> |
| <span class="sourceLineNo">199</span> assertEquals(compactionThreshold, s.getStorefilesCount());<a name="line.199"></a> |
| <span class="sourceLineNo">200</span> assertTrue(s.getStorefilesSize() > 15*1000);<a name="line.200"></a> |
| <span class="sourceLineNo">201</span> // and no new store files persisted past compactStores()<a name="line.201"></a> |
| <span class="sourceLineNo">202</span> // only one empty dir exists in temp dir<a name="line.202"></a> |
| <span class="sourceLineNo">203</span> FileStatus[] ls = r.getFilesystem().listStatus(r.getRegionFileSystem().getTempDir());<a name="line.203"></a> |
| <span class="sourceLineNo">204</span> assertEquals(1, ls.length);<a name="line.204"></a> |
| <span class="sourceLineNo">205</span> Path storeTempDir = new Path(r.getRegionFileSystem().getTempDir(), Bytes.toString(COLUMN_FAMILY));<a name="line.205"></a> |
| <span class="sourceLineNo">206</span> assertTrue(r.getFilesystem().exists(storeTempDir));<a name="line.206"></a> |
| <span class="sourceLineNo">207</span> ls = r.getFilesystem().listStatus(storeTempDir);<a name="line.207"></a> |
| <span class="sourceLineNo">208</span> assertEquals(0, ls.length);<a name="line.208"></a> |
| <span class="sourceLineNo">209</span> } finally {<a name="line.209"></a> |
| <span class="sourceLineNo">210</span> // don't mess up future tests<a name="line.210"></a> |
| <span class="sourceLineNo">211</span> r.writestate.writesEnabled = true;<a name="line.211"></a> |
| <span class="sourceLineNo">212</span> conf.setInt(SIZE_LIMIT_KEY, 10 * 1000 * 1000 /* 10 MB */);<a name="line.212"></a> |
| <span class="sourceLineNo">213</span><a name="line.213"></a> |
| <span class="sourceLineNo">214</span> // Delete all Store information once done using<a name="line.214"></a> |
| <span class="sourceLineNo">215</span> for (int i = 0; i < compactionThreshold; i++) {<a name="line.215"></a> |
| <span class="sourceLineNo">216</span> Delete delete = new Delete(Bytes.add(STARTROW, Bytes.toBytes(i)));<a name="line.216"></a> |
| <span class="sourceLineNo">217</span> byte [][] famAndQf = {COLUMN_FAMILY, null};<a name="line.217"></a> |
| <span class="sourceLineNo">218</span> delete.addFamily(famAndQf[0]);<a name="line.218"></a> |
| <span class="sourceLineNo">219</span> r.delete(delete);<a name="line.219"></a> |
| <span class="sourceLineNo">220</span> }<a name="line.220"></a> |
| <span class="sourceLineNo">221</span> r.flush(true);<a name="line.221"></a> |
| <span class="sourceLineNo">222</span><a name="line.222"></a> |
| <span class="sourceLineNo">223</span> // Multiple versions allowed for an entry, so the delete isn't enough<a name="line.223"></a> |
| <span class="sourceLineNo">224</span> // Lower TTL and expire to ensure that all our entries have been wiped<a name="line.224"></a> |
| <span class="sourceLineNo">225</span> final int ttl = 1000;<a name="line.225"></a> |
| <span class="sourceLineNo">226</span> for (HStore store : this.r.stores.values()) {<a name="line.226"></a> |
| <span class="sourceLineNo">227</span> ScanInfo old = store.getScanInfo();<a name="line.227"></a> |
| <span class="sourceLineNo">228</span> ScanInfo si = old.customize(old.getMaxVersions(), ttl, old.getKeepDeletedCells());<a name="line.228"></a> |
| <span class="sourceLineNo">229</span> store.setScanInfo(si);<a name="line.229"></a> |
| <span class="sourceLineNo">230</span> }<a name="line.230"></a> |
| <span class="sourceLineNo">231</span> Thread.sleep(ttl);<a name="line.231"></a> |
| <span class="sourceLineNo">232</span><a name="line.232"></a> |
| <span class="sourceLineNo">233</span> r.compact(true);<a name="line.233"></a> |
| <span class="sourceLineNo">234</span> assertEquals(0, count());<a name="line.234"></a> |
| <span class="sourceLineNo">235</span> }<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> @Test<a name="line.238"></a> |
| <span class="sourceLineNo">239</span> public void testInterruptCompactionByTime() throws Exception {<a name="line.239"></a> |
| <span class="sourceLineNo">240</span> assertEquals(0, count());<a name="line.240"></a> |
| <span class="sourceLineNo">241</span><a name="line.241"></a> |
| <span class="sourceLineNo">242</span> // lower the polling interval for this test<a name="line.242"></a> |
| <span class="sourceLineNo">243</span> conf.setLong(TIME_LIMIT_KEY, 1 /* 1ms */);<a name="line.243"></a> |
| <span class="sourceLineNo">244</span><a name="line.244"></a> |
| <span class="sourceLineNo">245</span> try {<a name="line.245"></a> |
| <span class="sourceLineNo">246</span> // Create a couple store files w/ 15KB (over 10KB interval)<a name="line.246"></a> |
| <span class="sourceLineNo">247</span> int jmax = (int) Math.ceil(15.0/compactionThreshold);<a name="line.247"></a> |
| <span class="sourceLineNo">248</span> byte [] pad = new byte[1000]; // 1 KB chunk<a name="line.248"></a> |
| <span class="sourceLineNo">249</span> for (int i = 0; i < compactionThreshold; i++) {<a name="line.249"></a> |
| <span class="sourceLineNo">250</span> Table loader = new RegionAsTable(r);<a name="line.250"></a> |
| <span class="sourceLineNo">251</span> Put p = new Put(Bytes.add(STARTROW, Bytes.toBytes(i)));<a name="line.251"></a> |
| <span class="sourceLineNo">252</span> p.setDurability(Durability.SKIP_WAL);<a name="line.252"></a> |
| <span class="sourceLineNo">253</span> for (int j = 0; j < jmax; j++) {<a name="line.253"></a> |
| <span class="sourceLineNo">254</span> p.addColumn(COLUMN_FAMILY, Bytes.toBytes(j), pad);<a name="line.254"></a> |
| <span class="sourceLineNo">255</span> }<a name="line.255"></a> |
| <span class="sourceLineNo">256</span> HTestConst.addContent(loader, Bytes.toString(COLUMN_FAMILY));<a name="line.256"></a> |
| <span class="sourceLineNo">257</span> loader.put(p);<a name="line.257"></a> |
| <span class="sourceLineNo">258</span> r.flush(true);<a name="line.258"></a> |
| <span class="sourceLineNo">259</span> }<a name="line.259"></a> |
| <span class="sourceLineNo">260</span><a name="line.260"></a> |
| <span class="sourceLineNo">261</span> HRegion spyR = spy(r);<a name="line.261"></a> |
| <span class="sourceLineNo">262</span> doAnswer(new Answer<Object>() {<a name="line.262"></a> |
| <span class="sourceLineNo">263</span> @Override<a name="line.263"></a> |
| <span class="sourceLineNo">264</span> public Object answer(InvocationOnMock invocation) throws Throwable {<a name="line.264"></a> |
| <span class="sourceLineNo">265</span> r.writestate.writesEnabled = false;<a name="line.265"></a> |
| <span class="sourceLineNo">266</span> return invocation.callRealMethod();<a name="line.266"></a> |
| <span class="sourceLineNo">267</span> }<a name="line.267"></a> |
| <span class="sourceLineNo">268</span> }).when(spyR).doRegionCompactionPrep();<a name="line.268"></a> |
| <span class="sourceLineNo">269</span><a name="line.269"></a> |
| <span class="sourceLineNo">270</span> // force a minor compaction, but not before requesting a stop<a name="line.270"></a> |
| <span class="sourceLineNo">271</span> spyR.compactStores();<a name="line.271"></a> |
| <span class="sourceLineNo">272</span><a name="line.272"></a> |
| <span class="sourceLineNo">273</span> // ensure that the compaction stopped, all old files are intact,<a name="line.273"></a> |
| <span class="sourceLineNo">274</span> HStore s = r.getStore(COLUMN_FAMILY);<a name="line.274"></a> |
| <span class="sourceLineNo">275</span> assertEquals(compactionThreshold, s.getStorefilesCount());<a name="line.275"></a> |
| <span class="sourceLineNo">276</span> assertTrue(s.getStorefilesSize() > 15*1000);<a name="line.276"></a> |
| <span class="sourceLineNo">277</span> // and no new store files persisted past compactStores()<a name="line.277"></a> |
| <span class="sourceLineNo">278</span> // only one empty dir exists in temp dir<a name="line.278"></a> |
| <span class="sourceLineNo">279</span> FileStatus[] ls = r.getFilesystem().listStatus(r.getRegionFileSystem().getTempDir());<a name="line.279"></a> |
| <span class="sourceLineNo">280</span> assertEquals(1, ls.length);<a name="line.280"></a> |
| <span class="sourceLineNo">281</span> Path storeTempDir =<a name="line.281"></a> |
| <span class="sourceLineNo">282</span> new Path(r.getRegionFileSystem().getTempDir(), Bytes.toString(COLUMN_FAMILY));<a name="line.282"></a> |
| <span class="sourceLineNo">283</span> assertTrue(r.getFilesystem().exists(storeTempDir));<a name="line.283"></a> |
| <span class="sourceLineNo">284</span> ls = r.getFilesystem().listStatus(storeTempDir);<a name="line.284"></a> |
| <span class="sourceLineNo">285</span> assertEquals(0, ls.length);<a name="line.285"></a> |
| <span class="sourceLineNo">286</span> } finally {<a name="line.286"></a> |
| <span class="sourceLineNo">287</span> // don't mess up future tests<a name="line.287"></a> |
| <span class="sourceLineNo">288</span> r.writestate.writesEnabled = true;<a name="line.288"></a> |
| <span class="sourceLineNo">289</span> conf.setLong(TIME_LIMIT_KEY, 10 * 1000L /* 10 s */);<a name="line.289"></a> |
| <span class="sourceLineNo">290</span><a name="line.290"></a> |
| <span class="sourceLineNo">291</span> // Delete all Store information once done using<a name="line.291"></a> |
| <span class="sourceLineNo">292</span> for (int i = 0; i < compactionThreshold; i++) {<a name="line.292"></a> |
| <span class="sourceLineNo">293</span> Delete delete = new Delete(Bytes.add(STARTROW, Bytes.toBytes(i)));<a name="line.293"></a> |
| <span class="sourceLineNo">294</span> byte [][] famAndQf = {COLUMN_FAMILY, null};<a name="line.294"></a> |
| <span class="sourceLineNo">295</span> delete.addFamily(famAndQf[0]);<a name="line.295"></a> |
| <span class="sourceLineNo">296</span> r.delete(delete);<a name="line.296"></a> |
| <span class="sourceLineNo">297</span> }<a name="line.297"></a> |
| <span class="sourceLineNo">298</span> r.flush(true);<a name="line.298"></a> |
| <span class="sourceLineNo">299</span><a name="line.299"></a> |
| <span class="sourceLineNo">300</span> // Multiple versions allowed for an entry, so the delete isn't enough<a name="line.300"></a> |
| <span class="sourceLineNo">301</span> // Lower TTL and expire to ensure that all our entries have been wiped<a name="line.301"></a> |
| <span class="sourceLineNo">302</span> final int ttl = 1000;<a name="line.302"></a> |
| <span class="sourceLineNo">303</span> for (HStore store : this.r.stores.values()) {<a name="line.303"></a> |
| <span class="sourceLineNo">304</span> ScanInfo old = store.getScanInfo();<a name="line.304"></a> |
| <span class="sourceLineNo">305</span> ScanInfo si = old.customize(old.getMaxVersions(), ttl, old.getKeepDeletedCells());<a name="line.305"></a> |
| <span class="sourceLineNo">306</span> store.setScanInfo(si);<a name="line.306"></a> |
| <span class="sourceLineNo">307</span> }<a name="line.307"></a> |
| <span class="sourceLineNo">308</span> Thread.sleep(ttl);<a name="line.308"></a> |
| <span class="sourceLineNo">309</span><a name="line.309"></a> |
| <span class="sourceLineNo">310</span> r.compact(true);<a name="line.310"></a> |
| <span class="sourceLineNo">311</span> assertEquals(0, count());<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><a name="line.314"></a> |
| <span class="sourceLineNo">315</span> private int count() throws IOException {<a name="line.315"></a> |
| <span class="sourceLineNo">316</span> int count = 0;<a name="line.316"></a> |
| <span class="sourceLineNo">317</span> for (HStoreFile f : this.r.stores.get(COLUMN_FAMILY_TEXT).getStorefiles()) {<a name="line.317"></a> |
| <span class="sourceLineNo">318</span> HFileScanner scanner = f.getReader().getScanner(false, false);<a name="line.318"></a> |
| <span class="sourceLineNo">319</span> if (!scanner.seekTo()) {<a name="line.319"></a> |
| <span class="sourceLineNo">320</span> continue;<a name="line.320"></a> |
| <span class="sourceLineNo">321</span> }<a name="line.321"></a> |
| <span class="sourceLineNo">322</span> do {<a name="line.322"></a> |
| <span class="sourceLineNo">323</span> count++;<a name="line.323"></a> |
| <span class="sourceLineNo">324</span> } while (scanner.next());<a name="line.324"></a> |
| <span class="sourceLineNo">325</span> }<a name="line.325"></a> |
| <span class="sourceLineNo">326</span> return count;<a name="line.326"></a> |
| <span class="sourceLineNo">327</span> }<a name="line.327"></a> |
| <span class="sourceLineNo">328</span><a name="line.328"></a> |
| <span class="sourceLineNo">329</span> private void createStoreFile(final HRegion region) throws IOException {<a name="line.329"></a> |
| <span class="sourceLineNo">330</span> createStoreFile(region, Bytes.toString(COLUMN_FAMILY));<a name="line.330"></a> |
| <span class="sourceLineNo">331</span> }<a name="line.331"></a> |
| <span class="sourceLineNo">332</span><a name="line.332"></a> |
| <span class="sourceLineNo">333</span> private void createStoreFile(final HRegion region, String family) throws IOException {<a name="line.333"></a> |
| <span class="sourceLineNo">334</span> Table loader = new RegionAsTable(region);<a name="line.334"></a> |
| <span class="sourceLineNo">335</span> HTestConst.addContent(loader, family);<a name="line.335"></a> |
| <span class="sourceLineNo">336</span> region.flush(true);<a name="line.336"></a> |
| <span class="sourceLineNo">337</span> }<a name="line.337"></a> |
| <span class="sourceLineNo">338</span><a name="line.338"></a> |
| <span class="sourceLineNo">339</span> @Test<a name="line.339"></a> |
| <span class="sourceLineNo">340</span> public void testCompactionWithCorruptResult() throws Exception {<a name="line.340"></a> |
| <span class="sourceLineNo">341</span> int nfiles = 10;<a name="line.341"></a> |
| <span class="sourceLineNo">342</span> for (int i = 0; i < nfiles; i++) {<a name="line.342"></a> |
| <span class="sourceLineNo">343</span> createStoreFile(r);<a name="line.343"></a> |
| <span class="sourceLineNo">344</span> }<a name="line.344"></a> |
| <span class="sourceLineNo">345</span> HStore store = r.getStore(COLUMN_FAMILY);<a name="line.345"></a> |
| <span class="sourceLineNo">346</span><a name="line.346"></a> |
| <span class="sourceLineNo">347</span> Collection<HStoreFile> storeFiles = store.getStorefiles();<a name="line.347"></a> |
| <span class="sourceLineNo">348</span> DefaultCompactor tool = (DefaultCompactor) store.storeEngine.getCompactor();<a name="line.348"></a> |
| <span class="sourceLineNo">349</span> CompactionRequestImpl request = new CompactionRequestImpl(storeFiles);<a name="line.349"></a> |
| <span class="sourceLineNo">350</span> tool.compact(request, NoLimitThroughputController.INSTANCE, null);<a name="line.350"></a> |
| <span class="sourceLineNo">351</span><a name="line.351"></a> |
| <span class="sourceLineNo">352</span> // Now lets corrupt the compacted file.<a name="line.352"></a> |
| <span class="sourceLineNo">353</span> FileSystem fs = store.getFileSystem();<a name="line.353"></a> |
| <span class="sourceLineNo">354</span> // default compaction policy created one and only one new compacted file<a name="line.354"></a> |
| <span class="sourceLineNo">355</span> Path tmpPath = store.getRegionFileSystem().createTempName();<a name="line.355"></a> |
| <span class="sourceLineNo">356</span> try (FSDataOutputStream stream = fs.create(tmpPath, null, true, 512, (short) 3, 1024L, null)) {<a name="line.356"></a> |
| <span class="sourceLineNo">357</span> stream.writeChars("CORRUPT FILE!!!!");<a name="line.357"></a> |
| <span class="sourceLineNo">358</span> }<a name="line.358"></a> |
| <span class="sourceLineNo">359</span> // The complete compaction should fail and the corrupt file should remain<a name="line.359"></a> |
| <span class="sourceLineNo">360</span> // in the 'tmp' directory;<a name="line.360"></a> |
| <span class="sourceLineNo">361</span> assertThrows(IOException.class, () -> store.doCompaction(null, null, null,<a name="line.361"></a> |
| <span class="sourceLineNo">362</span> EnvironmentEdgeManager.currentTime(), Collections.singletonList(tmpPath)));<a name="line.362"></a> |
| <span class="sourceLineNo">363</span> assertTrue(fs.exists(tmpPath));<a name="line.363"></a> |
| <span class="sourceLineNo">364</span> }<a name="line.364"></a> |
| <span class="sourceLineNo">365</span><a name="line.365"></a> |
| <span class="sourceLineNo">366</span> /**<a name="line.366"></a> |
| <span class="sourceLineNo">367</span> * Create a custom compaction request and be sure that we can track it through the queue, knowing<a name="line.367"></a> |
| <span class="sourceLineNo">368</span> * when the compaction is completed.<a name="line.368"></a> |
| <span class="sourceLineNo">369</span> */<a name="line.369"></a> |
| <span class="sourceLineNo">370</span> @Test<a name="line.370"></a> |
| <span class="sourceLineNo">371</span> public void testTrackingCompactionRequest() throws Exception {<a name="line.371"></a> |
| <span class="sourceLineNo">372</span> // setup a compact/split thread on a mock server<a name="line.372"></a> |
| <span class="sourceLineNo">373</span> HRegionServer mockServer = Mockito.mock(HRegionServer.class);<a name="line.373"></a> |
| <span class="sourceLineNo">374</span> Mockito.when(mockServer.getConfiguration()).thenReturn(r.getBaseConf());<a name="line.374"></a> |
| <span class="sourceLineNo">375</span> CompactSplit thread = new CompactSplit(mockServer);<a name="line.375"></a> |
| <span class="sourceLineNo">376</span> Mockito.when(mockServer.getCompactSplitThread()).thenReturn(thread);<a name="line.376"></a> |
| <span class="sourceLineNo">377</span><a name="line.377"></a> |
| <span class="sourceLineNo">378</span> // setup a region/store with some files<a name="line.378"></a> |
| <span class="sourceLineNo">379</span> HStore store = r.getStore(COLUMN_FAMILY);<a name="line.379"></a> |
| <span class="sourceLineNo">380</span> createStoreFile(r);<a name="line.380"></a> |
| <span class="sourceLineNo">381</span> for (int i = 0; i < MAX_FILES_TO_COMPACT + 1; i++) {<a name="line.381"></a> |
| <span class="sourceLineNo">382</span> createStoreFile(r);<a name="line.382"></a> |
| <span class="sourceLineNo">383</span> }<a name="line.383"></a> |
| <span class="sourceLineNo">384</span><a name="line.384"></a> |
| <span class="sourceLineNo">385</span> CountDownLatch latch = new CountDownLatch(1);<a name="line.385"></a> |
| <span class="sourceLineNo">386</span> Tracker tracker = new Tracker(latch);<a name="line.386"></a> |
| <span class="sourceLineNo">387</span> thread.requestCompaction(r, store, "test custom comapction", PRIORITY_USER, tracker,<a name="line.387"></a> |
| <span class="sourceLineNo">388</span> null);<a name="line.388"></a> |
| <span class="sourceLineNo">389</span> // wait for the latch to complete.<a name="line.389"></a> |
| <span class="sourceLineNo">390</span> latch.await();<a name="line.390"></a> |
| <span class="sourceLineNo">391</span><a name="line.391"></a> |
| <span class="sourceLineNo">392</span> thread.interruptIfNecessary();<a name="line.392"></a> |
| <span class="sourceLineNo">393</span> }<a name="line.393"></a> |
| <span class="sourceLineNo">394</span><a name="line.394"></a> |
| <span class="sourceLineNo">395</span> @Test<a name="line.395"></a> |
| <span class="sourceLineNo">396</span> public void testCompactionFailure() throws Exception {<a name="line.396"></a> |
| <span class="sourceLineNo">397</span> // setup a compact/split thread on a mock server<a name="line.397"></a> |
| <span class="sourceLineNo">398</span> HRegionServer mockServer = Mockito.mock(HRegionServer.class);<a name="line.398"></a> |
| <span class="sourceLineNo">399</span> Mockito.when(mockServer.getConfiguration()).thenReturn(r.getBaseConf());<a name="line.399"></a> |
| <span class="sourceLineNo">400</span> CompactSplit thread = new CompactSplit(mockServer);<a name="line.400"></a> |
| <span class="sourceLineNo">401</span> Mockito.when(mockServer.getCompactSplitThread()).thenReturn(thread);<a name="line.401"></a> |
| <span class="sourceLineNo">402</span><a name="line.402"></a> |
| <span class="sourceLineNo">403</span> // setup a region/store with some files<a name="line.403"></a> |
| <span class="sourceLineNo">404</span> HStore store = r.getStore(COLUMN_FAMILY);<a name="line.404"></a> |
| <span class="sourceLineNo">405</span> createStoreFile(r);<a name="line.405"></a> |
| <span class="sourceLineNo">406</span> for (int i = 0; i < HStore.DEFAULT_BLOCKING_STOREFILE_COUNT - 1; i++) {<a name="line.406"></a> |
| <span class="sourceLineNo">407</span> createStoreFile(r);<a name="line.407"></a> |
| <span class="sourceLineNo">408</span> }<a name="line.408"></a> |
| <span class="sourceLineNo">409</span><a name="line.409"></a> |
| <span class="sourceLineNo">410</span> HRegion mockRegion = Mockito.spy(r);<a name="line.410"></a> |
| <span class="sourceLineNo">411</span> Mockito.when(mockRegion.checkSplit()).<a name="line.411"></a> |
| <span class="sourceLineNo">412</span> thenThrow(new RuntimeException("Thrown intentionally by test!"));<a name="line.412"></a> |
| <span class="sourceLineNo">413</span><a name="line.413"></a> |
| <span class="sourceLineNo">414</span> try (MetricsRegionWrapperImpl metricsWrapper = new MetricsRegionWrapperImpl(r)) {<a name="line.414"></a> |
| <span class="sourceLineNo">415</span><a name="line.415"></a> |
| <span class="sourceLineNo">416</span> long preCompletedCount = metricsWrapper.getNumCompactionsCompleted();<a name="line.416"></a> |
| <span class="sourceLineNo">417</span> long preFailedCount = metricsWrapper.getNumCompactionsFailed();<a name="line.417"></a> |
| <span class="sourceLineNo">418</span><a name="line.418"></a> |
| <span class="sourceLineNo">419</span> CountDownLatch latch = new CountDownLatch(1);<a name="line.419"></a> |
| <span class="sourceLineNo">420</span> Tracker tracker = new Tracker(latch);<a name="line.420"></a> |
| <span class="sourceLineNo">421</span> thread.requestCompaction(mockRegion, store, "test custom comapction", PRIORITY_USER, tracker,<a name="line.421"></a> |
| <span class="sourceLineNo">422</span> null);<a name="line.422"></a> |
| <span class="sourceLineNo">423</span> // wait for the latch to complete.<a name="line.423"></a> |
| <span class="sourceLineNo">424</span> latch.await(120, TimeUnit.SECONDS);<a name="line.424"></a> |
| <span class="sourceLineNo">425</span><a name="line.425"></a> |
| <span class="sourceLineNo">426</span> // compaction should have completed and been marked as failed due to error in split request<a name="line.426"></a> |
| <span class="sourceLineNo">427</span> long postCompletedCount = metricsWrapper.getNumCompactionsCompleted();<a name="line.427"></a> |
| <span class="sourceLineNo">428</span> long postFailedCount = metricsWrapper.getNumCompactionsFailed();<a name="line.428"></a> |
| <span class="sourceLineNo">429</span><a name="line.429"></a> |
| <span class="sourceLineNo">430</span> assertTrue("Completed count should have increased (pre=" + preCompletedCount + ", post=" +<a name="line.430"></a> |
| <span class="sourceLineNo">431</span> postCompletedCount + ")", postCompletedCount > preCompletedCount);<a name="line.431"></a> |
| <span class="sourceLineNo">432</span> assertTrue("Failed count should have increased (pre=" + preFailedCount + ", post=" +<a name="line.432"></a> |
| <span class="sourceLineNo">433</span> postFailedCount + ")", postFailedCount > preFailedCount);<a name="line.433"></a> |
| <span class="sourceLineNo">434</span> }<a name="line.434"></a> |
| <span class="sourceLineNo">435</span> }<a name="line.435"></a> |
| <span class="sourceLineNo">436</span><a name="line.436"></a> |
| <span class="sourceLineNo">437</span> /**<a name="line.437"></a> |
| <span class="sourceLineNo">438</span> * Test no new Compaction requests are generated after calling stop compactions<a name="line.438"></a> |
| <span class="sourceLineNo">439</span> */<a name="line.439"></a> |
| <span class="sourceLineNo">440</span> @Test<a name="line.440"></a> |
| <span class="sourceLineNo">441</span> public void testStopStartCompaction() throws IOException {<a name="line.441"></a> |
| <span class="sourceLineNo">442</span> // setup a compact/split thread on a mock server<a name="line.442"></a> |
| <span class="sourceLineNo">443</span> HRegionServer mockServer = Mockito.mock(HRegionServer.class);<a name="line.443"></a> |
| <span class="sourceLineNo">444</span> Mockito.when(mockServer.getConfiguration()).thenReturn(r.getBaseConf());<a name="line.444"></a> |
| <span class="sourceLineNo">445</span> final CompactSplit thread = new CompactSplit(mockServer);<a name="line.445"></a> |
| <span class="sourceLineNo">446</span> Mockito.when(mockServer.getCompactSplitThread()).thenReturn(thread);<a name="line.446"></a> |
| <span class="sourceLineNo">447</span> // setup a region/store with some files<a name="line.447"></a> |
| <span class="sourceLineNo">448</span> HStore store = r.getStore(COLUMN_FAMILY);<a name="line.448"></a> |
| <span class="sourceLineNo">449</span> createStoreFile(r);<a name="line.449"></a> |
| <span class="sourceLineNo">450</span> for (int i = 0; i < HStore.DEFAULT_BLOCKING_STOREFILE_COUNT - 1; i++) {<a name="line.450"></a> |
| <span class="sourceLineNo">451</span> createStoreFile(r);<a name="line.451"></a> |
| <span class="sourceLineNo">452</span> }<a name="line.452"></a> |
| <span class="sourceLineNo">453</span> thread.switchCompaction(false);<a name="line.453"></a> |
| <span class="sourceLineNo">454</span> thread.requestCompaction(r, store, "test", Store.PRIORITY_USER,<a name="line.454"></a> |
| <span class="sourceLineNo">455</span> CompactionLifeCycleTracker.DUMMY, null);<a name="line.455"></a> |
| <span class="sourceLineNo">456</span> assertFalse(thread.isCompactionsEnabled());<a name="line.456"></a> |
| <span class="sourceLineNo">457</span> int longCompactions = thread.getLongCompactions().getActiveCount();<a name="line.457"></a> |
| <span class="sourceLineNo">458</span> int shortCompactions = thread.getShortCompactions().getActiveCount();<a name="line.458"></a> |
| <span class="sourceLineNo">459</span> assertEquals("longCompactions=" + longCompactions + "," +<a name="line.459"></a> |
| <span class="sourceLineNo">460</span> "shortCompactions=" + shortCompactions, 0, longCompactions + shortCompactions);<a name="line.460"></a> |
| <span class="sourceLineNo">461</span> thread.switchCompaction(true);<a name="line.461"></a> |
| <span class="sourceLineNo">462</span> assertTrue(thread.isCompactionsEnabled());<a name="line.462"></a> |
| <span class="sourceLineNo">463</span> // Make sure no compactions have run.<a name="line.463"></a> |
| <span class="sourceLineNo">464</span> assertEquals(0, thread.getLongCompactions().getCompletedTaskCount() +<a name="line.464"></a> |
| <span class="sourceLineNo">465</span> thread.getShortCompactions().getCompletedTaskCount());<a name="line.465"></a> |
| <span class="sourceLineNo">466</span> // Request a compaction and make sure it is submitted successfully.<a name="line.466"></a> |
| <span class="sourceLineNo">467</span> thread.requestCompaction(r, store, "test", Store.PRIORITY_USER,<a name="line.467"></a> |
| <span class="sourceLineNo">468</span> CompactionLifeCycleTracker.DUMMY, null);<a name="line.468"></a> |
| <span class="sourceLineNo">469</span> // Wait until the compaction finishes.<a name="line.469"></a> |
| <span class="sourceLineNo">470</span> Waiter.waitFor(UTIL.getConfiguration(), 5000,<a name="line.470"></a> |
| <span class="sourceLineNo">471</span> (Waiter.Predicate<Exception>) () -> thread.getLongCompactions().getCompletedTaskCount() +<a name="line.471"></a> |
| <span class="sourceLineNo">472</span> thread.getShortCompactions().getCompletedTaskCount() == 1);<a name="line.472"></a> |
| <span class="sourceLineNo">473</span> // Make sure there are no compactions running.<a name="line.473"></a> |
| <span class="sourceLineNo">474</span> assertEquals(0, thread.getLongCompactions().getActiveCount()<a name="line.474"></a> |
| <span class="sourceLineNo">475</span> + thread.getShortCompactions().getActiveCount());<a name="line.475"></a> |
| <span class="sourceLineNo">476</span> }<a name="line.476"></a> |
| <span class="sourceLineNo">477</span><a name="line.477"></a> |
| <span class="sourceLineNo">478</span> @Test public void testInterruptingRunningCompactions() throws Exception {<a name="line.478"></a> |
| <span class="sourceLineNo">479</span> // setup a compact/split thread on a mock server<a name="line.479"></a> |
| <span class="sourceLineNo">480</span> conf.set(CompactionThroughputControllerFactory.HBASE_THROUGHPUT_CONTROLLER_KEY,<a name="line.480"></a> |
| <span class="sourceLineNo">481</span> WaitThroughPutController.class.getName());<a name="line.481"></a> |
| <span class="sourceLineNo">482</span> HRegionServer mockServer = Mockito.mock(HRegionServer.class);<a name="line.482"></a> |
| <span class="sourceLineNo">483</span> Mockito.when(mockServer.getConfiguration()).thenReturn(r.getBaseConf());<a name="line.483"></a> |
| <span class="sourceLineNo">484</span> CompactSplit thread = new CompactSplit(mockServer);<a name="line.484"></a> |
| <span class="sourceLineNo">485</span><a name="line.485"></a> |
| <span class="sourceLineNo">486</span> Mockito.when(mockServer.getCompactSplitThread()).thenReturn(thread);<a name="line.486"></a> |
| <span class="sourceLineNo">487</span><a name="line.487"></a> |
| <span class="sourceLineNo">488</span> // setup a region/store with some files<a name="line.488"></a> |
| <span class="sourceLineNo">489</span> HStore store = r.getStore(COLUMN_FAMILY);<a name="line.489"></a> |
| <span class="sourceLineNo">490</span> int jmax = (int) Math.ceil(15.0 / compactionThreshold);<a name="line.490"></a> |
| <span class="sourceLineNo">491</span> byte[] pad = new byte[1000]; // 1 KB chunk<a name="line.491"></a> |
| <span class="sourceLineNo">492</span> for (int i = 0; i < compactionThreshold; i++) {<a name="line.492"></a> |
| <span class="sourceLineNo">493</span> Table loader = new RegionAsTable(r);<a name="line.493"></a> |
| <span class="sourceLineNo">494</span> Put p = new Put(Bytes.add(STARTROW, Bytes.toBytes(i)));<a name="line.494"></a> |
| <span class="sourceLineNo">495</span> p.setDurability(Durability.SKIP_WAL);<a name="line.495"></a> |
| <span class="sourceLineNo">496</span> for (int j = 0; j < jmax; j++) {<a name="line.496"></a> |
| <span class="sourceLineNo">497</span> p.addColumn(COLUMN_FAMILY, Bytes.toBytes(j), pad);<a name="line.497"></a> |
| <span class="sourceLineNo">498</span> }<a name="line.498"></a> |
| <span class="sourceLineNo">499</span> HTestConst.addContent(loader, Bytes.toString(COLUMN_FAMILY));<a name="line.499"></a> |
| <span class="sourceLineNo">500</span> loader.put(p);<a name="line.500"></a> |
| <span class="sourceLineNo">501</span> r.flush(true);<a name="line.501"></a> |
| <span class="sourceLineNo">502</span> }<a name="line.502"></a> |
| <span class="sourceLineNo">503</span> HStore s = r.getStore(COLUMN_FAMILY);<a name="line.503"></a> |
| <span class="sourceLineNo">504</span> int initialFiles = s.getStorefilesCount();<a name="line.504"></a> |
| <span class="sourceLineNo">505</span><a name="line.505"></a> |
| <span class="sourceLineNo">506</span> thread.requestCompaction(r, store, "test custom comapction", PRIORITY_USER,<a name="line.506"></a> |
| <span class="sourceLineNo">507</span> CompactionLifeCycleTracker.DUMMY, null);<a name="line.507"></a> |
| <span class="sourceLineNo">508</span><a name="line.508"></a> |
| <span class="sourceLineNo">509</span> Thread.sleep(3000);<a name="line.509"></a> |
| <span class="sourceLineNo">510</span> thread.switchCompaction(false);<a name="line.510"></a> |
| <span class="sourceLineNo">511</span> assertEquals(initialFiles, s.getStorefilesCount());<a name="line.511"></a> |
| <span class="sourceLineNo">512</span> //don't mess up future tests<a name="line.512"></a> |
| <span class="sourceLineNo">513</span> thread.switchCompaction(true);<a name="line.513"></a> |
| <span class="sourceLineNo">514</span> }<a name="line.514"></a> |
| <span class="sourceLineNo">515</span><a name="line.515"></a> |
| <span class="sourceLineNo">516</span> /**<a name="line.516"></a> |
| <span class="sourceLineNo">517</span> * HBASE-7947: Regression test to ensure adding to the correct list in the<a name="line.517"></a> |
| <span class="sourceLineNo">518</span> * {@link CompactSplit}<a name="line.518"></a> |
| <span class="sourceLineNo">519</span> * @throws Exception on failure<a name="line.519"></a> |
| <span class="sourceLineNo">520</span> */<a name="line.520"></a> |
| <span class="sourceLineNo">521</span> @Test<a name="line.521"></a> |
| <span class="sourceLineNo">522</span> public void testMultipleCustomCompactionRequests() throws Exception {<a name="line.522"></a> |
| <span class="sourceLineNo">523</span> // setup a compact/split thread on a mock server<a name="line.523"></a> |
| <span class="sourceLineNo">524</span> HRegionServer mockServer = Mockito.mock(HRegionServer.class);<a name="line.524"></a> |
| <span class="sourceLineNo">525</span> Mockito.when(mockServer.getConfiguration()).thenReturn(r.getBaseConf());<a name="line.525"></a> |
| <span class="sourceLineNo">526</span> CompactSplit thread = new CompactSplit(mockServer);<a name="line.526"></a> |
| <span class="sourceLineNo">527</span> Mockito.when(mockServer.getCompactSplitThread()).thenReturn(thread);<a name="line.527"></a> |
| <span class="sourceLineNo">528</span><a name="line.528"></a> |
| <span class="sourceLineNo">529</span> // setup a region/store with some files<a name="line.529"></a> |
| <span class="sourceLineNo">530</span> int numStores = r.getStores().size();<a name="line.530"></a> |
| <span class="sourceLineNo">531</span> CountDownLatch latch = new CountDownLatch(numStores);<a name="line.531"></a> |
| <span class="sourceLineNo">532</span> Tracker tracker = new Tracker(latch);<a name="line.532"></a> |
| <span class="sourceLineNo">533</span> // create some store files and setup requests for each store on which we want to do a<a name="line.533"></a> |
| <span class="sourceLineNo">534</span> // compaction<a name="line.534"></a> |
| <span class="sourceLineNo">535</span> for (HStore store : r.getStores()) {<a name="line.535"></a> |
| <span class="sourceLineNo">536</span> createStoreFile(r, store.getColumnFamilyName());<a name="line.536"></a> |
| <span class="sourceLineNo">537</span> createStoreFile(r, store.getColumnFamilyName());<a name="line.537"></a> |
| <span class="sourceLineNo">538</span> createStoreFile(r, store.getColumnFamilyName());<a name="line.538"></a> |
| <span class="sourceLineNo">539</span> thread.requestCompaction(r, store, "test mulitple custom comapctions", PRIORITY_USER,<a name="line.539"></a> |
| <span class="sourceLineNo">540</span> tracker, null);<a name="line.540"></a> |
| <span class="sourceLineNo">541</span> }<a name="line.541"></a> |
| <span class="sourceLineNo">542</span> // wait for the latch to complete.<a name="line.542"></a> |
| <span class="sourceLineNo">543</span> latch.await();<a name="line.543"></a> |
| <span class="sourceLineNo">544</span><a name="line.544"></a> |
| <span class="sourceLineNo">545</span> thread.interruptIfNecessary();<a name="line.545"></a> |
| <span class="sourceLineNo">546</span> }<a name="line.546"></a> |
| <span class="sourceLineNo">547</span><a name="line.547"></a> |
| <span class="sourceLineNo">548</span> class StoreMockMaker extends StatefulStoreMockMaker {<a name="line.548"></a> |
| <span class="sourceLineNo">549</span> public ArrayList<HStoreFile> compacting = new ArrayList<>();<a name="line.549"></a> |
| <span class="sourceLineNo">550</span> public ArrayList<HStoreFile> notCompacting = new ArrayList<>();<a name="line.550"></a> |
| <span class="sourceLineNo">551</span> private final ArrayList<Integer> results;<a name="line.551"></a> |
| <span class="sourceLineNo">552</span><a name="line.552"></a> |
| <span class="sourceLineNo">553</span> public StoreMockMaker(ArrayList<Integer> results) {<a name="line.553"></a> |
| <span class="sourceLineNo">554</span> this.results = results;<a name="line.554"></a> |
| <span class="sourceLineNo">555</span> }<a name="line.555"></a> |
| <span class="sourceLineNo">556</span><a name="line.556"></a> |
| <span class="sourceLineNo">557</span> public class TestCompactionContext extends CompactionContext {<a name="line.557"></a> |
| <span class="sourceLineNo">558</span><a name="line.558"></a> |
| <span class="sourceLineNo">559</span> private List<HStoreFile> selectedFiles;<a name="line.559"></a> |
| <span class="sourceLineNo">560</span><a name="line.560"></a> |
| <span class="sourceLineNo">561</span> public TestCompactionContext(List<HStoreFile> selectedFiles) {<a name="line.561"></a> |
| <span class="sourceLineNo">562</span> super();<a name="line.562"></a> |
| <span class="sourceLineNo">563</span> this.selectedFiles = selectedFiles;<a name="line.563"></a> |
| <span class="sourceLineNo">564</span> }<a name="line.564"></a> |
| <span class="sourceLineNo">565</span><a name="line.565"></a> |
| <span class="sourceLineNo">566</span> @Override<a name="line.566"></a> |
| <span class="sourceLineNo">567</span> public List<HStoreFile> preSelect(List<HStoreFile> filesCompacting) {<a name="line.567"></a> |
| <span class="sourceLineNo">568</span> return new ArrayList<>();<a name="line.568"></a> |
| <span class="sourceLineNo">569</span> }<a name="line.569"></a> |
| <span class="sourceLineNo">570</span><a name="line.570"></a> |
| <span class="sourceLineNo">571</span> @Override<a name="line.571"></a> |
| <span class="sourceLineNo">572</span> public boolean select(List<HStoreFile> filesCompacting, boolean isUserCompaction,<a name="line.572"></a> |
| <span class="sourceLineNo">573</span> boolean mayUseOffPeak, boolean forceMajor) throws IOException {<a name="line.573"></a> |
| <span class="sourceLineNo">574</span> this.request = new CompactionRequestImpl(selectedFiles);<a name="line.574"></a> |
| <span class="sourceLineNo">575</span> this.request.setPriority(getPriority());<a name="line.575"></a> |
| <span class="sourceLineNo">576</span> return true;<a name="line.576"></a> |
| <span class="sourceLineNo">577</span> }<a name="line.577"></a> |
| <span class="sourceLineNo">578</span><a name="line.578"></a> |
| <span class="sourceLineNo">579</span> @Override<a name="line.579"></a> |
| <span class="sourceLineNo">580</span> public List<Path> compact(ThroughputController throughputController, User user)<a name="line.580"></a> |
| <span class="sourceLineNo">581</span> throws IOException {<a name="line.581"></a> |
| <span class="sourceLineNo">582</span> finishCompaction(this.selectedFiles);<a name="line.582"></a> |
| <span class="sourceLineNo">583</span> return new ArrayList<>();<a name="line.583"></a> |
| <span class="sourceLineNo">584</span> }<a name="line.584"></a> |
| <span class="sourceLineNo">585</span> }<a name="line.585"></a> |
| <span class="sourceLineNo">586</span><a name="line.586"></a> |
| <span class="sourceLineNo">587</span> @Override<a name="line.587"></a> |
| <span class="sourceLineNo">588</span> public synchronized Optional<CompactionContext> selectCompaction() {<a name="line.588"></a> |
| <span class="sourceLineNo">589</span> CompactionContext ctx = new TestCompactionContext(new ArrayList<>(notCompacting));<a name="line.589"></a> |
| <span class="sourceLineNo">590</span> compacting.addAll(notCompacting);<a name="line.590"></a> |
| <span class="sourceLineNo">591</span> notCompacting.clear();<a name="line.591"></a> |
| <span class="sourceLineNo">592</span> try {<a name="line.592"></a> |
| <span class="sourceLineNo">593</span> ctx.select(null, false, false, false);<a name="line.593"></a> |
| <span class="sourceLineNo">594</span> } catch (IOException ex) {<a name="line.594"></a> |
| <span class="sourceLineNo">595</span> fail("Shouldn't happen");<a name="line.595"></a> |
| <span class="sourceLineNo">596</span> }<a name="line.596"></a> |
| <span class="sourceLineNo">597</span> return Optional.of(ctx);<a name="line.597"></a> |
| <span class="sourceLineNo">598</span> }<a name="line.598"></a> |
| <span class="sourceLineNo">599</span><a name="line.599"></a> |
| <span class="sourceLineNo">600</span> @Override<a name="line.600"></a> |
| <span class="sourceLineNo">601</span> public synchronized void cancelCompaction(Object object) {<a name="line.601"></a> |
| <span class="sourceLineNo">602</span> TestCompactionContext ctx = (TestCompactionContext)object;<a name="line.602"></a> |
| <span class="sourceLineNo">603</span> compacting.removeAll(ctx.selectedFiles);<a name="line.603"></a> |
| <span class="sourceLineNo">604</span> notCompacting.addAll(ctx.selectedFiles);<a name="line.604"></a> |
| <span class="sourceLineNo">605</span> }<a name="line.605"></a> |
| <span class="sourceLineNo">606</span><a name="line.606"></a> |
| <span class="sourceLineNo">607</span> public synchronized void finishCompaction(List<HStoreFile> sfs) {<a name="line.607"></a> |
| <span class="sourceLineNo">608</span> if (sfs.isEmpty()) return;<a name="line.608"></a> |
| <span class="sourceLineNo">609</span> synchronized (results) {<a name="line.609"></a> |
| <span class="sourceLineNo">610</span> results.add(sfs.size());<a name="line.610"></a> |
| <span class="sourceLineNo">611</span> }<a name="line.611"></a> |
| <span class="sourceLineNo">612</span> compacting.removeAll(sfs);<a name="line.612"></a> |
| <span class="sourceLineNo">613</span> }<a name="line.613"></a> |
| <span class="sourceLineNo">614</span><a name="line.614"></a> |
| <span class="sourceLineNo">615</span> @Override<a name="line.615"></a> |
| <span class="sourceLineNo">616</span> public int getPriority() {<a name="line.616"></a> |
| <span class="sourceLineNo">617</span> return 7 - compacting.size() - notCompacting.size();<a name="line.617"></a> |
| <span class="sourceLineNo">618</span> }<a name="line.618"></a> |
| <span class="sourceLineNo">619</span> }<a name="line.619"></a> |
| <span class="sourceLineNo">620</span><a name="line.620"></a> |
| <span class="sourceLineNo">621</span> public class BlockingStoreMockMaker extends StatefulStoreMockMaker {<a name="line.621"></a> |
| <span class="sourceLineNo">622</span> BlockingCompactionContext blocked = null;<a name="line.622"></a> |
| <span class="sourceLineNo">623</span><a name="line.623"></a> |
| <span class="sourceLineNo">624</span> public class BlockingCompactionContext extends CompactionContext {<a name="line.624"></a> |
| <span class="sourceLineNo">625</span> public volatile boolean isInCompact = false;<a name="line.625"></a> |
| <span class="sourceLineNo">626</span><a name="line.626"></a> |
| <span class="sourceLineNo">627</span> public void unblock() {<a name="line.627"></a> |
| <span class="sourceLineNo">628</span> synchronized (this) {<a name="line.628"></a> |
| <span class="sourceLineNo">629</span> this.notifyAll();<a name="line.629"></a> |
| <span class="sourceLineNo">630</span> }<a name="line.630"></a> |
| <span class="sourceLineNo">631</span> }<a name="line.631"></a> |
| <span class="sourceLineNo">632</span><a name="line.632"></a> |
| <span class="sourceLineNo">633</span> @Override<a name="line.633"></a> |
| <span class="sourceLineNo">634</span> public List<Path> compact(ThroughputController throughputController, User user)<a name="line.634"></a> |
| <span class="sourceLineNo">635</span> throws IOException {<a name="line.635"></a> |
| <span class="sourceLineNo">636</span> try {<a name="line.636"></a> |
| <span class="sourceLineNo">637</span> isInCompact = true;<a name="line.637"></a> |
| <span class="sourceLineNo">638</span> synchronized (this) {<a name="line.638"></a> |
| <span class="sourceLineNo">639</span> this.wait();<a name="line.639"></a> |
| <span class="sourceLineNo">640</span> }<a name="line.640"></a> |
| <span class="sourceLineNo">641</span> } catch (InterruptedException e) {<a name="line.641"></a> |
| <span class="sourceLineNo">642</span> Assume.assumeNoException(e);<a name="line.642"></a> |
| <span class="sourceLineNo">643</span> }<a name="line.643"></a> |
| <span class="sourceLineNo">644</span> return new ArrayList<>();<a name="line.644"></a> |
| <span class="sourceLineNo">645</span> }<a name="line.645"></a> |
| <span class="sourceLineNo">646</span><a name="line.646"></a> |
| <span class="sourceLineNo">647</span> @Override<a name="line.647"></a> |
| <span class="sourceLineNo">648</span> public List<HStoreFile> preSelect(List<HStoreFile> filesCompacting) {<a name="line.648"></a> |
| <span class="sourceLineNo">649</span> return new ArrayList<>();<a name="line.649"></a> |
| <span class="sourceLineNo">650</span> }<a name="line.650"></a> |
| <span class="sourceLineNo">651</span><a name="line.651"></a> |
| <span class="sourceLineNo">652</span> @Override<a name="line.652"></a> |
| <span class="sourceLineNo">653</span> public boolean select(List<HStoreFile> f, boolean i, boolean m, boolean e)<a name="line.653"></a> |
| <span class="sourceLineNo">654</span> throws IOException {<a name="line.654"></a> |
| <span class="sourceLineNo">655</span> this.request = new CompactionRequestImpl(new ArrayList<>());<a name="line.655"></a> |
| <span class="sourceLineNo">656</span> return true;<a name="line.656"></a> |
| <span class="sourceLineNo">657</span> }<a name="line.657"></a> |
| <span class="sourceLineNo">658</span> }<a name="line.658"></a> |
| <span class="sourceLineNo">659</span><a name="line.659"></a> |
| <span class="sourceLineNo">660</span> @Override<a name="line.660"></a> |
| <span class="sourceLineNo">661</span> public Optional<CompactionContext> selectCompaction() {<a name="line.661"></a> |
| <span class="sourceLineNo">662</span> this.blocked = new BlockingCompactionContext();<a name="line.662"></a> |
| <span class="sourceLineNo">663</span> try {<a name="line.663"></a> |
| <span class="sourceLineNo">664</span> this.blocked.select(null, false, false, false);<a name="line.664"></a> |
| <span class="sourceLineNo">665</span> } catch (IOException ex) {<a name="line.665"></a> |
| <span class="sourceLineNo">666</span> fail("Shouldn't happen");<a name="line.666"></a> |
| <span class="sourceLineNo">667</span> }<a name="line.667"></a> |
| <span class="sourceLineNo">668</span> return Optional.of(blocked);<a name="line.668"></a> |
| <span class="sourceLineNo">669</span> }<a name="line.669"></a> |
| <span class="sourceLineNo">670</span><a name="line.670"></a> |
| <span class="sourceLineNo">671</span> @Override<a name="line.671"></a> |
| <span class="sourceLineNo">672</span> public void cancelCompaction(Object object) {}<a name="line.672"></a> |
| <span class="sourceLineNo">673</span><a name="line.673"></a> |
| <span class="sourceLineNo">674</span> @Override<a name="line.674"></a> |
| <span class="sourceLineNo">675</span> public int getPriority() {<a name="line.675"></a> |
| <span class="sourceLineNo">676</span> return Integer.MIN_VALUE; // some invalid value, see createStoreMock<a name="line.676"></a> |
| <span class="sourceLineNo">677</span> }<a name="line.677"></a> |
| <span class="sourceLineNo">678</span><a name="line.678"></a> |
| <span class="sourceLineNo">679</span> public BlockingCompactionContext waitForBlocking() {<a name="line.679"></a> |
| <span class="sourceLineNo">680</span> while (this.blocked == null || !this.blocked.isInCompact) {<a name="line.680"></a> |
| <span class="sourceLineNo">681</span> Threads.sleepWithoutInterrupt(50);<a name="line.681"></a> |
| <span class="sourceLineNo">682</span> }<a name="line.682"></a> |
| <span class="sourceLineNo">683</span> BlockingCompactionContext ctx = this.blocked;<a name="line.683"></a> |
| <span class="sourceLineNo">684</span> this.blocked = null;<a name="line.684"></a> |
| <span class="sourceLineNo">685</span> return ctx;<a name="line.685"></a> |
| <span class="sourceLineNo">686</span> }<a name="line.686"></a> |
| <span class="sourceLineNo">687</span><a name="line.687"></a> |
| <span class="sourceLineNo">688</span> @Override<a name="line.688"></a> |
| <span class="sourceLineNo">689</span> public HStore createStoreMock(String name) throws Exception {<a name="line.689"></a> |
| <span class="sourceLineNo">690</span> return createStoreMock(Integer.MIN_VALUE, name);<a name="line.690"></a> |
| <span class="sourceLineNo">691</span> }<a name="line.691"></a> |
| <span class="sourceLineNo">692</span><a name="line.692"></a> |
| <span class="sourceLineNo">693</span> public HStore createStoreMock(int priority, String name) throws Exception {<a name="line.693"></a> |
| <span class="sourceLineNo">694</span> // Override the mock to always return the specified priority.<a name="line.694"></a> |
| <span class="sourceLineNo">695</span> HStore s = super.createStoreMock(name);<a name="line.695"></a> |
| <span class="sourceLineNo">696</span> when(s.getCompactPriority()).thenReturn(priority);<a name="line.696"></a> |
| <span class="sourceLineNo">697</span> return s;<a name="line.697"></a> |
| <span class="sourceLineNo">698</span> }<a name="line.698"></a> |
| <span class="sourceLineNo">699</span> }<a name="line.699"></a> |
| <span class="sourceLineNo">700</span><a name="line.700"></a> |
| <span class="sourceLineNo">701</span> /** Test compaction priority management and multiple compactions per store (HBASE-8665). */<a name="line.701"></a> |
| <span class="sourceLineNo">702</span> @Test<a name="line.702"></a> |
| <span class="sourceLineNo">703</span> public void testCompactionQueuePriorities() throws Exception {<a name="line.703"></a> |
| <span class="sourceLineNo">704</span> // Setup a compact/split thread on a mock server.<a name="line.704"></a> |
| <span class="sourceLineNo">705</span> final Configuration conf = HBaseConfiguration.create();<a name="line.705"></a> |
| <span class="sourceLineNo">706</span> HRegionServer mockServer = mock(HRegionServer.class);<a name="line.706"></a> |
| <span class="sourceLineNo">707</span> when(mockServer.isStopped()).thenReturn(false);<a name="line.707"></a> |
| <span class="sourceLineNo">708</span> when(mockServer.getConfiguration()).thenReturn(conf);<a name="line.708"></a> |
| <span class="sourceLineNo">709</span> when(mockServer.getChoreService()).thenReturn(new ChoreService("test"));<a name="line.709"></a> |
| <span class="sourceLineNo">710</span> CompactSplit cst = new CompactSplit(mockServer);<a name="line.710"></a> |
| <span class="sourceLineNo">711</span> when(mockServer.getCompactSplitThread()).thenReturn(cst);<a name="line.711"></a> |
| <span class="sourceLineNo">712</span> //prevent large compaction thread pool stealing job from small compaction queue.<a name="line.712"></a> |
| <span class="sourceLineNo">713</span> cst.shutdownLongCompactions();<a name="line.713"></a> |
| <span class="sourceLineNo">714</span> // Set up the region mock that redirects compactions.<a name="line.714"></a> |
| <span class="sourceLineNo">715</span> HRegion r = mock(HRegion.class);<a name="line.715"></a> |
| <span class="sourceLineNo">716</span> when(<a name="line.716"></a> |
| <span class="sourceLineNo">717</span> r.compact(any(), any(), any(), any())).then(new Answer<Boolean>() {<a name="line.717"></a> |
| <span class="sourceLineNo">718</span> @Override<a name="line.718"></a> |
| <span class="sourceLineNo">719</span> public Boolean answer(InvocationOnMock invocation) throws Throwable {<a name="line.719"></a> |
| <span class="sourceLineNo">720</span> invocation.<CompactionContext>getArgument(0).compact(invocation.getArgument(2), null);<a name="line.720"></a> |
| <span class="sourceLineNo">721</span> return true;<a name="line.721"></a> |
| <span class="sourceLineNo">722</span> }<a name="line.722"></a> |
| <span class="sourceLineNo">723</span> });<a name="line.723"></a> |
| <span class="sourceLineNo">724</span><a name="line.724"></a> |
| <span class="sourceLineNo">725</span> // Set up store mocks for 2 "real" stores and the one we use for blocking CST.<a name="line.725"></a> |
| <span class="sourceLineNo">726</span> ArrayList<Integer> results = new ArrayList<>();<a name="line.726"></a> |
| <span class="sourceLineNo">727</span> StoreMockMaker sm = new StoreMockMaker(results), sm2 = new StoreMockMaker(results);<a name="line.727"></a> |
| <span class="sourceLineNo">728</span> HStore store = sm.createStoreMock("store1");<a name="line.728"></a> |
| <span class="sourceLineNo">729</span> HStore store2 = sm2.createStoreMock("store2");<a name="line.729"></a> |
| <span class="sourceLineNo">730</span> BlockingStoreMockMaker blocker = new BlockingStoreMockMaker();<a name="line.730"></a> |
| <span class="sourceLineNo">731</span><a name="line.731"></a> |
| <span class="sourceLineNo">732</span> // First, block the compaction thread so that we could muck with queue.<a name="line.732"></a> |
| <span class="sourceLineNo">733</span> cst.requestSystemCompaction(r, blocker.createStoreMock(1, "b-pri1"), "b-pri1");<a name="line.733"></a> |
| <span class="sourceLineNo">734</span> BlockingStoreMockMaker.BlockingCompactionContext currentBlock = blocker.waitForBlocking();<a name="line.734"></a> |
| <span class="sourceLineNo">735</span><a name="line.735"></a> |
| <span class="sourceLineNo">736</span> // Add 4 files to store1, 3 to store2, and queue compactions; pri 3 and 4 respectively.<a name="line.736"></a> |
| <span class="sourceLineNo">737</span> for (int i = 0; i < 4; ++i) {<a name="line.737"></a> |
| <span class="sourceLineNo">738</span> sm.notCompacting.add(createFile());<a name="line.738"></a> |
| <span class="sourceLineNo">739</span> }<a name="line.739"></a> |
| <span class="sourceLineNo">740</span> cst.requestSystemCompaction(r, store, "s1-pri3");<a name="line.740"></a> |
| <span class="sourceLineNo">741</span> for (int i = 0; i < 3; ++i) {<a name="line.741"></a> |
| <span class="sourceLineNo">742</span> sm2.notCompacting.add(createFile());<a name="line.742"></a> |
| <span class="sourceLineNo">743</span> }<a name="line.743"></a> |
| <span class="sourceLineNo">744</span> cst.requestSystemCompaction(r, store2, "s2-pri4");<a name="line.744"></a> |
| <span class="sourceLineNo">745</span> // Now add 2 more files to store1 and queue compaction - pri 1.<a name="line.745"></a> |
| <span class="sourceLineNo">746</span> for (int i = 0; i < 2; ++i) {<a name="line.746"></a> |
| <span class="sourceLineNo">747</span> sm.notCompacting.add(createFile());<a name="line.747"></a> |
| <span class="sourceLineNo">748</span> }<a name="line.748"></a> |
| <span class="sourceLineNo">749</span> cst.requestSystemCompaction(r, store, "s1-pri1");<a name="line.749"></a> |
| <span class="sourceLineNo">750</span> // Finally add blocking compaction with priority 2.<a name="line.750"></a> |
| <span class="sourceLineNo">751</span> cst.requestSystemCompaction(r, blocker.createStoreMock(2, "b-pri2"), "b-pri2");<a name="line.751"></a> |
| <span class="sourceLineNo">752</span><a name="line.752"></a> |
| <span class="sourceLineNo">753</span> // Unblock the blocking compaction; we should run pri1 and become block again in pri2.<a name="line.753"></a> |
| <span class="sourceLineNo">754</span> currentBlock.unblock();<a name="line.754"></a> |
| <span class="sourceLineNo">755</span> currentBlock = blocker.waitForBlocking();<a name="line.755"></a> |
| <span class="sourceLineNo">756</span> // Pri1 should have "compacted" all 6 files.<a name="line.756"></a> |
| <span class="sourceLineNo">757</span> assertEquals(1, results.size());<a name="line.757"></a> |
| <span class="sourceLineNo">758</span> assertEquals(6, results.get(0).intValue());<a name="line.758"></a> |
| <span class="sourceLineNo">759</span> // Add 2 files to store 1 (it has 2 files now).<a name="line.759"></a> |
| <span class="sourceLineNo">760</span> for (int i = 0; i < 2; ++i) {<a name="line.760"></a> |
| <span class="sourceLineNo">761</span> sm.notCompacting.add(createFile());<a name="line.761"></a> |
| <span class="sourceLineNo">762</span> }<a name="line.762"></a> |
| <span class="sourceLineNo">763</span> // Now we have pri4 for store 2 in queue, and pri3 for store1; store1's current priority<a name="line.763"></a> |
| <span class="sourceLineNo">764</span> // is 5, however, so it must not preempt store 2. Add blocking compaction at the end.<a name="line.764"></a> |
| <span class="sourceLineNo">765</span> cst.requestSystemCompaction(r, blocker.createStoreMock(7, "b-pri7"), "b-pri7");<a name="line.765"></a> |
| <span class="sourceLineNo">766</span> currentBlock.unblock();<a name="line.766"></a> |
| <span class="sourceLineNo">767</span> currentBlock = blocker.waitForBlocking();<a name="line.767"></a> |
| <span class="sourceLineNo">768</span> assertEquals(3, results.size());<a name="line.768"></a> |
| <span class="sourceLineNo">769</span> assertEquals(3, results.get(1).intValue()); // 3 files should go before 2 files.<a name="line.769"></a> |
| <span class="sourceLineNo">770</span> assertEquals(2, results.get(2).intValue());<a name="line.770"></a> |
| <span class="sourceLineNo">771</span><a name="line.771"></a> |
| <span class="sourceLineNo">772</span> currentBlock.unblock();<a name="line.772"></a> |
| <span class="sourceLineNo">773</span> cst.interruptIfNecessary();<a name="line.773"></a> |
| <span class="sourceLineNo">774</span> }<a name="line.774"></a> |
| <span class="sourceLineNo">775</span><a name="line.775"></a> |
| <span class="sourceLineNo">776</span> /**<a name="line.776"></a> |
| <span class="sourceLineNo">777</span> * Firstly write 10 cells (with different time stamp) to a qualifier and flush<a name="line.777"></a> |
| <span class="sourceLineNo">778</span> * to hfile1, then write 10 cells (with different time stamp) to the same<a name="line.778"></a> |
| <span class="sourceLineNo">779</span> * qualifier and flush to hfile2. The latest cell (cell-A) in hfile1 and the<a name="line.779"></a> |
| <span class="sourceLineNo">780</span> * oldest cell (cell-B) in hfile2 are with the same time stamp but different<a name="line.780"></a> |
| <span class="sourceLineNo">781</span> * sequence id, and will get scanned successively during compaction.<a name="line.781"></a> |
| <span class="sourceLineNo">782</span> * <p/><a name="line.782"></a> |
| <span class="sourceLineNo">783</span> * We set compaction.kv.max to 10 so compaction will scan 10 versions each<a name="line.783"></a> |
| <span class="sourceLineNo">784</span> * round, meanwhile we set keepSeqIdPeriod=0 in {@link DummyCompactor} so all<a name="line.784"></a> |
| <span class="sourceLineNo">785</span> * 10 versions of hfile2 will be written out with seqId cleaned (set to 0)<a name="line.785"></a> |
| <span class="sourceLineNo">786</span> * including cell-B, then when scanner goes to cell-A it will cause a scan<a name="line.786"></a> |
| <span class="sourceLineNo">787</span> * out-of-order assertion error before HBASE-16931<a name="line.787"></a> |
| <span class="sourceLineNo">788</span> *<a name="line.788"></a> |
| <span class="sourceLineNo">789</span> * @throws Exception<a name="line.789"></a> |
| <span class="sourceLineNo">790</span> * if error occurs during the test<a name="line.790"></a> |
| <span class="sourceLineNo">791</span> */<a name="line.791"></a> |
| <span class="sourceLineNo">792</span> @Test<a name="line.792"></a> |
| <span class="sourceLineNo">793</span> public void testCompactionSeqId() throws Exception {<a name="line.793"></a> |
| <span class="sourceLineNo">794</span> final byte[] ROW = Bytes.toBytes("row");<a name="line.794"></a> |
| <span class="sourceLineNo">795</span> final byte[] QUALIFIER = Bytes.toBytes("qualifier");<a name="line.795"></a> |
| <span class="sourceLineNo">796</span><a name="line.796"></a> |
| <span class="sourceLineNo">797</span> long timestamp = 10000;<a name="line.797"></a> |
| <span class="sourceLineNo">798</span><a name="line.798"></a> |
| <span class="sourceLineNo">799</span> // row1/cf:a/10009/Put/vlen=2/seqid=11 V: v9<a name="line.799"></a> |
| <span class="sourceLineNo">800</span> // row1/cf:a/10008/Put/vlen=2/seqid=10 V: v8<a name="line.800"></a> |
| <span class="sourceLineNo">801</span> // row1/cf:a/10007/Put/vlen=2/seqid=9 V: v7<a name="line.801"></a> |
| <span class="sourceLineNo">802</span> // row1/cf:a/10006/Put/vlen=2/seqid=8 V: v6<a name="line.802"></a> |
| <span class="sourceLineNo">803</span> // row1/cf:a/10005/Put/vlen=2/seqid=7 V: v5<a name="line.803"></a> |
| <span class="sourceLineNo">804</span> // row1/cf:a/10004/Put/vlen=2/seqid=6 V: v4<a name="line.804"></a> |
| <span class="sourceLineNo">805</span> // row1/cf:a/10003/Put/vlen=2/seqid=5 V: v3<a name="line.805"></a> |
| <span class="sourceLineNo">806</span> // row1/cf:a/10002/Put/vlen=2/seqid=4 V: v2<a name="line.806"></a> |
| <span class="sourceLineNo">807</span> // row1/cf:a/10001/Put/vlen=2/seqid=3 V: v1<a name="line.807"></a> |
| <span class="sourceLineNo">808</span> // row1/cf:a/10000/Put/vlen=2/seqid=2 V: v0<a name="line.808"></a> |
| <span class="sourceLineNo">809</span> for (int i = 0; i < 10; i++) {<a name="line.809"></a> |
| <span class="sourceLineNo">810</span> Put put = new Put(ROW);<a name="line.810"></a> |
| <span class="sourceLineNo">811</span> put.addColumn(FAMILY, QUALIFIER, timestamp + i, Bytes.toBytes("v" + i));<a name="line.811"></a> |
| <span class="sourceLineNo">812</span> r.put(put);<a name="line.812"></a> |
| <span class="sourceLineNo">813</span> }<a name="line.813"></a> |
| <span class="sourceLineNo">814</span> r.flush(true);<a name="line.814"></a> |
| <span class="sourceLineNo">815</span><a name="line.815"></a> |
| <span class="sourceLineNo">816</span> // row1/cf:a/10018/Put/vlen=3/seqid=16 V: v18<a name="line.816"></a> |
| <span class="sourceLineNo">817</span> // row1/cf:a/10017/Put/vlen=3/seqid=17 V: v17<a name="line.817"></a> |
| <span class="sourceLineNo">818</span> // row1/cf:a/10016/Put/vlen=3/seqid=18 V: v16<a name="line.818"></a> |
| <span class="sourceLineNo">819</span> // row1/cf:a/10015/Put/vlen=3/seqid=19 V: v15<a name="line.819"></a> |
| <span class="sourceLineNo">820</span> // row1/cf:a/10014/Put/vlen=3/seqid=20 V: v14<a name="line.820"></a> |
| <span class="sourceLineNo">821</span> // row1/cf:a/10013/Put/vlen=3/seqid=21 V: v13<a name="line.821"></a> |
| <span class="sourceLineNo">822</span> // row1/cf:a/10012/Put/vlen=3/seqid=22 V: v12<a name="line.822"></a> |
| <span class="sourceLineNo">823</span> // row1/cf:a/10011/Put/vlen=3/seqid=23 V: v11<a name="line.823"></a> |
| <span class="sourceLineNo">824</span> // row1/cf:a/10010/Put/vlen=3/seqid=24 V: v10<a name="line.824"></a> |
| <span class="sourceLineNo">825</span> // row1/cf:a/10009/Put/vlen=2/seqid=25 V: v9<a name="line.825"></a> |
| <span class="sourceLineNo">826</span> for (int i = 18; i > 8; i--) {<a name="line.826"></a> |
| <span class="sourceLineNo">827</span> Put put = new Put(ROW);<a name="line.827"></a> |
| <span class="sourceLineNo">828</span> put.addColumn(FAMILY, QUALIFIER, timestamp + i, Bytes.toBytes("v" + i));<a name="line.828"></a> |
| <span class="sourceLineNo">829</span> r.put(put);<a name="line.829"></a> |
| <span class="sourceLineNo">830</span> }<a name="line.830"></a> |
| <span class="sourceLineNo">831</span> r.flush(true);<a name="line.831"></a> |
| <span class="sourceLineNo">832</span> r.compact(true);<a name="line.832"></a> |
| <span class="sourceLineNo">833</span> }<a name="line.833"></a> |
| <span class="sourceLineNo">834</span><a name="line.834"></a> |
| <span class="sourceLineNo">835</span> public static class DummyCompactor extends DefaultCompactor {<a name="line.835"></a> |
| <span class="sourceLineNo">836</span> public DummyCompactor(Configuration conf, HStore store) {<a name="line.836"></a> |
| <span class="sourceLineNo">837</span> super(conf, store);<a name="line.837"></a> |
| <span class="sourceLineNo">838</span> this.keepSeqIdPeriod = 0;<a name="line.838"></a> |
| <span class="sourceLineNo">839</span> }<a name="line.839"></a> |
| <span class="sourceLineNo">840</span> }<a name="line.840"></a> |
| <span class="sourceLineNo">841</span><a name="line.841"></a> |
| <span class="sourceLineNo">842</span> private static HStoreFile createFile() throws Exception {<a name="line.842"></a> |
| <span class="sourceLineNo">843</span> HStoreFile sf = mock(HStoreFile.class);<a name="line.843"></a> |
| <span class="sourceLineNo">844</span> when(sf.getPath()).thenReturn(new Path("file"));<a name="line.844"></a> |
| <span class="sourceLineNo">845</span> StoreFileReader r = mock(StoreFileReader.class);<a name="line.845"></a> |
| <span class="sourceLineNo">846</span> when(r.length()).thenReturn(10L);<a name="line.846"></a> |
| <span class="sourceLineNo">847</span> when(sf.getReader()).thenReturn(r);<a name="line.847"></a> |
| <span class="sourceLineNo">848</span> return sf;<a name="line.848"></a> |
| <span class="sourceLineNo">849</span> }<a name="line.849"></a> |
| <span class="sourceLineNo">850</span><a name="line.850"></a> |
| <span class="sourceLineNo">851</span> /**<a name="line.851"></a> |
| <span class="sourceLineNo">852</span> * Simple {@link CompactionLifeCycleTracker} on which you can wait until the requested compaction<a name="line.852"></a> |
| <span class="sourceLineNo">853</span> * finishes.<a name="line.853"></a> |
| <span class="sourceLineNo">854</span> */<a name="line.854"></a> |
| <span class="sourceLineNo">855</span> public static class Tracker implements CompactionLifeCycleTracker {<a name="line.855"></a> |
| <span class="sourceLineNo">856</span><a name="line.856"></a> |
| <span class="sourceLineNo">857</span> private final CountDownLatch done;<a name="line.857"></a> |
| <span class="sourceLineNo">858</span><a name="line.858"></a> |
| <span class="sourceLineNo">859</span> public Tracker(CountDownLatch done) {<a name="line.859"></a> |
| <span class="sourceLineNo">860</span> this.done = done;<a name="line.860"></a> |
| <span class="sourceLineNo">861</span> }<a name="line.861"></a> |
| <span class="sourceLineNo">862</span><a name="line.862"></a> |
| <span class="sourceLineNo">863</span> @Override<a name="line.863"></a> |
| <span class="sourceLineNo">864</span> public void afterExecution(Store store) {<a name="line.864"></a> |
| <span class="sourceLineNo">865</span> done.countDown();<a name="line.865"></a> |
| <span class="sourceLineNo">866</span> }<a name="line.866"></a> |
| <span class="sourceLineNo">867</span> }<a name="line.867"></a> |
| <span class="sourceLineNo">868</span><a name="line.868"></a> |
| <span class="sourceLineNo">869</span> /**<a name="line.869"></a> |
| <span class="sourceLineNo">870</span> * Simple {@link CompactionLifeCycleTracker} on which you can wait until the requested compaction<a name="line.870"></a> |
| <span class="sourceLineNo">871</span> * finishes.<a name="line.871"></a> |
| <span class="sourceLineNo">872</span> */<a name="line.872"></a> |
| <span class="sourceLineNo">873</span> public static class WaitThroughPutController extends NoLimitThroughputController{<a name="line.873"></a> |
| <span class="sourceLineNo">874</span><a name="line.874"></a> |
| <span class="sourceLineNo">875</span> public WaitThroughPutController() {<a name="line.875"></a> |
| <span class="sourceLineNo">876</span> }<a name="line.876"></a> |
| <span class="sourceLineNo">877</span><a name="line.877"></a> |
| <span class="sourceLineNo">878</span> @Override<a name="line.878"></a> |
| <span class="sourceLineNo">879</span> public long control(String compactionName, long size) throws InterruptedException {<a name="line.879"></a> |
| <span class="sourceLineNo">880</span> Thread.sleep(6000000);<a name="line.880"></a> |
| <span class="sourceLineNo">881</span> return 6000000;<a name="line.881"></a> |
| <span class="sourceLineNo">882</span> }<a name="line.882"></a> |
| <span class="sourceLineNo">883</span> }<a name="line.883"></a> |
| <span class="sourceLineNo">884</span>}<a name="line.884"></a> |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| </pre> |
| </div> |
| </body> |
| </html> |