http://git-wip-us.apache.org/repos/asf/hbase-site/blob/0383a9c2/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/CompactSplit.html ---------------------------------------------------------------------- diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/CompactSplit.html b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/CompactSplit.html index 06b7a03..7dabb5e 100644 --- a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/CompactSplit.html +++ b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/CompactSplit.html @@ -216,519 +216,505 @@ <span class="sourceLineNo">208</span> return queueLists.toString();<a name="line.208"></a> <span class="sourceLineNo">209</span> }<a name="line.209"></a> <span class="sourceLineNo">210</span><a name="line.210"></a> -<span class="sourceLineNo">211</span> public synchronized void requestRegionsMerge(final Region a,<a name="line.211"></a> -<span class="sourceLineNo">212</span> final Region b, final boolean forcible, long masterSystemTime, User user) {<a name="line.212"></a> -<span class="sourceLineNo">213</span> try {<a name="line.213"></a> -<span class="sourceLineNo">214</span> mergePool.execute(new RegionMergeRequest(a, b, this.server, forcible, masterSystemTime,user));<a name="line.214"></a> -<span class="sourceLineNo">215</span> if (LOG.isDebugEnabled()) {<a name="line.215"></a> -<span class="sourceLineNo">216</span> LOG.debug("Region merge requested for " + a + "," + b + ", forcible="<a name="line.216"></a> -<span class="sourceLineNo">217</span> + forcible + ". " + this);<a name="line.217"></a> +<span class="sourceLineNo">211</span> public synchronized boolean requestSplit(final Region r) {<a name="line.211"></a> +<span class="sourceLineNo">212</span> // don't split regions that are blocking<a name="line.212"></a> +<span class="sourceLineNo">213</span> if (shouldSplitRegion() && ((HRegion)r).getCompactPriority() >= Store.PRIORITY_USER) {<a name="line.213"></a> +<span class="sourceLineNo">214</span> byte[] midKey = ((HRegion)r).checkSplit();<a name="line.214"></a> +<span class="sourceLineNo">215</span> if (midKey != null) {<a name="line.215"></a> +<span class="sourceLineNo">216</span> requestSplit(r, midKey);<a name="line.216"></a> +<span class="sourceLineNo">217</span> return true;<a name="line.217"></a> <span class="sourceLineNo">218</span> }<a name="line.218"></a> -<span class="sourceLineNo">219</span> } catch (RejectedExecutionException ree) {<a name="line.219"></a> -<span class="sourceLineNo">220</span> LOG.warn("Could not execute merge for " + a + "," + b + ", forcible="<a name="line.220"></a> -<span class="sourceLineNo">221</span> + forcible, ree);<a name="line.221"></a> -<span class="sourceLineNo">222</span> }<a name="line.222"></a> -<span class="sourceLineNo">223</span> }<a name="line.223"></a> -<span class="sourceLineNo">224</span><a name="line.224"></a> -<span class="sourceLineNo">225</span> public synchronized boolean requestSplit(final Region r) {<a name="line.225"></a> -<span class="sourceLineNo">226</span> // don't split regions that are blocking<a name="line.226"></a> -<span class="sourceLineNo">227</span> if (shouldSplitRegion() && ((HRegion)r).getCompactPriority() >= Store.PRIORITY_USER) {<a name="line.227"></a> -<span class="sourceLineNo">228</span> byte[] midKey = ((HRegion)r).checkSplit();<a name="line.228"></a> -<span class="sourceLineNo">229</span> if (midKey != null) {<a name="line.229"></a> -<span class="sourceLineNo">230</span> requestSplit(r, midKey);<a name="line.230"></a> -<span class="sourceLineNo">231</span> return true;<a name="line.231"></a> -<span class="sourceLineNo">232</span> }<a name="line.232"></a> -<span class="sourceLineNo">233</span> }<a name="line.233"></a> -<span class="sourceLineNo">234</span> return false;<a name="line.234"></a> -<span class="sourceLineNo">235</span> }<a name="line.235"></a> -<span class="sourceLineNo">236</span><a name="line.236"></a> -<span class="sourceLineNo">237</span> public synchronized void requestSplit(final Region r, byte[] midKey) {<a name="line.237"></a> -<span class="sourceLineNo">238</span> requestSplit(r, midKey, null);<a name="line.238"></a> -<span class="sourceLineNo">239</span> }<a name="line.239"></a> -<span class="sourceLineNo">240</span><a name="line.240"></a> -<span class="sourceLineNo">241</span> /*<a name="line.241"></a> -<span class="sourceLineNo">242</span> * The User parameter allows the split thread to assume the correct user identity<a name="line.242"></a> -<span class="sourceLineNo">243</span> */<a name="line.243"></a> -<span class="sourceLineNo">244</span> public synchronized void requestSplit(final Region r, byte[] midKey, User user) {<a name="line.244"></a> -<span class="sourceLineNo">245</span> if (midKey == null) {<a name="line.245"></a> -<span class="sourceLineNo">246</span> LOG.debug("Region " + r.getRegionInfo().getRegionNameAsString() +<a name="line.246"></a> -<span class="sourceLineNo">247</span> " not splittable because midkey=null");<a name="line.247"></a> -<span class="sourceLineNo">248</span> if (((HRegion)r).shouldForceSplit()) {<a name="line.248"></a> -<span class="sourceLineNo">249</span> ((HRegion)r).clearSplit();<a name="line.249"></a> -<span class="sourceLineNo">250</span> }<a name="line.250"></a> -<span class="sourceLineNo">251</span> return;<a name="line.251"></a> -<span class="sourceLineNo">252</span> }<a name="line.252"></a> -<span class="sourceLineNo">253</span> try {<a name="line.253"></a> -<span class="sourceLineNo">254</span> this.splits.execute(new SplitRequest(r, midKey, this.server, user));<a name="line.254"></a> -<span class="sourceLineNo">255</span> if (LOG.isDebugEnabled()) {<a name="line.255"></a> -<span class="sourceLineNo">256</span> LOG.debug("Splitting " + r + ", " + this);<a name="line.256"></a> -<span class="sourceLineNo">257</span> }<a name="line.257"></a> -<span class="sourceLineNo">258</span> } catch (RejectedExecutionException ree) {<a name="line.258"></a> -<span class="sourceLineNo">259</span> LOG.info("Could not execute split for " + r, ree);<a name="line.259"></a> -<span class="sourceLineNo">260</span> }<a name="line.260"></a> -<span class="sourceLineNo">261</span> }<a name="line.261"></a> -<span class="sourceLineNo">262</span><a name="line.262"></a> -<span class="sourceLineNo">263</span> @Override<a name="line.263"></a> -<span class="sourceLineNo">264</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why)<a name="line.264"></a> -<span class="sourceLineNo">265</span> throws IOException {<a name="line.265"></a> -<span class="sourceLineNo">266</span> return requestCompaction(r, why, null);<a name="line.266"></a> -<span class="sourceLineNo">267</span> }<a name="line.267"></a> -<span class="sourceLineNo">268</span><a name="line.268"></a> -<span class="sourceLineNo">269</span> @Override<a name="line.269"></a> -<span class="sourceLineNo">270</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why,<a name="line.270"></a> -<span class="sourceLineNo">271</span> List<Pair<CompactionRequest, Store>> requests) throws IOException {<a name="line.271"></a> -<span class="sourceLineNo">272</span> return requestCompaction(r, why, Store.NO_PRIORITY, requests, null);<a name="line.272"></a> -<span class="sourceLineNo">273</span> }<a name="line.273"></a> -<span class="sourceLineNo">274</span><a name="line.274"></a> -<span class="sourceLineNo">275</span> @Override<a name="line.275"></a> -<span class="sourceLineNo">276</span> public synchronized CompactionRequest requestCompaction(final Region r, final Store s,<a name="line.276"></a> -<span class="sourceLineNo">277</span> final String why, CompactionRequest request) throws IOException {<a name="line.277"></a> -<span class="sourceLineNo">278</span> return requestCompaction(r, s, why, Store.NO_PRIORITY, request, null);<a name="line.278"></a> -<span class="sourceLineNo">279</span> }<a name="line.279"></a> -<span class="sourceLineNo">280</span><a name="line.280"></a> -<span class="sourceLineNo">281</span> @Override<a name="line.281"></a> -<span class="sourceLineNo">282</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why,<a name="line.282"></a> -<span class="sourceLineNo">283</span> int p, List<Pair<CompactionRequest, Store>> requests, User user) throws IOException {<a name="line.283"></a> -<span class="sourceLineNo">284</span> return requestCompactionInternal(r, why, p, requests, true, user);<a name="line.284"></a> -<span class="sourceLineNo">285</span> }<a name="line.285"></a> -<span class="sourceLineNo">286</span><a name="line.286"></a> -<span class="sourceLineNo">287</span> private List<CompactionRequest> requestCompactionInternal(final Region r, final String why,<a name="line.287"></a> -<span class="sourceLineNo">288</span> int p, List<Pair<CompactionRequest, Store>> requests, boolean selectNow, User user)<a name="line.288"></a> -<span class="sourceLineNo">289</span> throws IOException {<a name="line.289"></a> -<span class="sourceLineNo">290</span> // not a special compaction request, so make our own list<a name="line.290"></a> -<span class="sourceLineNo">291</span> List<CompactionRequest> ret = null;<a name="line.291"></a> -<span class="sourceLineNo">292</span> if (requests == null) {<a name="line.292"></a> -<span class="sourceLineNo">293</span> ret = selectNow ? new ArrayList<CompactionRequest>(r.getStores().size()) : null;<a name="line.293"></a> -<span class="sourceLineNo">294</span> for (Store s : r.getStores()) {<a name="line.294"></a> -<span class="sourceLineNo">295</span> CompactionRequest cr = requestCompactionInternal(r, s, why, p, null, selectNow, user);<a name="line.295"></a> -<span class="sourceLineNo">296</span> if (selectNow) ret.add(cr);<a name="line.296"></a> -<span class="sourceLineNo">297</span> }<a name="line.297"></a> -<span class="sourceLineNo">298</span> } else {<a name="line.298"></a> -<span class="sourceLineNo">299</span> Preconditions.checkArgument(selectNow); // only system requests have selectNow == false<a name="line.299"></a> -<span class="sourceLineNo">300</span> ret = new ArrayList<CompactionRequest>(requests.size());<a name="line.300"></a> -<span class="sourceLineNo">301</span> for (Pair<CompactionRequest, Store> pair : requests) {<a name="line.301"></a> -<span class="sourceLineNo">302</span> ret.add(requestCompaction(r, pair.getSecond(), why, p, pair.getFirst(), user));<a name="line.302"></a> -<span class="sourceLineNo">303</span> }<a name="line.303"></a> -<span class="sourceLineNo">304</span> }<a name="line.304"></a> -<span class="sourceLineNo">305</span> return ret;<a name="line.305"></a> -<span class="sourceLineNo">306</span> }<a name="line.306"></a> -<span class="sourceLineNo">307</span><a name="line.307"></a> -<span class="sourceLineNo">308</span> public CompactionRequest requestCompaction(final Region r, final Store s,<a name="line.308"></a> -<span class="sourceLineNo">309</span> final String why, int priority, CompactionRequest request, User user) throws IOException {<a name="line.309"></a> -<span class="sourceLineNo">310</span> return requestCompactionInternal(r, s, why, priority, request, true, user);<a name="line.310"></a> -<span class="sourceLineNo">311</span> }<a name="line.311"></a> -<span class="sourceLineNo">312</span><a name="line.312"></a> -<span class="sourceLineNo">313</span> public synchronized void requestSystemCompaction(<a name="line.313"></a> -<span class="sourceLineNo">314</span> final Region r, final String why) throws IOException {<a name="line.314"></a> -<span class="sourceLineNo">315</span> requestCompactionInternal(r, why, Store.NO_PRIORITY, null, false, null);<a name="line.315"></a> -<span class="sourceLineNo">316</span> }<a name="line.316"></a> -<span class="sourceLineNo">317</span><a name="line.317"></a> -<span class="sourceLineNo">318</span> public void requestSystemCompaction(<a name="line.318"></a> -<span class="sourceLineNo">319</span> final Region r, final Store s, final String why) throws IOException {<a name="line.319"></a> -<span class="sourceLineNo">320</span> requestCompactionInternal(r, s, why, Store.NO_PRIORITY, null, false, null);<a name="line.320"></a> -<span class="sourceLineNo">321</span> }<a name="line.321"></a> -<span class="sourceLineNo">322</span><a name="line.322"></a> -<span class="sourceLineNo">323</span> /**<a name="line.323"></a> -<span class="sourceLineNo">324</span> * @param r region store belongs to<a name="line.324"></a> -<span class="sourceLineNo">325</span> * @param s Store to request compaction on<a name="line.325"></a> -<span class="sourceLineNo">326</span> * @param why Why compaction requested -- used in debug messages<a name="line.326"></a> -<span class="sourceLineNo">327</span> * @param priority override the default priority (NO_PRIORITY == decide)<a name="line.327"></a> -<span class="sourceLineNo">328</span> * @param request custom compaction request. Can be <tt>null</tt> in which case a simple<a name="line.328"></a> -<span class="sourceLineNo">329</span> * compaction will be used.<a name="line.329"></a> -<span class="sourceLineNo">330</span> */<a name="line.330"></a> -<span class="sourceLineNo">331</span> private synchronized CompactionRequest requestCompactionInternal(final Region r, final Store s,<a name="line.331"></a> -<span class="sourceLineNo">332</span> final String why, int priority, CompactionRequest request, boolean selectNow, User user)<a name="line.332"></a> -<span class="sourceLineNo">333</span> throws IOException {<a name="line.333"></a> -<span class="sourceLineNo">334</span> if (this.server.isStopped()<a name="line.334"></a> -<span class="sourceLineNo">335</span> || (r.getTableDescriptor() != null && !r.getTableDescriptor().isCompactionEnabled())) {<a name="line.335"></a> -<span class="sourceLineNo">336</span> return null;<a name="line.336"></a> -<span class="sourceLineNo">337</span> }<a name="line.337"></a> -<span class="sourceLineNo">338</span><a name="line.338"></a> -<span class="sourceLineNo">339</span> CompactionContext compaction = null;<a name="line.339"></a> -<span class="sourceLineNo">340</span> if (selectNow) {<a name="line.340"></a> -<span class="sourceLineNo">341</span> compaction = selectCompaction(r, s, priority, request, user);<a name="line.341"></a> -<span class="sourceLineNo">342</span> if (compaction == null) return null; // message logged inside<a name="line.342"></a> -<span class="sourceLineNo">343</span> }<a name="line.343"></a> -<span class="sourceLineNo">344</span><a name="line.344"></a> -<span class="sourceLineNo">345</span> final RegionServerSpaceQuotaManager spaceQuotaManager =<a name="line.345"></a> -<span class="sourceLineNo">346</span> this.server.getRegionServerSpaceQuotaManager();<a name="line.346"></a> -<span class="sourceLineNo">347</span> if (spaceQuotaManager != null && spaceQuotaManager.areCompactionsDisabled(<a name="line.347"></a> -<span class="sourceLineNo">348</span> r.getTableDescriptor().getTableName())) {<a name="line.348"></a> -<span class="sourceLineNo">349</span> if (LOG.isDebugEnabled()) {<a name="line.349"></a> -<span class="sourceLineNo">350</span> LOG.debug("Ignoring compaction request for " + r + " as an active space quota violation "<a name="line.350"></a> -<span class="sourceLineNo">351</span> + " policy disallows compactions.");<a name="line.351"></a> -<span class="sourceLineNo">352</span> }<a name="line.352"></a> -<span class="sourceLineNo">353</span> return null;<a name="line.353"></a> -<span class="sourceLineNo">354</span> }<a name="line.354"></a> -<span class="sourceLineNo">355</span><a name="line.355"></a> -<span class="sourceLineNo">356</span> // We assume that most compactions are small. So, put system compactions into small<a name="line.356"></a> -<span class="sourceLineNo">357</span> // pool; we will do selection there, and move to large pool if necessary.<a name="line.357"></a> -<span class="sourceLineNo">358</span> ThreadPoolExecutor pool = (selectNow && s.throttleCompaction(compaction.getRequest().getSize()))<a name="line.358"></a> -<span class="sourceLineNo">359</span> ? longCompactions : shortCompactions;<a name="line.359"></a> -<span class="sourceLineNo">360</span> pool.execute(new CompactionRunner(s, r, compaction, pool, user));<a name="line.360"></a> -<span class="sourceLineNo">361</span> if (LOG.isDebugEnabled()) {<a name="line.361"></a> -<span class="sourceLineNo">362</span> String type = (pool == shortCompactions) ? "Small " : "Large ";<a name="line.362"></a> -<span class="sourceLineNo">363</span> LOG.debug(type + "Compaction requested: " + (selectNow ? compaction.toString() : "system")<a name="line.363"></a> -<span class="sourceLineNo">364</span> + (why != null && !why.isEmpty() ? "; Because: " + why : "") + "; " + this);<a name="line.364"></a> -<span class="sourceLineNo">365</span> }<a name="line.365"></a> -<span class="sourceLineNo">366</span> return selectNow ? compaction.getRequest() : null;<a name="line.366"></a> -<span class="sourceLineNo">367</span> }<a name="line.367"></a> -<span class="sourceLineNo">368</span><a name="line.368"></a> -<span class="sourceLineNo">369</span> private CompactionContext selectCompaction(final Region r, final Store s,<a name="line.369"></a> -<span class="sourceLineNo">370</span> int priority, CompactionRequest request, User user) throws IOException {<a name="line.370"></a> -<span class="sourceLineNo">371</span> CompactionContext compaction = s.requestCompaction(priority, request, user);<a name="line.371"></a> -<span class="sourceLineNo">372</span> if (compaction == null) {<a name="line.372"></a> -<span class="sourceLineNo">373</span> if(LOG.isDebugEnabled() && r.getRegionInfo() != null) {<a name="line.373"></a> -<span class="sourceLineNo">374</span> LOG.debug("Not compacting " + r.getRegionInfo().getRegionNameAsString() +<a name="line.374"></a> -<span class="sourceLineNo">375</span> " because compaction request was cancelled");<a name="line.375"></a> -<span class="sourceLineNo">376</span> }<a name="line.376"></a> -<span class="sourceLineNo">377</span> return null;<a name="line.377"></a> -<span class="sourceLineNo">378</span> }<a name="line.378"></a> -<span class="sourceLineNo">379</span> assert compaction.hasSelection();<a name="line.379"></a> -<span class="sourceLineNo">380</span> if (priority != Store.NO_PRIORITY) {<a name="line.380"></a> -<span class="sourceLineNo">381</span> compaction.getRequest().setPriority(priority);<a name="line.381"></a> -<span class="sourceLineNo">382</span> }<a name="line.382"></a> -<span class="sourceLineNo">383</span> return compaction;<a name="line.383"></a> -<span class="sourceLineNo">384</span> }<a name="line.384"></a> -<span class="sourceLineNo">385</span><a name="line.385"></a> -<span class="sourceLineNo">386</span> /**<a name="line.386"></a> -<span class="sourceLineNo">387</span> * Only interrupt once it's done with a run through the work loop.<a name="line.387"></a> -<span class="sourceLineNo">388</span> */<a name="line.388"></a> -<span class="sourceLineNo">389</span> void interruptIfNecessary() {<a name="line.389"></a> -<span class="sourceLineNo">390</span> splits.shutdown();<a name="line.390"></a> -<span class="sourceLineNo">391</span> longCompactions.shutdown();<a name="line.391"></a> -<span class="sourceLineNo">392</span> shortCompactions.shutdown();<a name="line.392"></a> -<span class="sourceLineNo">393</span> }<a name="line.393"></a> -<span class="sourceLineNo">394</span><a name="line.394"></a> -<span class="sourceLineNo">395</span> private void waitFor(ThreadPoolExecutor t, String name) {<a name="line.395"></a> -<span class="sourceLineNo">396</span> boolean done = false;<a name="line.396"></a> -<span class="sourceLineNo">397</span> while (!done) {<a name="line.397"></a> -<span class="sourceLineNo">398</span> try {<a name="line.398"></a> -<span class="sourceLineNo">399</span> done = t.awaitTermination(60, TimeUnit.SECONDS);<a name="line.399"></a> -<span class="sourceLineNo">400</span> LOG.info("Waiting for " + name + " to finish...");<a name="line.400"></a> -<span class="sourceLineNo">401</span> if (!done) {<a name="line.401"></a> -<span class="sourceLineNo">402</span> t.shutdownNow();<a name="line.402"></a> -<span class="sourceLineNo">403</span> }<a name="line.403"></a> -<span class="sourceLineNo">404</span> } catch (InterruptedException ie) {<a name="line.404"></a> -<span class="sourceLineNo">405</span> LOG.warn("Interrupted waiting for " + name + " to finish...");<a name="line.405"></a> -<span class="sourceLineNo">406</span> }<a name="line.406"></a> -<span class="sourceLineNo">407</span> }<a name="line.407"></a> -<span class="sourceLineNo">408</span> }<a name="line.408"></a> -<span class="sourceLineNo">409</span><a name="line.409"></a> -<span class="sourceLineNo">410</span> void join() {<a name="line.410"></a> -<span class="sourceLineNo">411</span> waitFor(splits, "Split Thread");<a name="line.411"></a> -<span class="sourceLineNo">412</span> waitFor(longCompactions, "Large Compaction Thread");<a name="line.412"></a> -<span class="sourceLineNo">413</span> waitFor(shortCompactions, "Small Compaction Thread");<a name="line.413"></a> +<span class="sourceLineNo">219</span> }<a name="line.219"></a> +<span class="sourceLineNo">220</span> return false;<a name="line.220"></a> +<span class="sourceLineNo">221</span> }<a name="line.221"></a> +<span class="sourceLineNo">222</span><a name="line.222"></a> +<span class="sourceLineNo">223</span> public synchronized void requestSplit(final Region r, byte[] midKey) {<a name="line.223"></a> +<span class="sourceLineNo">224</span> requestSplit(r, midKey, null);<a name="line.224"></a> +<span class="sourceLineNo">225</span> }<a name="line.225"></a> +<span class="sourceLineNo">226</span><a name="line.226"></a> +<span class="sourceLineNo">227</span> /*<a name="line.227"></a> +<span class="sourceLineNo">228</span> * The User parameter allows the split thread to assume the correct user identity<a name="line.228"></a> +<span class="sourceLineNo">229</span> */<a name="line.229"></a> +<span class="sourceLineNo">230</span> public synchronized void requestSplit(final Region r, byte[] midKey, User user) {<a name="line.230"></a> +<span class="sourceLineNo">231</span> if (midKey == null) {<a name="line.231"></a> +<span class="sourceLineNo">232</span> LOG.debug("Region " + r.getRegionInfo().getRegionNameAsString() +<a name="line.232"></a> +<span class="sourceLineNo">233</span> " not splittable because midkey=null");<a name="line.233"></a> +<span class="sourceLineNo">234</span> if (((HRegion)r).shouldForceSplit()) {<a name="line.234"></a> +<span class="sourceLineNo">235</span> ((HRegion)r).clearSplit();<a name="line.235"></a> +<span class="sourceLineNo">236</span> }<a name="line.236"></a> +<span class="sourceLineNo">237</span> return;<a name="line.237"></a> +<span class="sourceLineNo">238</span> }<a name="line.238"></a> +<span class="sourceLineNo">239</span> try {<a name="line.239"></a> +<span class="sourceLineNo">240</span> this.splits.execute(new SplitRequest(r, midKey, this.server, user));<a name="line.240"></a> +<span class="sourceLineNo">241</span> if (LOG.isDebugEnabled()) {<a name="line.241"></a> +<span class="sourceLineNo">242</span> LOG.debug("Splitting " + r + ", " + this);<a name="line.242"></a> +<span class="sourceLineNo">243</span> }<a name="line.243"></a> +<span class="sourceLineNo">244</span> } catch (RejectedExecutionException ree) {<a name="line.244"></a> +<span class="sourceLineNo">245</span> LOG.info("Could not execute split for " + r, ree);<a name="line.245"></a> +<span class="sourceLineNo">246</span> }<a name="line.246"></a> +<span class="sourceLineNo">247</span> }<a name="line.247"></a> +<span class="sourceLineNo">248</span><a name="line.248"></a> +<span class="sourceLineNo">249</span> @Override<a name="line.249"></a> +<span class="sourceLineNo">250</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why)<a name="line.250"></a> +<span class="sourceLineNo">251</span> throws IOException {<a name="line.251"></a> +<span class="sourceLineNo">252</span> return requestCompaction(r, why, null);<a name="line.252"></a> +<span class="sourceLineNo">253</span> }<a name="line.253"></a> +<span class="sourceLineNo">254</span><a name="line.254"></a> +<span class="sourceLineNo">255</span> @Override<a name="line.255"></a> +<span class="sourceLineNo">256</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why,<a name="line.256"></a> +<span class="sourceLineNo">257</span> List<Pair<CompactionRequest, Store>> requests) throws IOException {<a name="line.257"></a> +<span class="sourceLineNo">258</span> return requestCompaction(r, why, Store.NO_PRIORITY, requests, null);<a name="line.258"></a> +<span class="sourceLineNo">259</span> }<a name="line.259"></a> +<span class="sourceLineNo">260</span><a name="line.260"></a> +<span class="sourceLineNo">261</span> @Override<a name="line.261"></a> +<span class="sourceLineNo">262</span> public synchronized CompactionRequest requestCompaction(final Region r, final Store s,<a name="line.262"></a> +<span class="sourceLineNo">263</span> final String why, CompactionRequest request) throws IOException {<a name="line.263"></a> +<span class="sourceLineNo">264</span> return requestCompaction(r, s, why, Store.NO_PRIORITY, request, null);<a name="line.264"></a> +<span class="sourceLineNo">265</span> }<a name="line.265"></a> +<span class="sourceLineNo">266</span><a name="line.266"></a> +<span class="sourceLineNo">267</span> @Override<a name="line.267"></a> +<span class="sourceLineNo">268</span> public synchronized List<CompactionRequest> requestCompaction(final Region r, final String why,<a name="line.268"></a> +<span class="sourceLineNo">269</span> int p, List<Pair<CompactionRequest, Store>> requests, User user) throws IOException {<a name="line.269"></a> +<span class="sourceLineNo">270</span> return requestCompactionInternal(r, why, p, requests, true, user);<a name="line.270"></a> +<span class="sourceLineNo">271</span> }<a name="line.271"></a> +<span class="sourceLineNo">272</span><a name="line.272"></a> +<span class="sourceLineNo">273</span> private List<CompactionRequest> requestCompactionInternal(final Region r, final String why,<a name="line.273"></a> +<span class="sourceLineNo">274</span> int p, List<Pair<CompactionRequest, Store>> requests, boolean selectNow, User user)<a name="line.274"></a> +<span class="sourceLineNo">275</span> throws IOException {<a name="line.275"></a> +<span class="sourceLineNo">276</span> // not a special compaction request, so make our own list<a name="line.276"></a> +<span class="sourceLineNo">277</span> List<CompactionRequest> ret = null;<a name="line.277"></a> +<span class="sourceLineNo">278</span> if (requests == null) {<a name="line.278"></a> +<span class="sourceLineNo">279</span> ret = selectNow ? new ArrayList<CompactionRequest>(r.getStores().size()) : null;<a name="line.279"></a> +<span class="sourceLineNo">280</span> for (Store s : r.getStores()) {<a name="line.280"></a> +<span class="sourceLineNo">281</span> CompactionRequest cr = requestCompactionInternal(r, s, why, p, null, selectNow, user);<a name="line.281"></a> +<span class="sourceLineNo">282</span> if (selectNow) ret.add(cr);<a name="line.282"></a> +<span class="sourceLineNo">283</span> }<a name="line.283"></a> +<span class="sourceLineNo">284</span> } else {<a name="line.284"></a> +<span class="sourceLineNo">285</span> Preconditions.checkArgument(selectNow); // only system requests have selectNow == false<a name="line.285"></a> +<span class="sourceLineNo">286</span> ret = new ArrayList<CompactionRequest>(requests.size());<a name="line.286"></a> +<span class="sourceLineNo">287</span> for (Pair<CompactionRequest, Store> pair : requests) {<a name="line.287"></a> +<span class="sourceLineNo">288</span> ret.add(requestCompaction(r, pair.getSecond(), why, p, pair.getFirst(), user));<a name="line.288"></a> +<span class="sourceLineNo">289</span> }<a name="line.289"></a> +<span class="sourceLineNo">290</span> }<a name="line.290"></a> +<span class="sourceLineNo">291</span> return ret;<a name="line.291"></a> +<span class="sourceLineNo">292</span> }<a name="line.292"></a> +<span class="sourceLineNo">293</span><a name="line.293"></a> +<span class="sourceLineNo">294</span> public CompactionRequest requestCompaction(final Region r, final Store s,<a name="line.294"></a> +<span class="sourceLineNo">295</span> final String why, int priority, CompactionRequest request, User user) throws IOException {<a name="line.295"></a> +<span class="sourceLineNo">296</span> return requestCompactionInternal(r, s, why, priority, request, true, user);<a name="line.296"></a> +<span class="sourceLineNo">297</span> }<a name="line.297"></a> +<span class="sourceLineNo">298</span><a name="line.298"></a> +<span class="sourceLineNo">299</span> public synchronized void requestSystemCompaction(<a name="line.299"></a> +<span class="sourceLineNo">300</span> final Region r, final String why) throws IOException {<a name="line.300"></a> +<span class="sourceLineNo">301</span> requestCompactionInternal(r, why, Store.NO_PRIORITY, null, false, null);<a name="line.301"></a> +<span class="sourceLineNo">302</span> }<a name="line.302"></a> +<span class="sourceLineNo">303</span><a name="line.303"></a> +<span class="sourceLineNo">304</span> public void requestSystemCompaction(<a name="line.304"></a> +<span class="sourceLineNo">305</span> final Region r, final Store s, final String why) throws IOException {<a name="line.305"></a> +<span class="sourceLineNo">306</span> requestCompactionInternal(r, s, why, Store.NO_PRIORITY, null, false, null);<a name="line.306"></a> +<span class="sourceLineNo">307</span> }<a name="line.307"></a> +<span class="sourceLineNo">308</span><a name="line.308"></a> +<span class="sourceLineNo">309</span> /**<a name="line.309"></a> +<span class="sourceLineNo">310</span> * @param r region store belongs to<a name="line.310"></a> +<span class="sourceLineNo">311</span> * @param s Store to request compaction on<a name="line.311"></a> +<span class="sourceLineNo">312</span> * @param why Why compaction requested -- used in debug messages<a name="line.312"></a> +<span class="sourceLineNo">313</span> * @param priority override the default priority (NO_PRIORITY == decide)<a name="line.313"></a> +<span class="sourceLineNo">314</span> * @param request custom compaction request. Can be <tt>null</tt> in which case a simple<a name="line.314"></a> +<span class="sourceLineNo">315</span> * compaction will be used.<a name="line.315"></a> +<span class="sourceLineNo">316</span> */<a name="line.316"></a> +<span class="sourceLineNo">317</span> private synchronized CompactionRequest requestCompactionInternal(final Region r, final Store s,<a name="line.317"></a> +<span class="sourceLineNo">318</span> final String why, int priority, CompactionRequest request, boolean selectNow, User user)<a name="line.318"></a> +<span class="sourceLineNo">319</span> throws IOException {<a name="line.319"></a> +<span class="sourceLineNo">320</span> if (this.server.isStopped()<a name="line.320"></a> +<span class="sourceLineNo">321</span> || (r.getTableDescriptor() != null && !r.getTableDescriptor().isCompactionEnabled())) {<a name="line.321"></a> +<span class="sourceLineNo">322</span> return null;<a name="line.322"></a> +<span class="sourceLineNo">323</span> }<a name="line.323"></a> +<span class="sourceLineNo">324</span><a name="line.324"></a> +<span class="sourceLineNo">325</span> CompactionContext compaction = null;<a name="line.325"></a> +<span class="sourceLineNo">326</span> if (selectNow) {<a name="line.326"></a> +<span class="sourceLineNo">327</span> compaction = selectCompaction(r, s, priority, request, user);<a name="line.327"></a> +<span class="sourceLineNo">328</span> if (compaction == null) return null; // message logged inside<a name="line.328"></a> +<span class="sourceLineNo">329</span> }<a name="line.329"></a> +<span class="sourceLineNo">330</span><a name="line.330"></a> +<span class="sourceLineNo">331</span> final RegionServerSpaceQuotaManager spaceQuotaManager =<a name="line.331"></a> +<span class="sourceLineNo">332</span> this.server.getRegionServerSpaceQuotaManager();<a name="line.332"></a> +<span class="sourceLineNo">333</span> if (spaceQuotaManager != null && spaceQuotaManager.areCompactionsDisabled(<a name="line.333"></a> +<span class="sourceLineNo">334</span> r.getTableDescriptor().getTableName())) {<a name="line.334"></a> +<span class="sourceLineNo">335</span> if (LOG.isDebugEnabled()) {<a name="line.335"></a> +<span class="sourceLineNo">336</span> LOG.debug("Ignoring compaction request for " + r + " as an active space quota violation "<a name="line.336"></a> +<span class="sourceLineNo">337</span> + " policy disallows compactions.");<a name="line.337"></a> +<span class="sourceLineNo">338</span> }<a name="line.338"></a> +<span class="sourceLineNo">339</span> return null;<a name="line.339"></a> +<span class="sourceLineNo">340</span> }<a name="line.340"></a> +<span class="sourceLineNo">341</span><a name="line.341"></a> +<span class="sourceLineNo">342</span> // We assume that most compactions are small. So, put system compactions into small<a name="line.342"></a> +<span class="sourceLineNo">343</span> // pool; we will do selection there, and move to large pool if necessary.<a name="line.343"></a> +<span class="sourceLineNo">344</span> ThreadPoolExecutor pool = (selectNow && s.throttleCompaction(compaction.getRequest().getSize()))<a name="line.344"></a> +<span class="sourceLineNo">345</span> ? longCompactions : shortCompactions;<a name="line.345"></a> +<span class="sourceLineNo">346</span> pool.execute(new CompactionRunner(s, r, compaction, pool, user));<a name="line.346"></a> +<span class="sourceLineNo">347</span> if (LOG.isDebugEnabled()) {<a name="line.347"></a> +<span class="sourceLineNo">348</span> String type = (pool == shortCompactions) ? "Small " : "Large ";<a name="line.348"></a> +<span class="sourceLineNo">349</span> LOG.debug(type + "Compaction requested: " + (selectNow ? compaction.toString() : "system")<a name="line.349"></a> +<span class="sourceLineNo">350</span> + (why != null && !why.isEmpty() ? "; Because: " + why : "") + "; " + this);<a name="line.350"></a> +<span class="sourceLineNo">351</span> }<a name="line.351"></a> +<span class="sourceLineNo">352</span> return selectNow ? compaction.getRequest() : null;<a name="line.352"></a> +<span class="sourceLineNo">353</span> }<a name="line.353"></a> +<span class="sourceLineNo">354</span><a name="line.354"></a> +<span class="sourceLineNo">355</span> private CompactionContext selectCompaction(final Region r, final Store s,<a name="line.355"></a> +<span class="sourceLineNo">356</span> int priority, CompactionRequest request, User user) throws IOException {<a name="line.356"></a> +<span class="sourceLineNo">357</span> CompactionContext compaction = s.requestCompaction(priority, request, user);<a name="line.357"></a> +<span class="sourceLineNo">358</span> if (compaction == null) {<a name="line.358"></a> +<span class="sourceLineNo">359</span> if(LOG.isDebugEnabled() && r.getRegionInfo() != null) {<a name="line.359"></a> +<span class="sourceLineNo">360</span> LOG.debug("Not compacting " + r.getRegionInfo().getRegionNameAsString() +<a name="line.360"></a> +<span class="sourceLineNo">361</span> " because compaction request was cancelled");<a name="line.361"></a> +<span class="sourceLineNo">362</span> }<a name="line.362"></a> +<span class="sourceLineNo">363</span> return null;<a name="line.363"></a> +<span class="sourceLineNo">364</span> }<a name="line.364"></a> +<span class="sourceLineNo">365</span> assert compaction.hasSelection();<a name="line.365"></a> +<span class="sourceLineNo">366</span> if (priority != Store.NO_PRIORITY) {<a name="line.366"></a> +<span class="sourceLineNo">367</span> compaction.getRequest().setPriority(priority);<a name="line.367"></a> +<span class="sourceLineNo">368</span> }<a name="line.368"></a> +<span class="sourceLineNo">369</span> return compaction;<a name="line.369"></a> +<span class="sourceLineNo">370</span> }<a name="line.370"></a> +<span class="sourceLineNo">371</span><a name="line.371"></a> +<span class="sourceLineNo">372</span> /**<a name="line.372"></a> +<span class="sourceLineNo">373</span> * Only interrupt once it's done with a run through the work loop.<a name="line.373"></a> +<span class="sourceLineNo">374</span> */<a name="line.374"></a> +<span class="sourceLineNo">375</span> void interruptIfNecessary() {<a name="line.375"></a> +<span class="sourceLineNo">376</span> splits.shutdown();<a name="line.376"></a> +<span class="sourceLineNo">377</span> longCompactions.shutdown();<a name="line.377"></a> +<span class="sourceLineNo">378</span> shortCompactions.shutdown();<a name="line.378"></a> +<span class="sourceLineNo">379</span> }<a name="line.379"></a> +<span class="sourceLineNo">380</span><a name="line.380"></a> +<span class="sourceLineNo">381</span> private void waitFor(ThreadPoolExecutor t, String name) {<a name="line.381"></a> +<span class="sourceLineNo">382</span> boolean done = false;<a name="line.382"></a> +<span class="sourceLineNo">383</span> while (!done) {<a name="line.383"></a> +<span class="sourceLineNo">384</span> try {<a name="line.384"></a> +<span class="sourceLineNo">385</span> done = t.awaitTermination(60, TimeUnit.SECONDS);<a name="line.385"></a> +<span class="sourceLineNo">386</span> LOG.info("Waiting for " + name + " to finish...");<a name="line.386"></a> +<span class="sourceLineNo">387</span> if (!done) {<a name="line.387"></a> +<span class="sourceLineNo">388</span> t.shutdownNow();<a name="line.388"></a> +<span class="sourceLineNo">389</span> }<a name="line.389"></a> +<span class="sourceLineNo">390</span> } catch (InterruptedException ie) {<a name="line.390"></a> +<span class="sourceLineNo">391</span> LOG.warn("Interrupted waiting for " + name + " to finish...");<a name="line.391"></a> +<span class="sourceLineNo">392</span> }<a name="line.392"></a> +<span class="sourceLineNo">393</span> }<a name="line.393"></a> +<span class="sourceLineNo">394</span> }<a name="line.394"></a> +<span class="sourceLineNo">395</span><a name="line.395"></a> +<span class="sourceLineNo">396</span> void join() {<a name="line.396"></a> +<span class="sourceLineNo">397</span> waitFor(splits, "Split Thread");<a name="line.397"></a> +<span class="sourceLineNo">398</span> waitFor(longCompactions, "Large Compaction Thread");<a name="line.398"></a> +<span class="sourceLineNo">399</span> waitFor(shortCompactions, "Small Compaction Thread");<a name="line.399"></a> +<span class="sourceLineNo">400</span> }<a name="line.400"></a> +<span class="sourceLineNo">401</span><a name="line.401"></a> +<span class="sourceLineNo">402</span> /**<a name="line.402"></a> +<span class="sourceLineNo">403</span> * Returns the current size of the queue containing regions that are<a name="line.403"></a> +<span class="sourceLineNo">404</span> * processed.<a name="line.404"></a> +<span class="sourceLineNo">405</span> *<a name="line.405"></a> +<span class="sourceLineNo">406</span> * @return The current size of the regions queue.<a name="line.406"></a> +<span class="sourceLineNo">407</span> */<a name="line.407"></a> +<span class="sourceLineNo">408</span> public int getCompactionQueueSize() {<a name="line.408"></a> +<span class="sourceLineNo">409</span> return longCompactions.getQueue().size() + shortCompactions.getQueue().size();<a name="line.409"></a> +<span class="sourceLineNo">410</span> }<a name="line.410"></a> +<span class="sourceLineNo">411</span><a name="line.411"></a> +<span class="sourceLineNo">412</span> public int getLargeCompactionQueueSize() {<a name="line.412"></a> +<span class="sourceLineNo">413</span> return longCompactions.getQueue().size();<a name="line.413"></a> <span class="sourceLineNo">414</span> }<a name="line.414"></a> <span class="sourceLineNo">415</span><a name="line.415"></a> -<span class="sourceLineNo">416</span> /**<a name="line.416"></a> -<span class="sourceLineNo">417</span> * Returns the current size of the queue containing regions that are<a name="line.417"></a> -<span class="sourceLineNo">418</span> * processed.<a name="line.418"></a> -<span class="sourceLineNo">419</span> *<a name="line.419"></a> -<span class="sourceLineNo">420</span> * @return The current size of the regions queue.<a name="line.420"></a> -<span class="sourceLineNo">421</span> */<a name="line.421"></a> -<span class="sourceLineNo">422</span> public int getCompactionQueueSize() {<a name="line.422"></a> -<span class="sourceLineNo">423</span> return longCompactions.getQueue().size() + shortCompactions.getQueue().size();<a name="line.423"></a> -<span class="sourceLineNo">424</span> }<a name="line.424"></a> -<span class="sourceLineNo">425</span><a name="line.425"></a> -<span class="sourceLineNo">426</span> public int getLargeCompactionQueueSize() {<a name="line.426"></a> -<span class="sourceLineNo">427</span> return longCompactions.getQueue().size();<a name="line.427"></a> -<span class="sourceLineNo">428</span> }<a name="line.428"></a> -<span class="sourceLineNo">429</span><a name="line.429"></a> -<span class="sourceLineNo">430</span><a name="line.430"></a> -<span class="sourceLineNo">431</span> public int getSmallCompactionQueueSize() {<a name="line.431"></a> -<span class="sourceLineNo">432</span> return shortCompactions.getQueue().size();<a name="line.432"></a> -<span class="sourceLineNo">433</span> }<a name="line.433"></a> -<span class="sourceLineNo">434</span><a name="line.434"></a> -<span class="sourceLineNo">435</span> public int getSplitQueueSize() {<a name="line.435"></a> -<span class="sourceLineNo">436</span> return splits.getQueue().size();<a name="line.436"></a> -<span class="sourceLineNo">437</span> }<a name="line.437"></a> -<span class="sourceLineNo">438</span><a name="line.438"></a> -<span class="sourceLineNo">439</span> private boolean shouldSplitRegion() {<a name="line.439"></a> -<span class="sourceLineNo">440</span> if(server.getNumberOfOnlineRegions() > 0.9*regionSplitLimit) {<a name="line.440"></a> -<span class="sourceLineNo">441</span> LOG.warn("Total number of regions is approaching the upper limit " + regionSplitLimit + ". "<a name="line.441"></a> -<span class="sourceLineNo">442</span> + "Please consider taking a look at http://hbase.apache.org/book.html#ops.regionmgt");<a name="line.442"></a> -<span class="sourceLineNo">443</span> }<a name="line.443"></a> -<span class="sourceLineNo">444</span> return (regionSplitLimit > server.getNumberOfOnlineRegions());<a name="line.444"></a> -<span class="sourceLineNo">445</span> }<a name="line.445"></a> -<span class="sourceLineNo">446</span><a name="line.446"></a> -<span class="sourceLineNo">447</span> /**<a name="line.447"></a> -<span class="sourceLineNo">448</span> * @return the regionSplitLimit<a name="line.448"></a> -<span class="sourceLineNo">449</span> */<a name="line.449"></a> -<span class="sourceLineNo">450</span> public int getRegionSplitLimit() {<a name="line.450"></a> -<span class="sourceLineNo">451</span> return this.regionSplitLimit;<a name="line.451"></a> -<span class="sourceLineNo">452</span> }<a name="line.452"></a> -<span class="sourceLineNo">453</span><a name="line.453"></a> -<span class="sourceLineNo">454</span> @edu.umd.cs.findbugs.annotations.SuppressWarnings(value="EQ_COMPARETO_USE_OBJECT_EQUALS",<a name="line.454"></a> -<span class="sourceLineNo">455</span> justification="Contrived use of compareTo")<a name="line.455"></a> -<span class="sourceLineNo">456</span> private class CompactionRunner implements Runnable, Comparable<CompactionRunner> {<a name="line.456"></a> -<span class="sourceLineNo">457</span> private final Store store;<a name="line.457"></a> -<span class="sourceLineNo">458</span> private final HRegion region;<a name="line.458"></a> -<span class="sourceLineNo">459</span> private CompactionContext compaction;<a name="line.459"></a> -<span class="sourceLineNo">460</span> private int queuedPriority;<a name="line.460"></a> -<span class="sourceLineNo">461</span> private ThreadPoolExecutor parent;<a name="line.461"></a> -<span class="sourceLineNo">462</span> private User user;<a name="line.462"></a> -<span class="sourceLineNo">463</span> private long time;<a name="line.463"></a> -<span class="sourceLineNo">464</span><a name="line.464"></a> -<span class="sourceLineNo">465</span> public CompactionRunner(Store store, Region region,<a name="line.465"></a> -<span class="sourceLineNo">466</span> CompactionContext compaction, ThreadPoolExecutor parent, User user) {<a name="line.466"></a> -<span class="sourceLineNo">467</span> super();<a name="line.467"></a> -<span class="sourceLineNo">468</span> this.store = store;<a name="line.468"></a> -<span class="sourceLineNo">469</span> this.region = (HRegion)region;<a name="line.469"></a> -<span class="sourceLineNo">470</span> this.compaction = compaction;<a name="line.470"></a> -<span class="sourceLineNo">471</span> this.queuedPriority = (this.compaction == null)<a name="line.471"></a> -<span class="sourceLineNo">472</span> ? store.getCompactPriority() : compaction.getRequest().getPriority();<a name="line.472"></a> -<span class="sourceLineNo">473</span> this.parent = parent;<a name="line.473"></a> -<span class="sourceLineNo">474</span> this.user = user;<a name="line.474"></a> -<span class="sourceLineNo">475</span> this.time = System.currentTimeMillis();<a name="line.475"></a> -<span class="sourceLineNo">476</span> }<a name="line.476"></a> -<span class="sourceLineNo">477</span><a name="line.477"></a> -<span class="sourceLineNo">478</span> @Override<a name="line.478"></a> -<span class="sourceLineNo">479</span> public String toString() {<a name="line.479"></a> -<span class="sourceLineNo">480</span> return (this.compaction != null) ? ("Request = " + compaction.getRequest())<a name="line.480"></a> -<span class="sourceLineNo">481</span> : ("regionName = " + region.toString() + ", storeName = " + store.toString() +<a name="line.481"></a> -<span class="sourceLineNo">482</span> ", priority = " + queuedPriority + ", time = " + time);<a name="line.482"></a> -<span class="sourceLineNo">483</span> }<a name="line.483"></a> -<span class="sourceLineNo">484</span><a name="line.484"></a> -<span class="sourceLineNo">485</span> private void doCompaction(User user) {<a name="line.485"></a> -<span class="sourceLineNo">486</span> // Common case - system compaction without a file selection. Select now.<a name="line.486"></a> -<span class="sourceLineNo">487</span> if (this.compaction == null) {<a name="line.487"></a> -<span class="sourceLineNo">488</span> int oldPriority = this.queuedPriority;<a name="line.488"></a> -<span class="sourceLineNo">489</span> this.queuedPriority = this.store.getCompactPriority();<a name="line.489"></a> -<span class="sourceLineNo">490</span> if (this.queuedPriority > oldPriority) {<a name="line.490"></a> -<span class="sourceLineNo">491</span> // Store priority decreased while we were in queue (due to some other compaction?),<a name="line.491"></a> -<span class="sourceLineNo">492</span> // requeue with new priority to avoid blocking potential higher priorities.<a name="line.492"></a> -<span class="sourceLineNo">493</span> this.parent.execute(this);<a name="line.493"></a> -<span class="sourceLineNo">494</span> return;<a name="line.494"></a> -<span class="sourceLineNo">495</span> }<a name="line.495"></a> -<span class="sourceLineNo">496</span> try {<a name="line.496"></a> -<span class="sourceLineNo">497</span> this.compaction = selectCompaction(this.region, this.store, queuedPriority, null, user);<a name="line.497"></a> -<span class="sourceLineNo">498</span> } catch (IOException ex) {<a name="line.498"></a> -<span class="sourceLineNo">499</span> LOG.error("Compaction selection failed " + this, ex);<a name="line.499"></a> -<span class="sourceLineNo">500</span> server.checkFileSystem();<a name="line.500"></a> -<span class="sourceLineNo">501</span> return;<a name="line.501"></a> -<span class="sourceLineNo">502</span> }<a name="line.502"></a> -<span class="sourceLineNo">503</span> if (this.compaction == null) return; // nothing to do<a name="line.503"></a> -<span class="sourceLineNo">504</span> // Now see if we are in correct pool for the size; if not, go to the correct one.<a name="line.504"></a> -<span class="sourceLineNo">505</span> // We might end up waiting for a while, so cancel the selection.<a name="line.505"></a> -<span class="sourceLineNo">506</span> assert this.compaction.hasSelection();<a name="line.506"></a> -<span class="sourceLineNo">507</span> ThreadPoolExecutor pool = store.throttleCompaction(<a name="line.507"></a> -<span class="sourceLineNo">508</span> compaction.getRequest().getSize()) ? longCompactions : shortCompactions;<a name="line.508"></a> -<span class="sourceLineNo">509</span><a name="line.509"></a> -<span class="sourceLineNo">510</span> // Long compaction pool can process small job<a name="line.510"></a> -<span class="sourceLineNo">511</span> // Short compaction pool should not process large job<a name="line.511"></a> -<span class="sourceLineNo">512</span> if (this.parent == shortCompactions && pool == longCompactions) {<a name="line.512"></a> -<span class="sourceLineNo">513</span> this.store.cancelRequestedCompaction(this.compaction);<a name="line.513"></a> -<span class="sourceLineNo">514</span> this.compaction = null;<a name="line.514"></a> -<span class="sourceLineNo">515</span> this.parent = pool;<a name="line.515"></a> -<span class="sourceLineNo">516</span> this.parent.execute(this);<a name="line.516"></a> -<span class="sourceLineNo">517</span> return;<a name="line.517"></a> -<span class="sourceLineNo">518</span> }<a name="line.518"></a> -<span class="sourceLineNo">519</span> }<a name="line.519"></a> -<span class="sourceLineNo">520</span> // Finally we can compact something.<a name="line.520"></a> -<span class="sourceLineNo">521</span> assert this.compaction != null;<a name="line.521"></a> -<span class="sourceLineNo">522</span><a name="line.522"></a> -<span class="sourceLineNo">523</span> this.compaction.getRequest().beforeExecute();<a name="line.523"></a> -<span class="sourceLineNo">524</span> try {<a name="line.524"></a> -<span class="sourceLineNo">525</span> // Note: please don't put single-compaction logic here;<a name="line.525"></a> -<span class="sourceLineNo">526</span> // put it into region/store/etc. This is CST logic.<a name="line.526"></a> -<span class="sourceLineNo">527</span> long start = EnvironmentEdgeManager.currentTime();<a name="line.527"></a> -<span class="sourceLineNo">528</span> boolean completed =<a name="line.528"></a> -<span class="sourceLineNo">529</span> region.compact(compaction, store, compactionThroughputController, user);<a name="line.529"></a> -<span class="sourceLineNo">530</span> long now = EnvironmentEdgeManager.currentTime();<a name="line.530"></a> -<span class="sourceLineNo">531</span> LOG.info(((completed) ? "Completed" : "Aborted") + " compaction: " +<a name="line.531"></a> -<span class="sourceLineNo">532</span> this + "; duration=" + StringUtils.formatTimeDiff(now, start));<a name="line.532"></a> -<span class="sourceLineNo">533</span> if (completed) {<a name="line.533"></a> -<span class="sourceLineNo">534</span> // degenerate case: blocked regions require recursive enqueues<a name="line.534"></a> -<span class="sourceLineNo">535</span> if (store.getCompactPriority() <= 0) {<a name="line.535"></a> -<span class="sourceLineNo">536</span> requestSystemCompaction(region, store, "Recursive enqueue");<a name="line.536"></a> -<span class="sourceLineNo">537</span> } else {<a name="line.537"></a> -<span class="sourceLineNo">538</span> // see if the compaction has caused us to exceed max region size<a name="line.538"></a> -<span class="sourceLineNo">539</span> requestSplit(region);<a name="line.539"></a> -<span class="sourceLineNo">540</span> }<a name="line.540"></a> -<span class="sourceLineNo">541</span> }<a name="line.541"></a> -<span class="sourceLineNo">542</span> } catch (IOException ex) {<a name="line.542"></a> -<span class="sourceLineNo">543</span> IOException remoteEx =<a name="line.543"></a> -<span class="sourceLineNo">544</span> ex instanceof RemoteException ? ((RemoteException) ex).unwrapRemoteException() : ex;<a name="line.544"></a> -<span class="sourceLineNo">545</span> LOG.error("Compaction failed " + this, remoteEx);<a name="line.545"></a> -<span class="sourceLineNo">546</span> if (remoteEx != ex) {<a name="line.546"></a> -<span class="sourceLineNo">547</span> LOG.info("Compaction failed at original callstack: " + formatStackTrace(ex));<a name="line.547"></a> -<span class="sourceLineNo">548</span> }<a name="line.548"></a> -<span class="sourceLineNo">549</span> region.reportCompactionRequestFailure();<a name="line.549"></a> -<span class="sourceLineNo">550</span> server.checkFileSystem();<a name="line.550"></a> -<span class="sourceLineNo">551</span> } catch (Exception ex) {<a name="line.551"></a> -<span class="sourceLineNo">552</span> LOG.error("Compaction failed " + this, ex);<a name="line.552"></a> -<span class="sourceLineNo">553</span> region.reportCompactionRequestFailure();<a name="line.553"></a> -<span class="sourceLineNo">554</span> server.checkFileSystem();<a name="line.554"></a> -<span class="sourceLineNo">555</span> } finally {<a name="line.555"></a> -<span class="sourceLineNo">556</span> LOG.debug("CompactSplitThread Status: " + CompactSplit.this);<a name="line.556"></a> -<span class="sourceLineNo">557</span> }<a name="line.557"></a> -<span class="sourceLineNo">558</span> this.compaction.getRequest().afterExecute();<a name="line.558"></a> -<span class="sourceLineNo">559</span> }<a name="line.559"></a> -<span class="sourceLineNo">560</span><a name="line.560"></a> -<span class="sourceLineNo">561</span> @Override<a name="line.561"></a> -<span class="sourceLineNo">562</span> public void run() {<a name="line.562"></a> -<span class="sourceLineNo">563</span> Preconditions.checkNotNull(server);<a name="line.563"></a> -<span class="sourceLineNo">564</span> if (server.isStopped()<a name="line.564"></a> -<span class="sourceLineNo">565</span> || (region.getTableDescriptor() != null && !region.getTableDescriptor().isCompactionEnabled())) {<a name="line.565"></a> -<span class="sourceLineNo">566</span> return;<a name="line.566"></a> -<span class="sourceLineNo">567</span> }<a name="line.567"></a> -<span class="sourceLineNo">568</span> doCompaction(user);<a name="line.568"></a> -<span class="sourceLineNo">569</span> }<a name="line.569"></a> -<span class="sourceLineNo">570</span><a name="line.570"></a> -<span class="sourceLineNo">571</span> private String formatStackTrace(Exception ex) {<a name="line.571"></a> -<span class="sourceLineNo">572</span> StringWriter sw = new StringWriter();<a name="line.572"></a> -<span class="sourceLineNo">573</span> PrintWriter pw = new PrintWriter(sw);<a name="line.573"></a> -<span class="sourceLineNo">574</span> ex.printStackTrace(pw);<a name="line.574"></a> -<span class="sourceLineNo">575</span> pw.flush();<a name="line.575"></a> -<span class="sourceLineNo">576</span> return sw.toString();<a name="line.576"></a> -<span class="sourceLineNo">577</span> }<a name="line.577"></a> -<span class="sourceLineNo">578</span><a name="line.578"></a> -<span class="sourceLineNo">579</span> @Override<a name="line.579"></a> -<span class="sourceLineNo">580</span> public int compareTo(CompactionRunner o) {<a name="line.580"></a> -<span class="sourceLineNo">581</span> // Only compare the underlying request (if any), for queue sorting purposes.<a name="line.581"></a> -<span class="sourceLineNo">582</span> int compareVal = queuedPriority - o.queuedPriority; // compare priority<a name="line.582"></a> -<span class="sourceLineNo">583</span> if (compareVal != 0) return compareVal;<a name="line.583"></a> -<span class="sourceLineNo">584</span> CompactionContext tc = this.compaction, oc = o.compaction;<a name="line.584"></a> -<span class="sourceLineNo">585</span> // Sort pre-selected (user?) compactions before system ones with equal priority.<a name="line.585"></a> -<span class="sourceLineNo">586</span> return (tc == null) ? ((oc == null) ? 0 : 1)<a name="line.586"></a> -<span class="sourceLineNo">587</span> : ((oc == null) ? -1 : tc.getRequest().compareTo(oc.getRequest()));<a name="line.587"></a> +<span class="sourceLineNo">416</span><a name="line.416"></a> +<span class="sourceLineNo">417</span> public int getSmallCompactionQueueSize() {<a name="line.417"></a> +<span class="sourceLineNo">418</span> return shortCompactions.getQueue().size();<a name="line.418"></a> +<span class="sourceLineNo">419</span> }<a name="line.419"></a> +<span class="sourceLineNo">420</span><a name="line.420"></a> +<span class="sourceLineNo">421</span> public int getSplitQueueSize() {<a name="line.421"></a> +<span class="sourceLineNo">422</span> return splits.getQueue().size();<a name="line.422"></a> +<span class="sourceLineNo">423</span> }<a name="line.423"></a> +<span class="sourceLineNo">424</span><a name="line.424"></a> +<span class="sourceLineNo">425</span> private boolean shouldSplitRegion() {<a name="line.425"></a> +<span class="sourceLineNo">426</span> if(server.getNumberOfOnlineRegions() > 0.9*regionSplitLimit) {<a name="line.426"></a> +<span class="sourceLineNo">427</span> LOG.warn("Total number of regions is approaching the upper limit " + regionSplitLimit + ". "<a name="line.427"></a> +<span class="sourceLineNo">428</span> + "Please consider taking a look at http://hbase.apache.org/book.html#ops.regionmgt");<a name="line.428"></a> +<span class="sourceLineNo">429</span> }<a name="line.429"></a> +<span class="sourceLineNo">430</span> return (regionSplitLimit > server.getNumberOfOnlineRegions());<a name="line.430"></a> +<span class="sourceLineNo">431</span> }<a name="line.431"></a> +<span class="sourceLineNo">432</span><a name="line.432"></a> +<span class="sourceLineNo">433</span> /**<a name="line.433"></a> +<span class="sourceLineNo">434</span> * @return the regionSplitLimit<a name="line.434"></a> +<span class="sourceLineNo">435</span> */<a name="line.435"></a> +<span class="sourceLineNo">436</span> public int getRegionSplitLimit() {<a name="line.436"></a> +<span class="sourceLineNo">437</span> return this.regionSplitLimit;<a name="line.437"></a> +<span class="sourceLineNo">438</span> }<a name="line.438"></a> +<span class="sourceLineNo">439</span><a name="line.439"></a> +<span class="sourceLineNo">440</span> @edu.umd.cs.findbugs.annotations.SuppressWarnings(value="EQ_COMPARETO_USE_OBJECT_EQUALS",<a name="line.440"></a> +<span class="sourceLineNo">441</span> justification="Contrived use of compareTo")<a name="line.441"></a> +<span class="sourceLineNo">442</span> private class CompactionRunner implements Runnable, Comparable<CompactionRunner> {<a name="line.442"></a> +<span class="sourceLineNo">443</span> private final Store store;<a name="line.443"></a> +<span class="sourceLineNo">444</span> private final HRegion region;<a name="line.444"></a> +<span class="sourceLineNo">445</span> private CompactionContext compaction;<a name="line.445"></a> +<span class="sourceLineNo">446</span> private int queuedPriority;<a name="line.446"></a> +<span class="sourceLineNo">447</span> private ThreadPoolExecutor parent;<a name="line.447"></a> +<span class="sourceLineNo">448</span> private User user;<a name="line.448"></a> +<span class="sourceLineNo">449</span> private long time;<a name="line.449"></a> +<span class="sourceLineNo">450</span><a name="line.450"></a> +<span class="sourceLineNo">451</span> public CompactionRunner(Store store, Region region,<a name="line.451"></a> +<span class="sourceLineNo">452</span> CompactionContext compaction, ThreadPoolExecutor parent, User user) {<a name="line.452"></a> +<span class="sourceLineNo">453</span> super();<a name="line.453"></a> +<span class="sourceLineNo">454</span> this.store = store;<a name="line.454"></a> +<span class="sourceLineNo">455</span> this.region = (HRegion)region;<a name="line.455"></a> +<span class="sourceLineNo">456</span> this.compaction = compaction;<a name="line.456"></a> +<span class="sourceLineNo">457</span> this.queuedPriority = (this.compaction == null)<a name="line.457"></a> +<span class="sourceLineNo">458</span> ? store.getCompactPriority() : compaction.getRequest().getPriority();<a name="line.458"></a> +<span class="sourceLineNo">459</span> this.parent = parent;<a name="line.459"></a> +<span class="sourceLineNo">460</span> this.user = user;<a name="line.460"></a> +<span class="sourceLineNo">461</span> this.time = System.currentTimeMillis();<a name="line.461"></a> +<span class="sourceLineNo">462</span> }<a name="line.462"></a> +<span class="sourceLineNo">463</span><a name="line.463"></a> +<span class="sourceLineNo">464</span> @Override<a name="line.464"></a> +<span class="sourceLineNo">465</span> public String toString() {<a name="line.465"></a> +<span class="sourceLineNo">466</span> return (this.compaction != null) ? ("Request = " + compaction.getRequest())<a name="line.466"></a> +<span class="sourceLineNo">467</span> : ("regionName = " + region.toString() + ", storeName = " + store.toString() +<a name="line.467"></a> +<span class="sourceLineNo">468</span> ", priority = " + queuedPriority + ", time = " + time);<a name="line.468"></a> +<span class="sourceLineNo">469</span> }<a name="line.469"></a> +<span class="sourceLineNo">470</span><a name="line.470"></a> +<span class="sourceLineNo">471</span> private void doCompaction(User user) {<a name="line.471"></a> +<span class="sourceLineNo">472</span> // Common case - system compaction without a file selection. Select now.<a name="line.472"></a> +<span class="sourceLineNo">473</span> if (this.compaction == null) {<a name="line.473"></a> +<span class="sourceLineNo">474</span> int oldPriority = this.queuedPriority;<a name="line.474"></a> +<span class="sourceLineNo">475</span> this.queuedPriority = this.store.getCompactPriority();<a name="line.475"></a> +<span class="sourceLineNo">476</span> if (this.queuedPriority > oldPriority) {<a name="line.476"></a> +<span class="sourceLineNo">477</span> // Store priority decreased while we were in queue (due to some other compaction?),<a name="line.477"></a> +<span class="sourceLineNo">478</span> // requeue with new priority to avoid blocking potential higher priorities.<a name="line.478"></a> +<span class="sourceLineNo">479</span> this.parent.execute(this);<a name="line.479"></a> +<span class="sourceLineNo">480</span> return;<a name="line.480"></a> +<span class="sourceLineNo">481</span> }<a name="line.481"></a> +<span class="sourceLineNo">482</span> try {<a name="line.482"></a> +<span class="sourceLineNo">483</span> this.compaction = selectCompaction(this.region, this.store, queuedPriority, null, user);<a name="line.483"></a> +<span class="sourceLineNo">484</span> } catch (IOException ex) {<a name="line.484"></a> +<span class="sourceLineNo">485</span> LOG.error("Compaction selection failed " + this, ex);<a name="line.485"></a> +<span class="sourceLineNo">486</span> server.checkFileSystem();<a name="line.486"></a> +<span class="sourceLineNo">487</span> return;<a name="line.487"></a> +<span class="sourceLineNo">488</span> }<a name="line.488"></a> +<span class="sourceLineNo">489</span> if (this.compaction == null) return; // nothing to do<a name="line.489"></a> +<span class="sourceLineNo">490</span> // Now see if we are in correct pool for the size; if not, go to the correct one.<a name="line.490"></a> +<span class="sourceLineNo">491</span> // We might end up waiting for a while, so cancel the selection.<a name="line.491"></a> +<span class="sourceLineNo">492</span> assert this.compaction.hasSelection();<a name="line.492"></a> +<span class="sourceLineNo">493</span> ThreadPoolExecutor pool = store.throttleCompaction(<a name="line.493"></a> +<span class="sourceLineNo">494</span> compaction.getRequest().getSize()) ? longCompactions : shortCompactions;<a name="line.494"></a> +<span class="sourceLineNo">495</span><a name="line.495"></a> +<span class="sourceLineNo">496</span> // Long compaction pool can process small job<a name="line.496"></a> +<span class="sourceLineNo">497</span> // Short compaction pool should not process large job<a name="line.497"></a> +<span class="sourceLineNo">498</span> if (this.parent == shortCompactions && pool == longCompactions) {<a name="line.498"></a> +<span class="sourceLineNo">499</span> this.store.cancelRequestedCompaction(this.compaction);<a name="line.499"></a> +<span class="sourceLineNo">500</span> this.compaction = null;<a name="line.500"></a> +<span class="sourceLineNo">501</span> this.parent = pool;<a name="line.501"></a> +<span class="sourceLineNo">502</span> this.parent.execute(this);<a name="line.502"></a> +<span class="sourceLineNo">503</span> return;<a name="line.503"></a> +<span class="sourceLineNo">504</span> }<a name="line.504"></a> +<span class="sourceLineNo">505</span> }<a name="line.505"></a> +<span class="sourceLineNo">506</span> // Finally we can compact something.<a name="line.506"></a> +<span class="sourceLineNo">507</span> assert this.compaction != null;<a name="line.507"></a> +<span class="sourceLineNo">508</span><a name="line.508"></a> +<span class="sourceLineNo">509</span> this.compaction.getRequest().beforeExecute();<a name="line.509"></a> +<span class="sourceLineNo">510</span> try {<a name="line.510"></a> +<span class="sourceLineNo">511</span> // Note: please don't put single-compaction logic here;<a name="line.511"></a> +<span class="sourceLineNo">512</span> // put it into region/store/etc. This is CST logic.<a name="line.512"></a> +<span class="sourceLineNo">513</span> long start = EnvironmentEdgeManager.currentTime();<a name="line.513"></a> +<span class="sourceLineNo">514</span> boolean completed =<a name="line.514"></a> +<span class="sourceLineNo">515</span> region.compact(compaction, store, compactionThroughputController, user);<a name="line.515"></a> +<span class="sourceLineNo">516</span> long now = EnvironmentEdgeManager.currentTime();<a name="line.516"></a> +<span class="sourceLineNo">517</span> LOG.info(((completed) ? "Completed" : "Aborted") + " compaction: " +<a name="line.517"></a> +<span class="sourceLineNo">518</span> this + "; duration=" + StringUtils.formatTimeDiff(now, start));<a name="line.518"></a> +<span class="sourceLineNo">519</span> if (completed) {<a name="line.519"></a> +<span class="sourceLineNo">520</span> // degenerate case: blocked regions require recursive enqueues<a name="line.520"></a> +<span class="sourceLineNo">521</span> if (store.getCompactPriority() <= 0) {<a name="line.521"></a> +<span class="sourceLineNo">522</span> requestSystemCompaction(region, store, "Recursive enqueue");<a name="line.522"></a> +<span class="sourceLineNo">523</span> } else {<a name="line.523"></a> +<span class="sourceLineNo">524</span> // see if the compaction has caused us to exceed max region size<a name="line.524"></a> +<span class="sourceLineNo">525</span> requestSplit(region);<a name="line.525"></a> +<span class="sourceLineNo">526</span> }<a name="line.526"></a> +<span class="sourceLineNo">527</span> }<a name="line.527"></a> +<span class="sourceLineNo">528</span> } catch (IOException ex) {<a name="line.528"></a> +<span class="sourceLineNo">529</span> IOException remoteEx =<a name="line.529"></a> +<span class="sourceLineNo">530</span> ex instanceof RemoteException ? ((RemoteException) ex).unwrapRemoteException() : ex;<a name="line.530"></a> +<span class="sourceLineNo">531</span> LOG.error("Compaction failed " + this, remoteEx);<a name="line.531"></a> +<span class="sourceLineNo">532</span> if (remoteEx != ex) {<a name="line.532"></a> +<span class="sourceLineNo">533</span> LOG.info("Compaction failed at original callstack: " + formatStackTrace(ex));<a name="line.533"></a> +<span class="sourceLineNo">534</span> }<a name="line.534"></a> +<span class="sourceLineNo">535</span> region.reportCompactionRequestFailure();<a name="line.535"></a> +<span class="sourceLineNo">536</span> server.checkFileSystem();<a name="line.536"></a> +<span class="sourceLineNo">537</span> } catch (Exception ex) {<a name="line.537"></a> +<span class="sourceLineNo">538</span> LOG.error("Compaction failed " + this, ex);<a name="line.538"></a> +<span class="sourceLineNo">539</span> region.reportCompactionRequestFailure();<a name="line.539"></a> +<span class="sourceLineNo">540</span> server.checkFileSystem();<a name="line.540"></a> +<span class="sourceLineNo">541</span> } finally {<a name="line.541"></a> +<span class="sourceLineNo">542</span> LOG.debug("CompactSplitThread Status: " + CompactSplit.this);<a name="line.542"></a> +<span class="sourceLineNo">543</span> }<a name="line.543"></a> +<span class="sourceLineNo">544</span> this.compaction.getRequest().afterExecute();<a name="line.544"></a> +<span class="sourceLineNo">545</span> }<a name="line.545"></a> +<span class="sourceLineNo">546</span><a name="line.546"></a> +<span class="sourceLineNo">547</span> @Override<a name="line.547"></a> +<span class="sourceLineNo">548</span> public void run() {<a name="line.548"></a> +<span class="sourceLineNo">549</span> Preconditions.checkNotNull(server);<a name="line.549"></a> +<span class="sourceLineNo">550</span> if (server.isStopped()<a name="line.550"></a> +<span class="sourceLineNo">551</span> || (region.getTableDescriptor() != null && !region.getTableDescriptor().isCompactionEnabled())) {<a name="line.551"></a> +<span class="sourceLineNo">552</span> return;<a name="line.552"></a> +<span class="sourceLineNo">553</span> }<a name="line.553"></a> +<span class="sourceLineNo">554</span> doCompaction(user);<a name="line.554"></a> +<span class="sourceLineNo">555</span> }<a name="line.555"></a> +<span class="sourceLineNo">556</span><a name="line.556"></a> +<span class="sourceLineNo">557</span> private String formatStackTrace(Exception ex) {<a name="line.557"></a> +<span class="sourceLineNo">558</span> StringWriter sw = new StringWriter();<a name="line.558"></a> +<span class="sourceLineNo">559</span> PrintWriter pw = new PrintWriter(sw);<a name="line.559"></a> +<span class="sourceLineNo">560</span> ex.printStackTrace(pw);<a name="line.560"></a> +<span class="sourceLineNo">561</span> pw.flush();<a name="line.561"></a> +<span class="sourceLineNo">562</span> return sw.toString();<a name="line.562"></a> +<span class="sourceLineNo">563</span> }<a name="line.563"></a> +<span class="sourceLineNo">564</span><a name="line.564"></a> +<span class="sourceLineNo">565</span> @Override<a name="line.565"></a> +<span class="sourceLineNo">566</span> public int compareTo(CompactionRunner o) {<a name="line.566"></a> +<span class="sourceLineNo">567</span> // Only compare the underlying request (if any), for queue sorting purposes.<a name="line.567"></a> +<span class="sourceLineNo">568</span> int compareVal = queuedPriority - o.queuedPriority; // compare priority<a name="line.568"></a> +<span class="sourceLineNo">569</span> if (compareVal != 0) return compareVal;<a name="line.569"></a> +<span class="sourceLineNo">570</span> CompactionContext tc = this.compaction, oc = o.compaction;<a name="line.570"></a> +<span class="sourceLineNo">571</span> // Sort pre-selected (user?) compactions before system ones with equal priority.<a name="line.571"></a> +<span class="sourceLineNo">572</span> return (tc == null) ? ((oc == null) ? 0 : 1)<a name="line.572"></a> +<span class="sourceLineNo">573</span> : ((oc == null) ? -1 : tc.getRequest().compareTo(oc.getRequest()));<a name="line.573"></a> +<span class="sourceLineNo">574</span> }<a name="line.574"></a> +<span class="sourceLineNo">575</span> }<a name="line.575"></a> +<span class="sourceLineNo">576</span><a name="line.576"></a> +<span class="sourceLineNo">577</span> /**<a name="line.577"></a> +<span class="sourceLineNo">578</span> * Cleanup class to use when rejecting a compaction request from the queue.<a name="line.578"></a> +<span class="sourceLineNo">579</span> */<a name="line.579"></a> +<span class="sourceLineNo">580</span> private static class Rejection implements RejectedExecutionHandler {<a name="line.580"></a> +<span class="sourceLineNo">581</span> @Override<a name="line.581"></a> +<span class="sourceLineNo">582</span> public void rejectedExecution(Runnable runnable, ThreadPoolExecutor pool) {<a name="line.582"></a> +<span class="sourceLineNo">583</span> if (runnable instanceof CompactionRunner) {<a name="line.583"></a> +<span class="sourceLineNo">584</span> CompactionRunner runner = (CompactionRunner)runnable;<a name="line.584"></a> +<span class="sourceLineNo">585</span> LOG.debug("Compaction Rejected: " + runner);<a name="line.585"></a> +<span class="sourceLineNo">586</span> runner.store.cancelRequestedCompaction(runner.compaction);<a name="line.586"></a> +<span class="sourceLineNo">587</span> }<a name="line.587"></a> <span class="sourceLineNo">588</span> }<a name="line.588"></a> <span class="sourceLineNo">589</span> }<a name="line.589"></a> <span class="sourceLineNo">590</span><a name="line.590"></a> <span class="sourceLineNo">591</span> /**<a name="line.591"></a> -<span class="sourceLineNo">592</span> * Cleanup class to use when rejecting a compaction request from the queue.<a name="line.592"></a> +<span class="sourceLineNo">592</span> * {@inheritDoc}<a name="line.592"></a> <span class="sourceLineNo">593</span> */<a name="line.593"></a> -<span class="sourceLineNo">594</span> private static class Rejection implements RejectedExecutionHandler {<a name="line.594"></a> -<span class="sourceLineNo">595</span> @Override<a name="line.595"></a> -<span class="sourceLineNo">596</span> public void rejectedExecution(Runnable runnable, ThreadPoolExecutor pool) {<a name="line.596"></a> -<span class="sourceLineNo">597</span> if (runnable instanceof CompactionRunner) {<a name="line.597"></a> -<span class="sourceLineNo">598</span> CompactionRunner runner = (CompactionRunner)runnable;<a name="line.598"></a> -<span class="sourceLineNo">599</span> LOG.debug("Compaction Rejected: " + runner);<a name="line.599"></a> -<span class="sourceLineNo">600</span> runner.store.cancelRequestedCompaction(runner.compaction);<a name="line.600"></a> -<span class="sourceLineNo">601</span> }<a name="line.601"></a> -<span class="sourceLineNo">602</span> }<a name="line.602"></a> -<span class="sourceLineNo">603</span> }<a name="line.603"></a> -<span class="sourceLineNo">604</span><a name="line.604"></a> -<span class="sourceLineNo">605</span> /**<a name="line.605"></a> -<span class="sourceLineNo">606</span> * {@inheritDoc}<a name="line.606"></a> -<span class="sourceLineNo">607</span> */<a name="line.607"></a> -<span class="sourceLineNo">608</span> @Override<a name="line.608"></a> -<span class="sourceLineNo">609</span> public void onConfigurationChange(Configuration newConf) {<a name="line.609"></a> -<span class="sourceLineNo">610</span> // Check if number of large / small compaction threads has changed, and then<a name="line.610"></a> -<span class="sourceLineNo">611</span> // adjust the core pool size of the thread pools, by using the<a name="line.611"></a> -<span class="sourceLineNo">612</span> // setCorePoolSize() method. According to the javadocs, it is safe to<a name="line.612"></a> -<span class="sourceLineNo">613</span> // change the core pool size on-the-fly. We need to reset the maximum<a name="line.613"></a> -<span class="sourceLineNo">614</span> // pool size, as well.<a name="line.614"></a> -<span class="sourceLineNo">615</span> int largeThreads = Math.max(1, newConf.getInt(<a name="line.615"></a> -<span class="sourceLineNo">616</span> LARGE_COMPACTION_THREADS,<a name="line.616"></a> -<span class="sourceLineNo">617</span> LARGE_COMPACTION_THREADS_DEFAULT));<a name="line.617"></a> -<span class="sourceLineNo">618</span> if (this.longCompactions.getCorePoolSize() != largeThreads) {<a name="line.618"></a> -<span class="sourceLineNo">619</span> LOG.info("Changing the value of " + LARGE_COMPACTION_THREADS +<a name="line.619"></a> -<span class="sourceLineNo">620</span> " from " + this.longCompactions.getCorePoolSize() + " to " +<a name="line.620"></a> -<span class="sourceLineNo">621</span> largeThreads);<a name="line.621"></a> -<span class="sourceLineNo">622</span> if(this.longCompactions.getCorePoolSize() < largeThreads) {<a name="line.622"></a> -<span class="sourceLineNo">623</span> this.longCompactions.setMaximumPoolSize(largeThreads);<a name="line.623"></a> -<span class="sourceLineNo">624</span> this.longCompactions.setCorePoolSize(largeThreads);<a name="line.624"></a> -<span class="sourceLineNo">625</span> } else {<a name="line.625"></a> -<span class="sourceLineNo">626</span> this.longCompactions.setCorePoolSize(largeThreads);<a name="line.626"></a> -<span class="sourceLineNo">627</span> this.longCompactions.setMaximumPoolSize(largeThreads);<a name="line.627"></a> -<span class="sourceLineNo">628</span> }<a name="line.628"></a> -<span class="sourceLineNo">629</span> }<a name="line.629"></a> -<span class="sourceLineNo">630</span><a name="line.630"></a> -<span class="sourceLineNo">631</span> int smallThreads = newConf.getInt(SMALL_COMPACTION_THREADS,<a name="line.631"></a> -<span class="sourceLineNo">632</span> SMALL_COMPACTION_THREADS_DEFAULT);<a name="line.632"></a> -<span class="sourceLineNo">633</span> if (this.shortCompactions.getCorePoolSize() != smallThreads) {<a name="line.633"></a> -<span class="sourceLineNo">634</span> LOG.info("Changing the value of " + SMALL_COMPACTION_THREADS +<a name="line.634"></a> -<span class="sourceLineNo">635</span> " from " + this.shortCompactions.getCorePoolSize() + " to " +<a name="line.635"></a> -<span class="sourceLineNo">636</span> smallThreads);<a name="line.636"></a> -<span class="sourceLineNo">637</span> if(this.shortCompactions.getCorePoolSize() < smallThreads) {<a name="line.637"></a> -<span class="sourceLineNo">638</span> this.shortCompactions.setMaximumPoolSize(smallThreads);<a name="line.638"></a> -<span class="sourceLineNo">639</span> this.shortCompactions.setCorePoolSize(smallThreads);<a name="line.639"></a> -<span class="sourceLineNo">640</span> } else {<a name="line.640"></a> -<span class="sourceLineNo">641</span> this.shortCompactions.setCorePoolSize(smallThreads);<a name="line.641"></a> -<span class="sourceLineNo">642</span> this.shortCompactions.setMaximumPoolSize(smallThreads);<a name="line.642"></a> -<span class="sourceLineNo">643</span> }<a name="line.643"></a> -<span class="sourceLineNo">644</span> }<a name="line.644"></a> -<span class="sourceLineNo">645</span><a name="line.645"></a> -<span class="sourceLineNo">646</span> int splitThreads = newConf.getInt(SPLIT_THREADS,<a name="line.646"></a> -<span class="sourceLineNo">647</span> SPLIT_THREADS_DEFAULT);<a name="line.647"></a> -<span class="sourceLineNo">648</span> if (this.splits.getCorePoolSize() != splitThreads) {<a name="line.648"></a> -<span class="sourceLineNo">649</span> LOG.info("Changing the value of " + SPLIT_THREADS +<a name="line.649"></a> -<span class="sourceLineNo">650</span> " from " + this.splits.getCorePoolSize() + " to " +<a name="line.650"></a> -<span class="sourceLineNo">651</span> splitThreads);<a name="line.651"></a> -<span class="sourceLineNo">652</span> if(this.splits.getCorePoolSize() < splitThreads) {<a name="line.652"></a> -<span class="sourceLineNo">653</span> this.splits.setMaximumPoolSize(splitThreads);<a name="line.653"></a> -<span class="sourceLineNo">654</span> this.splits.setCorePoolSize(splitThreads);<a name="line.654"></a> -<span class="sourceLineNo">655</span> } else {<a name="line.655"></a> -<span class="sourceLineNo">656</span> this.splits.setCorePoolSize(splitThreads);<a name="line.656"></a> -<span class="sourceLineNo">657</span> this.splits.setMaximumPoolSize(splitThreads);<a name="line.657"></a> -<span class="sourceLineNo">658</span> }<a name="line.658"></a> -<span class="sourceLineNo">659</span> }<a name="line.659"></a> -<span class="sourceLineNo">660</span><a name="line.660"></a> -<span class="sourceLineNo">661</span> ThroughputController old = this.compactionThroughputController;<a name="line.661"></a> -<span class="sourceLineNo">662</span> if (old != null) {<a name="line.662"></a> -<span class="sourceLineNo">663</span> old.stop("configuration change");<a name="line.663"></a> -<span class="sourceLineNo">664</span> }<a name="line.664"></a> -<span class="sourceLineNo">665</span> this.compactionThroughputController =<a name="line.665"></a> -<span class="sourceLineNo">666</span> CompactionThroughputControllerFactory.create(server, newConf);<a name="line.666"></a> -<span class="sourceLineNo">667</span><a name="line.667"></a> -<span class="sourceLineNo">668</span> // We change this atomically here instead of reloading the config in order that upstream<a name="line.668"></a> -<span class="sourceLineNo">669</span> // would be the only one with the flexibility to reload the config.<a name="line.669"></a> -<span class="sourceLineNo">670</span> this.conf.reloadConfiguration();<a name="line.670"></a> -<span class="sourceLineNo">671</span> }<a name="line.671"></a> -<span class="sourceLineNo">672</span><a name="line.672"></a> -<span class="sourceLineNo">673</span> protected int getSmallCompactionThreadNum() {<a name="line.673"></a> -<span class="sourceLineNo">674</span> return this.shortCompactions.getCorePoolSize();<a name="line.674"></a> -<span class="sourceLineNo">675</span> }<a name="line.675"></a> -<span class="sourceLineNo">676</span><a name="line.676"></a> -<span class="sourceLineNo">677</span> protected int getLargeCompactionThreadNum() {<a name="line.677"></a> -<span class="sourceLineNo">678</span> return this.longCompactions.getCorePoolSize();<a name="line.678"></a> -<span class="sourceLineNo">679</span> }<a name="line.679"></a> -<span class="sourceLineNo">680</span><a name="line.680"></a> -<span class="sourceLineNo">681</span> protected int getSplitThreadNum() {<a name="line.681"></a> -<span class="sourceLineNo">682</span> return this.splits.getCorePoolSize();<a name="line.682"></a> -<span class="sourceLineNo">683</span> }<a name="line.683"></a> -<span class="sourceLineNo">684</span><a name="line.684"></a> -<span class="sourceLineNo">685</span> /**<a name="line.685"></a> -<span class="sourceLineNo">686</span> * {@inheritDoc}<a name="line.686"></a> -<span class="sourceLineNo">687</span> */<a name="line.687"></a> -<span class="sourceLineNo">688</span> @Override<a name="line.688"></a> -<span class="sourceLineNo">689</span> public void registerChildren(ConfigurationManager manager) {<a name="line.689"></a> -<span class="sourceLineNo">690</span> // No children to register.<a name="line.690"></a> -<span class="sourceLineNo">691</span> }<a name="line.691"></a> -<span class="sourceLineNo">692</span><a name="line.692"></a> +<span class="sourceLineNo">594</span> @Override<a name="line.594"></a> +<span class="sourceLineNo">595</span> public void onConfigurationChange(Configuration newConf) {<a name="line.595"></a> +<span class="sourceLineNo">596</span> // Check if number of large / small compaction threads has changed, and then<a name="line.596"></a> +<span class="sourceLineNo">597</span> // adjust the core pool size of the thread pools, by using the<a name="line.597"></a> +<span class="sourceLineNo">598</span> // setCorePoolSize() method. According to the javadocs, it is safe to<a name="line.598"></a> +<span class="sourceLineNo">599</span> // change the core pool size on-the-fly. We need to reset the maximum<a name="line.599"></a> +<span class="sourceLineNo">600</span> // pool size, as well.<a name="line.600"></a> +<span class="sourceLineNo">601</span> int largeThreads = Math.max(1, newConf.getInt(<a name="line.601"></a> +<span class="sourceLineNo">602</span> LARGE_COMPACTION_THREADS,<a name="line.602"></a> +<span class="sourceLineNo">603</span> LARGE_COMPACTION_THREADS_DEFAULT));<a name="line.603"></a> +<span class="sourceLineNo">604</span> if (this.longCompactions.getCorePoolSize() != largeThreads) {<a name="line.604"></a> +<span class="sourceLineNo">605</span> LOG.info("Changing the value of " + LARGE_COMPACTION_THREADS +<a name="line.605"></a> +<span class="sourceLineNo">606</span> " from " + this.longCompactions.getCorePoolSize() + " to " +<a name="line.606"></a> +<span class="sourceLineNo">607</span> largeThreads);<a name="line.607"></a> +<span class="sourceLineNo">608</span> if(this.longCompactions.getCorePoolSize() < largeThreads) {<a name="line.608"></a> +<span class="sourceLineNo">609</span> this.longCompactions.setMaximumPoolSize(largeThreads);<a name="line.609"></a> +<span class="sourceLineNo">610</span> this.longCompactions.setCorePoolSize(largeThreads);<a name="line.610"></a> +<span class="sourceLineNo">611</span> } else {<a name="line.611"></a> +<span class="sourceLineNo">612</span> this.longCompactions.setCorePoolSize(largeThreads);<a name="line.612"></a> +<span class="sourceLineNo">613</span> this.longCompactions.setMaximumPoolSize(largeThreads);<a name="line.613"></a> +<span class="sourceLineNo">614</span> }<a name="line.614"></a> +<span class="sourceLineNo">615</span> }<a name="line.615"></a> +<span class="sourceLineNo">616</span><a name="line.616"></a> +<span class="sourceLineNo">617</span> int smallThreads = newConf.getInt(SMALL_COMPACTION_THREADS,<a name="line.617"></a> +<span class="sourceLineNo">618</span> SMALL_COMPACTION_THREADS_DEF
<TRUNCATED>