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