http://git-wip-us.apache.org/repos/asf/hbase-site/blob/9118853f/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/StripeStoreFlusher.html ---------------------------------------------------------------------- diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/StripeStoreFlusher.html b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/StripeStoreFlusher.html index c1a448b..39ff026 100644 --- a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/StripeStoreFlusher.html +++ b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/StripeStoreFlusher.html @@ -72,119 +72,116 @@ <span class="sourceLineNo">064</span><a name="line.64"></a> <span class="sourceLineNo">065</span> long smallestReadPoint = store.getSmallestReadPoint();<a name="line.65"></a> <span class="sourceLineNo">066</span> InternalScanner scanner = createScanner(snapshot.getScanners(), smallestReadPoint, tracker);<a name="line.66"></a> -<span class="sourceLineNo">067</span> if (scanner == null) {<a name="line.67"></a> -<span class="sourceLineNo">068</span> return result; // NULL scanner returned from coprocessor hooks means skip normal processing<a name="line.68"></a> -<span class="sourceLineNo">069</span> }<a name="line.69"></a> -<span class="sourceLineNo">070</span><a name="line.70"></a> -<span class="sourceLineNo">071</span> // Let policy select flush method.<a name="line.71"></a> -<span class="sourceLineNo">072</span> StripeFlushRequest req = this.policy.selectFlush(store.getComparator(), this.stripes,<a name="line.72"></a> -<span class="sourceLineNo">073</span> cellsCount);<a name="line.73"></a> -<span class="sourceLineNo">074</span><a name="line.74"></a> -<span class="sourceLineNo">075</span> boolean success = false;<a name="line.75"></a> -<span class="sourceLineNo">076</span> StripeMultiFileWriter mw = null;<a name="line.76"></a> -<span class="sourceLineNo">077</span> try {<a name="line.77"></a> -<span class="sourceLineNo">078</span> mw = req.createWriter(); // Writer according to the policy.<a name="line.78"></a> -<span class="sourceLineNo">079</span> StripeMultiFileWriter.WriterFactory factory = createWriterFactory(cellsCount);<a name="line.79"></a> -<span class="sourceLineNo">080</span> StoreScanner storeScanner = (scanner instanceof StoreScanner) ? (StoreScanner)scanner : null;<a name="line.80"></a> -<span class="sourceLineNo">081</span> mw.init(storeScanner, factory);<a name="line.81"></a> -<span class="sourceLineNo">082</span><a name="line.82"></a> -<span class="sourceLineNo">083</span> synchronized (flushLock) {<a name="line.83"></a> -<span class="sourceLineNo">084</span> performFlush(scanner, mw, smallestReadPoint, throughputController);<a name="line.84"></a> -<span class="sourceLineNo">085</span> result = mw.commitWriters(cacheFlushSeqNum, false);<a name="line.85"></a> -<span class="sourceLineNo">086</span> success = true;<a name="line.86"></a> -<span class="sourceLineNo">087</span> }<a name="line.87"></a> -<span class="sourceLineNo">088</span> } finally {<a name="line.88"></a> -<span class="sourceLineNo">089</span> if (!success && (mw != null)) {<a name="line.89"></a> -<span class="sourceLineNo">090</span> for (Path leftoverFile : mw.abortWriters()) {<a name="line.90"></a> -<span class="sourceLineNo">091</span> try {<a name="line.91"></a> -<span class="sourceLineNo">092</span> store.getFileSystem().delete(leftoverFile, false);<a name="line.92"></a> -<span class="sourceLineNo">093</span> } catch (Exception e) {<a name="line.93"></a> -<span class="sourceLineNo">094</span> LOG.error("Failed to delete a file after failed flush: " + e);<a name="line.94"></a> -<span class="sourceLineNo">095</span> }<a name="line.95"></a> -<span class="sourceLineNo">096</span> }<a name="line.96"></a> -<span class="sourceLineNo">097</span> }<a name="line.97"></a> -<span class="sourceLineNo">098</span> try {<a name="line.98"></a> -<span class="sourceLineNo">099</span> scanner.close();<a name="line.99"></a> -<span class="sourceLineNo">100</span> } catch (IOException ex) {<a name="line.100"></a> -<span class="sourceLineNo">101</span> LOG.warn("Failed to close flush scanner, ignoring", ex);<a name="line.101"></a> -<span class="sourceLineNo">102</span> }<a name="line.102"></a> -<span class="sourceLineNo">103</span> }<a name="line.103"></a> -<span class="sourceLineNo">104</span> return result;<a name="line.104"></a> -<span class="sourceLineNo">105</span> }<a name="line.105"></a> -<span class="sourceLineNo">106</span><a name="line.106"></a> -<span class="sourceLineNo">107</span> private StripeMultiFileWriter.WriterFactory createWriterFactory(final long kvCount) {<a name="line.107"></a> -<span class="sourceLineNo">108</span> return new StripeMultiFileWriter.WriterFactory() {<a name="line.108"></a> -<span class="sourceLineNo">109</span> @Override<a name="line.109"></a> -<span class="sourceLineNo">110</span> public StoreFileWriter createWriter() throws IOException {<a name="line.110"></a> -<span class="sourceLineNo">111</span> StoreFileWriter writer = store.createWriterInTmp(<a name="line.111"></a> -<span class="sourceLineNo">112</span> kvCount, store.getColumnFamilyDescriptor().getCompressionType(),<a name="line.112"></a> -<span class="sourceLineNo">113</span> /* isCompaction = */ false,<a name="line.113"></a> -<span class="sourceLineNo">114</span> /* includeMVCCReadpoint = */ true,<a name="line.114"></a> -<span class="sourceLineNo">115</span> /* includesTags = */ true,<a name="line.115"></a> -<span class="sourceLineNo">116</span> /* shouldDropBehind = */ false);<a name="line.116"></a> -<span class="sourceLineNo">117</span> return writer;<a name="line.117"></a> -<span class="sourceLineNo">118</span> }<a name="line.118"></a> -<span class="sourceLineNo">119</span> };<a name="line.119"></a> -<span class="sourceLineNo">120</span> }<a name="line.120"></a> +<span class="sourceLineNo">067</span><a name="line.67"></a> +<span class="sourceLineNo">068</span> // Let policy select flush method.<a name="line.68"></a> +<span class="sourceLineNo">069</span> StripeFlushRequest req = this.policy.selectFlush(store.getComparator(), this.stripes,<a name="line.69"></a> +<span class="sourceLineNo">070</span> cellsCount);<a name="line.70"></a> +<span class="sourceLineNo">071</span><a name="line.71"></a> +<span class="sourceLineNo">072</span> boolean success = false;<a name="line.72"></a> +<span class="sourceLineNo">073</span> StripeMultiFileWriter mw = null;<a name="line.73"></a> +<span class="sourceLineNo">074</span> try {<a name="line.74"></a> +<span class="sourceLineNo">075</span> mw = req.createWriter(); // Writer according to the policy.<a name="line.75"></a> +<span class="sourceLineNo">076</span> StripeMultiFileWriter.WriterFactory factory = createWriterFactory(cellsCount);<a name="line.76"></a> +<span class="sourceLineNo">077</span> StoreScanner storeScanner = (scanner instanceof StoreScanner) ? (StoreScanner)scanner : null;<a name="line.77"></a> +<span class="sourceLineNo">078</span> mw.init(storeScanner, factory);<a name="line.78"></a> +<span class="sourceLineNo">079</span><a name="line.79"></a> +<span class="sourceLineNo">080</span> synchronized (flushLock) {<a name="line.80"></a> +<span class="sourceLineNo">081</span> performFlush(scanner, mw, smallestReadPoint, throughputController);<a name="line.81"></a> +<span class="sourceLineNo">082</span> result = mw.commitWriters(cacheFlushSeqNum, false);<a name="line.82"></a> +<span class="sourceLineNo">083</span> success = true;<a name="line.83"></a> +<span class="sourceLineNo">084</span> }<a name="line.84"></a> +<span class="sourceLineNo">085</span> } finally {<a name="line.85"></a> +<span class="sourceLineNo">086</span> if (!success && (mw != null)) {<a name="line.86"></a> +<span class="sourceLineNo">087</span> for (Path leftoverFile : mw.abortWriters()) {<a name="line.87"></a> +<span class="sourceLineNo">088</span> try {<a name="line.88"></a> +<span class="sourceLineNo">089</span> store.getFileSystem().delete(leftoverFile, false);<a name="line.89"></a> +<span class="sourceLineNo">090</span> } catch (Exception e) {<a name="line.90"></a> +<span class="sourceLineNo">091</span> LOG.error("Failed to delete a file after failed flush: " + e);<a name="line.91"></a> +<span class="sourceLineNo">092</span> }<a name="line.92"></a> +<span class="sourceLineNo">093</span> }<a name="line.93"></a> +<span class="sourceLineNo">094</span> }<a name="line.94"></a> +<span class="sourceLineNo">095</span> try {<a name="line.95"></a> +<span class="sourceLineNo">096</span> scanner.close();<a name="line.96"></a> +<span class="sourceLineNo">097</span> } catch (IOException ex) {<a name="line.97"></a> +<span class="sourceLineNo">098</span> LOG.warn("Failed to close flush scanner, ignoring", ex);<a name="line.98"></a> +<span class="sourceLineNo">099</span> }<a name="line.99"></a> +<span class="sourceLineNo">100</span> }<a name="line.100"></a> +<span class="sourceLineNo">101</span> return result;<a name="line.101"></a> +<span class="sourceLineNo">102</span> }<a name="line.102"></a> +<span class="sourceLineNo">103</span><a name="line.103"></a> +<span class="sourceLineNo">104</span> private StripeMultiFileWriter.WriterFactory createWriterFactory(final long kvCount) {<a name="line.104"></a> +<span class="sourceLineNo">105</span> return new StripeMultiFileWriter.WriterFactory() {<a name="line.105"></a> +<span class="sourceLineNo">106</span> @Override<a name="line.106"></a> +<span class="sourceLineNo">107</span> public StoreFileWriter createWriter() throws IOException {<a name="line.107"></a> +<span class="sourceLineNo">108</span> StoreFileWriter writer = store.createWriterInTmp(<a name="line.108"></a> +<span class="sourceLineNo">109</span> kvCount, store.getColumnFamilyDescriptor().getCompressionType(),<a name="line.109"></a> +<span class="sourceLineNo">110</span> /* isCompaction = */ false,<a name="line.110"></a> +<span class="sourceLineNo">111</span> /* includeMVCCReadpoint = */ true,<a name="line.111"></a> +<span class="sourceLineNo">112</span> /* includesTags = */ true,<a name="line.112"></a> +<span class="sourceLineNo">113</span> /* shouldDropBehind = */ false);<a name="line.113"></a> +<span class="sourceLineNo">114</span> return writer;<a name="line.114"></a> +<span class="sourceLineNo">115</span> }<a name="line.115"></a> +<span class="sourceLineNo">116</span> };<a name="line.116"></a> +<span class="sourceLineNo">117</span> }<a name="line.117"></a> +<span class="sourceLineNo">118</span><a name="line.118"></a> +<span class="sourceLineNo">119</span> /** Stripe flush request wrapper that writes a non-striped file. */<a name="line.119"></a> +<span class="sourceLineNo">120</span> public static class StripeFlushRequest {<a name="line.120"></a> <span class="sourceLineNo">121</span><a name="line.121"></a> -<span class="sourceLineNo">122</span> /** Stripe flush request wrapper that writes a non-striped file. */<a name="line.122"></a> -<span class="sourceLineNo">123</span> public static class StripeFlushRequest {<a name="line.123"></a> -<span class="sourceLineNo">124</span><a name="line.124"></a> -<span class="sourceLineNo">125</span> protected final CellComparator comparator;<a name="line.125"></a> -<span class="sourceLineNo">126</span><a name="line.126"></a> -<span class="sourceLineNo">127</span> public StripeFlushRequest(CellComparator comparator) {<a name="line.127"></a> -<span class="sourceLineNo">128</span> this.comparator = comparator;<a name="line.128"></a> -<span class="sourceLineNo">129</span> }<a name="line.129"></a> -<span class="sourceLineNo">130</span><a name="line.130"></a> -<span class="sourceLineNo">131</span> @VisibleForTesting<a name="line.131"></a> -<span class="sourceLineNo">132</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.132"></a> -<span class="sourceLineNo">133</span> StripeMultiFileWriter writer = new StripeMultiFileWriter.SizeMultiWriter(comparator, 1,<a name="line.133"></a> -<span class="sourceLineNo">134</span> Long.MAX_VALUE, OPEN_KEY, OPEN_KEY);<a name="line.134"></a> -<span class="sourceLineNo">135</span> writer.setNoStripeMetadata();<a name="line.135"></a> -<span class="sourceLineNo">136</span> return writer;<a name="line.136"></a> -<span class="sourceLineNo">137</span> }<a name="line.137"></a> -<span class="sourceLineNo">138</span> }<a name="line.138"></a> -<span class="sourceLineNo">139</span><a name="line.139"></a> -<span class="sourceLineNo">140</span> /** Stripe flush request wrapper based on boundaries. */<a name="line.140"></a> -<span class="sourceLineNo">141</span> public static class BoundaryStripeFlushRequest extends StripeFlushRequest {<a name="line.141"></a> -<span class="sourceLineNo">142</span> private final List<byte[]> targetBoundaries;<a name="line.142"></a> -<span class="sourceLineNo">143</span><a name="line.143"></a> -<span class="sourceLineNo">144</span> /** @param targetBoundaries New files should be written with these boundaries. */<a name="line.144"></a> -<span class="sourceLineNo">145</span> public BoundaryStripeFlushRequest(CellComparator comparator, List<byte[]> targetBoundaries) {<a name="line.145"></a> -<span class="sourceLineNo">146</span> super(comparator);<a name="line.146"></a> -<span class="sourceLineNo">147</span> this.targetBoundaries = targetBoundaries;<a name="line.147"></a> -<span class="sourceLineNo">148</span> }<a name="line.148"></a> -<span class="sourceLineNo">149</span><a name="line.149"></a> -<span class="sourceLineNo">150</span> @Override<a name="line.150"></a> -<span class="sourceLineNo">151</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.151"></a> -<span class="sourceLineNo">152</span> return new StripeMultiFileWriter.BoundaryMultiWriter(comparator, targetBoundaries, null,<a name="line.152"></a> -<span class="sourceLineNo">153</span> null);<a name="line.153"></a> -<span class="sourceLineNo">154</span> }<a name="line.154"></a> -<span class="sourceLineNo">155</span> }<a name="line.155"></a> -<span class="sourceLineNo">156</span><a name="line.156"></a> -<span class="sourceLineNo">157</span> /** Stripe flush request wrapper based on size. */<a name="line.157"></a> -<span class="sourceLineNo">158</span> public static class SizeStripeFlushRequest extends StripeFlushRequest {<a name="line.158"></a> -<span class="sourceLineNo">159</span> private final int targetCount;<a name="line.159"></a> -<span class="sourceLineNo">160</span> private final long targetKvs;<a name="line.160"></a> -<span class="sourceLineNo">161</span><a name="line.161"></a> -<span class="sourceLineNo">162</span> /**<a name="line.162"></a> -<span class="sourceLineNo">163</span> * @param targetCount The maximum number of stripes to flush into.<a name="line.163"></a> -<span class="sourceLineNo">164</span> * @param targetKvs The KV count of each segment. If targetKvs*targetCount is less than<a name="line.164"></a> -<span class="sourceLineNo">165</span> * total number of kvs, all the overflow data goes into the last stripe.<a name="line.165"></a> -<span class="sourceLineNo">166</span> */<a name="line.166"></a> -<span class="sourceLineNo">167</span> public SizeStripeFlushRequest(CellComparator comparator, int targetCount, long targetKvs) {<a name="line.167"></a> -<span class="sourceLineNo">168</span> super(comparator);<a name="line.168"></a> -<span class="sourceLineNo">169</span> this.targetCount = targetCount;<a name="line.169"></a> -<span class="sourceLineNo">170</span> this.targetKvs = targetKvs;<a name="line.170"></a> -<span class="sourceLineNo">171</span> }<a name="line.171"></a> -<span class="sourceLineNo">172</span><a name="line.172"></a> -<span class="sourceLineNo">173</span> @Override<a name="line.173"></a> -<span class="sourceLineNo">174</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.174"></a> -<span class="sourceLineNo">175</span> return new StripeMultiFileWriter.SizeMultiWriter(comparator, this.targetCount, this.targetKvs,<a name="line.175"></a> -<span class="sourceLineNo">176</span> OPEN_KEY, OPEN_KEY);<a name="line.176"></a> -<span class="sourceLineNo">177</span> }<a name="line.177"></a> -<span class="sourceLineNo">178</span> }<a name="line.178"></a> -<span class="sourceLineNo">179</span>}<a name="line.179"></a> +<span class="sourceLineNo">122</span> protected final CellComparator comparator;<a name="line.122"></a> +<span class="sourceLineNo">123</span><a name="line.123"></a> +<span class="sourceLineNo">124</span> public StripeFlushRequest(CellComparator comparator) {<a name="line.124"></a> +<span class="sourceLineNo">125</span> this.comparator = comparator;<a name="line.125"></a> +<span class="sourceLineNo">126</span> }<a name="line.126"></a> +<span class="sourceLineNo">127</span><a name="line.127"></a> +<span class="sourceLineNo">128</span> @VisibleForTesting<a name="line.128"></a> +<span class="sourceLineNo">129</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.129"></a> +<span class="sourceLineNo">130</span> StripeMultiFileWriter writer = new StripeMultiFileWriter.SizeMultiWriter(comparator, 1,<a name="line.130"></a> +<span class="sourceLineNo">131</span> Long.MAX_VALUE, OPEN_KEY, OPEN_KEY);<a name="line.131"></a> +<span class="sourceLineNo">132</span> writer.setNoStripeMetadata();<a name="line.132"></a> +<span class="sourceLineNo">133</span> return writer;<a name="line.133"></a> +<span class="sourceLineNo">134</span> }<a name="line.134"></a> +<span class="sourceLineNo">135</span> }<a name="line.135"></a> +<span class="sourceLineNo">136</span><a name="line.136"></a> +<span class="sourceLineNo">137</span> /** Stripe flush request wrapper based on boundaries. */<a name="line.137"></a> +<span class="sourceLineNo">138</span> public static class BoundaryStripeFlushRequest extends StripeFlushRequest {<a name="line.138"></a> +<span class="sourceLineNo">139</span> private final List<byte[]> targetBoundaries;<a name="line.139"></a> +<span class="sourceLineNo">140</span><a name="line.140"></a> +<span class="sourceLineNo">141</span> /** @param targetBoundaries New files should be written with these boundaries. */<a name="line.141"></a> +<span class="sourceLineNo">142</span> public BoundaryStripeFlushRequest(CellComparator comparator, List<byte[]> targetBoundaries) {<a name="line.142"></a> +<span class="sourceLineNo">143</span> super(comparator);<a name="line.143"></a> +<span class="sourceLineNo">144</span> this.targetBoundaries = targetBoundaries;<a name="line.144"></a> +<span class="sourceLineNo">145</span> }<a name="line.145"></a> +<span class="sourceLineNo">146</span><a name="line.146"></a> +<span class="sourceLineNo">147</span> @Override<a name="line.147"></a> +<span class="sourceLineNo">148</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.148"></a> +<span class="sourceLineNo">149</span> return new StripeMultiFileWriter.BoundaryMultiWriter(comparator, targetBoundaries, null,<a name="line.149"></a> +<span class="sourceLineNo">150</span> null);<a name="line.150"></a> +<span class="sourceLineNo">151</span> }<a name="line.151"></a> +<span class="sourceLineNo">152</span> }<a name="line.152"></a> +<span class="sourceLineNo">153</span><a name="line.153"></a> +<span class="sourceLineNo">154</span> /** Stripe flush request wrapper based on size. */<a name="line.154"></a> +<span class="sourceLineNo">155</span> public static class SizeStripeFlushRequest extends StripeFlushRequest {<a name="line.155"></a> +<span class="sourceLineNo">156</span> private final int targetCount;<a name="line.156"></a> +<span class="sourceLineNo">157</span> private final long targetKvs;<a name="line.157"></a> +<span class="sourceLineNo">158</span><a name="line.158"></a> +<span class="sourceLineNo">159</span> /**<a name="line.159"></a> +<span class="sourceLineNo">160</span> * @param targetCount The maximum number of stripes to flush into.<a name="line.160"></a> +<span class="sourceLineNo">161</span> * @param targetKvs The KV count of each segment. If targetKvs*targetCount is less than<a name="line.161"></a> +<span class="sourceLineNo">162</span> * total number of kvs, all the overflow data goes into the last stripe.<a name="line.162"></a> +<span class="sourceLineNo">163</span> */<a name="line.163"></a> +<span class="sourceLineNo">164</span> public SizeStripeFlushRequest(CellComparator comparator, int targetCount, long targetKvs) {<a name="line.164"></a> +<span class="sourceLineNo">165</span> super(comparator);<a name="line.165"></a> +<span class="sourceLineNo">166</span> this.targetCount = targetCount;<a name="line.166"></a> +<span class="sourceLineNo">167</span> this.targetKvs = targetKvs;<a name="line.167"></a> +<span class="sourceLineNo">168</span> }<a name="line.168"></a> +<span class="sourceLineNo">169</span><a name="line.169"></a> +<span class="sourceLineNo">170</span> @Override<a name="line.170"></a> +<span class="sourceLineNo">171</span> public StripeMultiFileWriter createWriter() throws IOException {<a name="line.171"></a> +<span class="sourceLineNo">172</span> return new StripeMultiFileWriter.SizeMultiWriter(comparator, this.targetCount, this.targetKvs,<a name="line.172"></a> +<span class="sourceLineNo">173</span> OPEN_KEY, OPEN_KEY);<a name="line.173"></a> +<span class="sourceLineNo">174</span> }<a name="line.174"></a> +<span class="sourceLineNo">175</span> }<a name="line.175"></a> +<span class="sourceLineNo">176</span>}<a name="line.176"></a>
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/9118853f/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/compactions/Compactor.CellSinkFactory.html ---------------------------------------------------------------------- diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/compactions/Compactor.CellSinkFactory.html b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/compactions/Compactor.CellSinkFactory.html index 589ee04..3161b9b 100644 --- a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/compactions/Compactor.CellSinkFactory.html +++ b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/compactions/Compactor.CellSinkFactory.html @@ -328,195 +328,191 @@ <span class="sourceLineNo">320</span> ScanInfo scanInfo = preCompactScannerOpen(request, scanType, user);<a name="line.320"></a> <span class="sourceLineNo">321</span> scanner = postCompactScannerOpen(request, scanType,<a name="line.321"></a> <span class="sourceLineNo">322</span> scannerFactory.createScanner(scanInfo, scanners, scanType, fd, smallestReadPoint), user);<a name="line.322"></a> -<span class="sourceLineNo">323</span> if (scanner == null) {<a name="line.323"></a> -<span class="sourceLineNo">324</span> // NULL scanner returned from coprocessor hooks means skip normal processing.<a name="line.324"></a> -<span class="sourceLineNo">325</span> return new ArrayList<>();<a name="line.325"></a> -<span class="sourceLineNo">326</span> }<a name="line.326"></a> -<span class="sourceLineNo">327</span> boolean cleanSeqId = false;<a name="line.327"></a> -<span class="sourceLineNo">328</span> if (fd.minSeqIdToKeep > 0 && !store.getColumnFamilyDescriptor().isNewVersionBehavior()) {<a name="line.328"></a> -<span class="sourceLineNo">329</span> // For mvcc-sensitive family, we never set mvcc to 0.<a name="line.329"></a> -<span class="sourceLineNo">330</span> smallestReadPoint = Math.min(fd.minSeqIdToKeep, smallestReadPoint);<a name="line.330"></a> -<span class="sourceLineNo">331</span> cleanSeqId = true;<a name="line.331"></a> -<span class="sourceLineNo">332</span> }<a name="line.332"></a> -<span class="sourceLineNo">333</span> writer = sinkFactory.createWriter(scanner, fd, dropCache);<a name="line.333"></a> -<span class="sourceLineNo">334</span> finished = performCompaction(fd, scanner, writer, smallestReadPoint, cleanSeqId,<a name="line.334"></a> -<span class="sourceLineNo">335</span> throughputController, request.isAllFiles(), request.getFiles().size());<a name="line.335"></a> -<span class="sourceLineNo">336</span> if (!finished) {<a name="line.336"></a> -<span class="sourceLineNo">337</span> throw new InterruptedIOException("Aborting compaction of store " + store + " in region "<a name="line.337"></a> -<span class="sourceLineNo">338</span> + store.getRegionInfo().getRegionNameAsString() + " because it was interrupted.");<a name="line.338"></a> -<span class="sourceLineNo">339</span> }<a name="line.339"></a> -<span class="sourceLineNo">340</span> } finally {<a name="line.340"></a> -<span class="sourceLineNo">341</span> Closeables.close(scanner, true);<a name="line.341"></a> -<span class="sourceLineNo">342</span> if (!finished && writer != null) {<a name="line.342"></a> -<span class="sourceLineNo">343</span> abortWriter(writer);<a name="line.343"></a> -<span class="sourceLineNo">344</span> }<a name="line.344"></a> -<span class="sourceLineNo">345</span> }<a name="line.345"></a> -<span class="sourceLineNo">346</span> assert finished : "We should have exited the method on all error paths";<a name="line.346"></a> -<span class="sourceLineNo">347</span> assert writer != null : "Writer should be non-null if no error";<a name="line.347"></a> -<span class="sourceLineNo">348</span> return commitWriter(writer, fd, request);<a name="line.348"></a> -<span class="sourceLineNo">349</span> }<a name="line.349"></a> -<span class="sourceLineNo">350</span><a name="line.350"></a> -<span class="sourceLineNo">351</span> protected abstract List<Path> commitWriter(T writer, FileDetails fd,<a name="line.351"></a> -<span class="sourceLineNo">352</span> CompactionRequestImpl request) throws IOException;<a name="line.352"></a> -<span class="sourceLineNo">353</span><a name="line.353"></a> -<span class="sourceLineNo">354</span> protected abstract void abortWriter(T writer) throws IOException;<a name="line.354"></a> -<span class="sourceLineNo">355</span><a name="line.355"></a> -<span class="sourceLineNo">356</span> /**<a name="line.356"></a> -<span class="sourceLineNo">357</span> * Performs the compaction.<a name="line.357"></a> -<span class="sourceLineNo">358</span> * @param fd FileDetails of cell sink writer<a name="line.358"></a> -<span class="sourceLineNo">359</span> * @param scanner Where to read from.<a name="line.359"></a> -<span class="sourceLineNo">360</span> * @param writer Where to write to.<a name="line.360"></a> -<span class="sourceLineNo">361</span> * @param smallestReadPoint Smallest read point.<a name="line.361"></a> -<span class="sourceLineNo">362</span> * @param cleanSeqId When true, remove seqId(used to be mvcc) value which is &lt;=<a name="line.362"></a> -<span class="sourceLineNo">363</span> * smallestReadPoint<a name="line.363"></a> -<span class="sourceLineNo">364</span> * @param major Is a major compaction.<a name="line.364"></a> -<span class="sourceLineNo">365</span> * @param numofFilesToCompact the number of files to compact<a name="line.365"></a> -<span class="sourceLineNo">366</span> * @return Whether compaction ended; false if it was interrupted for some reason.<a name="line.366"></a> -<span class="sourceLineNo">367</span> */<a name="line.367"></a> -<span class="sourceLineNo">368</span> protected boolean performCompaction(FileDetails fd, InternalScanner scanner, CellSink writer,<a name="line.368"></a> -<span class="sourceLineNo">369</span> long smallestReadPoint, boolean cleanSeqId, ThroughputController throughputController,<a name="line.369"></a> -<span class="sourceLineNo">370</span> boolean major, int numofFilesToCompact) throws IOException {<a name="line.370"></a> -<span class="sourceLineNo">371</span> assert writer instanceof ShipperListener;<a name="line.371"></a> -<span class="sourceLineNo">372</span> long bytesWrittenProgressForCloseCheck = 0;<a name="line.372"></a> -<span class="sourceLineNo">373</span> long bytesWrittenProgressForLog = 0;<a name="line.373"></a> -<span class="sourceLineNo">374</span> long bytesWrittenProgressForShippedCall = 0;<a name="line.374"></a> -<span class="sourceLineNo">375</span> // Since scanner.next() can return 'false' but still be delivering data,<a name="line.375"></a> -<span class="sourceLineNo">376</span> // we have to use a do/while loop.<a name="line.376"></a> -<span class="sourceLineNo">377</span> List<Cell> cells = new ArrayList<>();<a name="line.377"></a> -<span class="sourceLineNo">378</span> long closeCheckSizeLimit = HStore.getCloseCheckInterval();<a name="line.378"></a> -<span class="sourceLineNo">379</span> long lastMillis = 0;<a name="line.379"></a> -<span class="sourceLineNo">380</span> if (LOG.isDebugEnabled()) {<a name="line.380"></a> -<span class="sourceLineNo">381</span> lastMillis = EnvironmentEdgeManager.currentTime();<a name="line.381"></a> -<span class="sourceLineNo">382</span> }<a name="line.382"></a> -<span class="sourceLineNo">383</span> String compactionName = ThroughputControlUtil.getNameForThrottling(store, "compaction");<a name="line.383"></a> -<span class="sourceLineNo">384</span> long now = 0;<a name="line.384"></a> -<span class="sourceLineNo">385</span> boolean hasMore;<a name="line.385"></a> -<span class="sourceLineNo">386</span> ScannerContext scannerContext =<a name="line.386"></a> -<span class="sourceLineNo">387</span> ScannerContext.newBuilder().setBatchLimit(compactionKVMax).build();<a name="line.387"></a> -<span class="sourceLineNo">388</span><a name="line.388"></a> -<span class="sourceLineNo">389</span> throughputController.start(compactionName);<a name="line.389"></a> -<span class="sourceLineNo">390</span> KeyValueScanner kvs = (scanner instanceof KeyValueScanner)? (KeyValueScanner)scanner : null;<a name="line.390"></a> -<span class="sourceLineNo">391</span> long shippedCallSizeLimit = (long) numofFilesToCompact * this.store.getColumnFamilyDescriptor().getBlocksize();<a name="line.391"></a> -<span class="sourceLineNo">392</span> try {<a name="line.392"></a> -<span class="sourceLineNo">393</span> do {<a name="line.393"></a> -<span class="sourceLineNo">394</span> hasMore = scanner.next(cells, scannerContext);<a name="line.394"></a> -<span class="sourceLineNo">395</span> if (LOG.isDebugEnabled()) {<a name="line.395"></a> -<span class="sourceLineNo">396</span> now = EnvironmentEdgeManager.currentTime();<a name="line.396"></a> -<span class="sourceLineNo">397</span> }<a name="line.397"></a> -<span class="sourceLineNo">398</span> // output to writer:<a name="line.398"></a> -<span class="sourceLineNo">399</span> Cell lastCleanCell = null;<a name="line.399"></a> -<span class="sourceLineNo">400</span> long lastCleanCellSeqId = 0;<a name="line.400"></a> -<span class="sourceLineNo">401</span> for (Cell c : cells) {<a name="line.401"></a> -<span class="sourceLineNo">402</span> if (cleanSeqId && c.getSequenceId() <= smallestReadPoint) {<a name="line.402"></a> -<span class="sourceLineNo">403</span> lastCleanCell = c;<a name="line.403"></a> -<span class="sourceLineNo">404</span> lastCleanCellSeqId = c.getSequenceId();<a name="line.404"></a> -<span class="sourceLineNo">405</span> PrivateCellUtil.setSequenceId(c, 0);<a name="line.405"></a> -<span class="sourceLineNo">406</span> } else {<a name="line.406"></a> -<span class="sourceLineNo">407</span> lastCleanCell = null;<a name="line.407"></a> -<span class="sourceLineNo">408</span> lastCleanCellSeqId = 0;<a name="line.408"></a> -<span class="sourceLineNo">409</span> }<a name="line.409"></a> -<span class="sourceLineNo">410</span> writer.append(c);<a name="line.410"></a> -<span class="sourceLineNo">411</span> int len = KeyValueUtil.length(c);<a name="line.411"></a> -<span class="sourceLineNo">412</span> ++progress.currentCompactedKVs;<a name="line.412"></a> -<span class="sourceLineNo">413</span> progress.totalCompactedSize += len;<a name="line.413"></a> -<span class="sourceLineNo">414</span> bytesWrittenProgressForShippedCall += len;<a name="line.414"></a> -<span class="sourceLineNo">415</span> if (LOG.isDebugEnabled()) {<a name="line.415"></a> -<span class="sourceLineNo">416</span> bytesWrittenProgressForLog += len;<a name="line.416"></a> -<span class="sourceLineNo">417</span> }<a name="line.417"></a> -<span class="sourceLineNo">418</span> throughputController.control(compactionName, len);<a name="line.418"></a> -<span class="sourceLineNo">419</span> // check periodically to see if a system stop is requested<a name="line.419"></a> -<span class="sourceLineNo">420</span> if (closeCheckSizeLimit > 0) {<a name="line.420"></a> -<span class="sourceLineNo">421</span> bytesWrittenProgressForCloseCheck += len;<a name="line.421"></a> -<span class="sourceLineNo">422</span> if (bytesWrittenProgressForCloseCheck > closeCheckSizeLimit) {<a name="line.422"></a> -<span class="sourceLineNo">423</span> bytesWrittenProgressForCloseCheck = 0;<a name="line.423"></a> -<span class="sourceLineNo">424</span> if (!store.areWritesEnabled()) {<a name="line.424"></a> -<span class="sourceLineNo">425</span> progress.cancel();<a name="line.425"></a> -<span class="sourceLineNo">426</span> return false;<a name="line.426"></a> -<span class="sourceLineNo">427</span> }<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> if (kvs != null && bytesWrittenProgressForShippedCall > shippedCallSizeLimit) {<a name="line.430"></a> -<span class="sourceLineNo">431</span> if (lastCleanCell != null) {<a name="line.431"></a> -<span class="sourceLineNo">432</span> // HBASE-16931, set back sequence id to avoid affecting scan order unexpectedly.<a name="line.432"></a> -<span class="sourceLineNo">433</span> // ShipperListener will do a clone of the last cells it refer, so need to set back<a name="line.433"></a> -<span class="sourceLineNo">434</span> // sequence id before ShipperListener.beforeShipped<a name="line.434"></a> -<span class="sourceLineNo">435</span> PrivateCellUtil.setSequenceId(lastCleanCell, lastCleanCellSeqId);<a name="line.435"></a> -<span class="sourceLineNo">436</span> }<a name="line.436"></a> -<span class="sourceLineNo">437</span> // Clone the cells that are in the writer so that they are freed of references,<a name="line.437"></a> -<span class="sourceLineNo">438</span> // if they are holding any.<a name="line.438"></a> -<span class="sourceLineNo">439</span> ((ShipperListener)writer).beforeShipped();<a name="line.439"></a> -<span class="sourceLineNo">440</span> // The SHARED block references, being read for compaction, will be kept in prevBlocks<a name="line.440"></a> -<span class="sourceLineNo">441</span> // list(See HFileScannerImpl#prevBlocks). In case of scan flow, after each set of cells<a name="line.441"></a> -<span class="sourceLineNo">442</span> // being returned to client, we will call shipped() which can clear this list. Here by<a name="line.442"></a> -<span class="sourceLineNo">443</span> // we are doing the similar thing. In between the compaction (after every N cells<a name="line.443"></a> -<span class="sourceLineNo">444</span> // written with collective size of 'shippedCallSizeLimit') we will call shipped which<a name="line.444"></a> -<span class="sourceLineNo">445</span> // may clear prevBlocks list.<a name="line.445"></a> -<span class="sourceLineNo">446</span> kvs.shipped();<a name="line.446"></a> -<span class="sourceLineNo">447</span> bytesWrittenProgressForShippedCall = 0;<a name="line.447"></a> -<span class="sourceLineNo">448</span> }<a name="line.448"></a> +<span class="sourceLineNo">323</span> boolean cleanSeqId = false;<a name="line.323"></a> +<span class="sourceLineNo">324</span> if (fd.minSeqIdToKeep > 0 && !store.getColumnFamilyDescriptor().isNewVersionBehavior()) {<a name="line.324"></a> +<span class="sourceLineNo">325</span> // For mvcc-sensitive family, we never set mvcc to 0.<a name="line.325"></a> +<span class="sourceLineNo">326</span> smallestReadPoint = Math.min(fd.minSeqIdToKeep, smallestReadPoint);<a name="line.326"></a> +<span class="sourceLineNo">327</span> cleanSeqId = true;<a name="line.327"></a> +<span class="sourceLineNo">328</span> }<a name="line.328"></a> +<span class="sourceLineNo">329</span> writer = sinkFactory.createWriter(scanner, fd, dropCache);<a name="line.329"></a> +<span class="sourceLineNo">330</span> finished = performCompaction(fd, scanner, writer, smallestReadPoint, cleanSeqId,<a name="line.330"></a> +<span class="sourceLineNo">331</span> throughputController, request.isAllFiles(), request.getFiles().size());<a name="line.331"></a> +<span class="sourceLineNo">332</span> if (!finished) {<a name="line.332"></a> +<span class="sourceLineNo">333</span> throw new InterruptedIOException("Aborting compaction of store " + store + " in region "<a name="line.333"></a> +<span class="sourceLineNo">334</span> + store.getRegionInfo().getRegionNameAsString() + " because it was interrupted.");<a name="line.334"></a> +<span class="sourceLineNo">335</span> }<a name="line.335"></a> +<span class="sourceLineNo">336</span> } finally {<a name="line.336"></a> +<span class="sourceLineNo">337</span> Closeables.close(scanner, true);<a name="line.337"></a> +<span class="sourceLineNo">338</span> if (!finished && writer != null) {<a name="line.338"></a> +<span class="sourceLineNo">339</span> abortWriter(writer);<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> assert finished : "We should have exited the method on all error paths";<a name="line.342"></a> +<span class="sourceLineNo">343</span> assert writer != null : "Writer should be non-null if no error";<a name="line.343"></a> +<span class="sourceLineNo">344</span> return commitWriter(writer, fd, request);<a name="line.344"></a> +<span class="sourceLineNo">345</span> }<a name="line.345"></a> +<span class="sourceLineNo">346</span><a name="line.346"></a> +<span class="sourceLineNo">347</span> protected abstract List<Path> commitWriter(T writer, FileDetails fd,<a name="line.347"></a> +<span class="sourceLineNo">348</span> CompactionRequestImpl request) throws IOException;<a name="line.348"></a> +<span class="sourceLineNo">349</span><a name="line.349"></a> +<span class="sourceLineNo">350</span> protected abstract void abortWriter(T writer) throws IOException;<a name="line.350"></a> +<span class="sourceLineNo">351</span><a name="line.351"></a> +<span class="sourceLineNo">352</span> /**<a name="line.352"></a> +<span class="sourceLineNo">353</span> * Performs the compaction.<a name="line.353"></a> +<span class="sourceLineNo">354</span> * @param fd FileDetails of cell sink writer<a name="line.354"></a> +<span class="sourceLineNo">355</span> * @param scanner Where to read from.<a name="line.355"></a> +<span class="sourceLineNo">356</span> * @param writer Where to write to.<a name="line.356"></a> +<span class="sourceLineNo">357</span> * @param smallestReadPoint Smallest read point.<a name="line.357"></a> +<span class="sourceLineNo">358</span> * @param cleanSeqId When true, remove seqId(used to be mvcc) value which is &lt;=<a name="line.358"></a> +<span class="sourceLineNo">359</span> * smallestReadPoint<a name="line.359"></a> +<span class="sourceLineNo">360</span> * @param major Is a major compaction.<a name="line.360"></a> +<span class="sourceLineNo">361</span> * @param numofFilesToCompact the number of files to compact<a name="line.361"></a> +<span class="sourceLineNo">362</span> * @return Whether compaction ended; false if it was interrupted for some reason.<a name="line.362"></a> +<span class="sourceLineNo">363</span> */<a name="line.363"></a> +<span class="sourceLineNo">364</span> protected boolean performCompaction(FileDetails fd, InternalScanner scanner, CellSink writer,<a name="line.364"></a> +<span class="sourceLineNo">365</span> long smallestReadPoint, boolean cleanSeqId, ThroughputController throughputController,<a name="line.365"></a> +<span class="sourceLineNo">366</span> boolean major, int numofFilesToCompact) throws IOException {<a name="line.366"></a> +<span class="sourceLineNo">367</span> assert writer instanceof ShipperListener;<a name="line.367"></a> +<span class="sourceLineNo">368</span> long bytesWrittenProgressForCloseCheck = 0;<a name="line.368"></a> +<span class="sourceLineNo">369</span> long bytesWrittenProgressForLog = 0;<a name="line.369"></a> +<span class="sourceLineNo">370</span> long bytesWrittenProgressForShippedCall = 0;<a name="line.370"></a> +<span class="sourceLineNo">371</span> // Since scanner.next() can return 'false' but still be delivering data,<a name="line.371"></a> +<span class="sourceLineNo">372</span> // we have to use a do/while loop.<a name="line.372"></a> +<span class="sourceLineNo">373</span> List<Cell> cells = new ArrayList<>();<a name="line.373"></a> +<span class="sourceLineNo">374</span> long closeCheckSizeLimit = HStore.getCloseCheckInterval();<a name="line.374"></a> +<span class="sourceLineNo">375</span> long lastMillis = 0;<a name="line.375"></a> +<span class="sourceLineNo">376</span> if (LOG.isDebugEnabled()) {<a name="line.376"></a> +<span class="sourceLineNo">377</span> lastMillis = EnvironmentEdgeManager.currentTime();<a name="line.377"></a> +<span class="sourceLineNo">378</span> }<a name="line.378"></a> +<span class="sourceLineNo">379</span> String compactionName = ThroughputControlUtil.getNameForThrottling(store, "compaction");<a name="line.379"></a> +<span class="sourceLineNo">380</span> long now = 0;<a name="line.380"></a> +<span class="sourceLineNo">381</span> boolean hasMore;<a name="line.381"></a> +<span class="sourceLineNo">382</span> ScannerContext scannerContext =<a name="line.382"></a> +<span class="sourceLineNo">383</span> ScannerContext.newBuilder().setBatchLimit(compactionKVMax).build();<a name="line.383"></a> +<span class="sourceLineNo">384</span><a name="line.384"></a> +<span class="sourceLineNo">385</span> throughputController.start(compactionName);<a name="line.385"></a> +<span class="sourceLineNo">386</span> KeyValueScanner kvs = (scanner instanceof KeyValueScanner)? (KeyValueScanner)scanner : null;<a name="line.386"></a> +<span class="sourceLineNo">387</span> long shippedCallSizeLimit = (long) numofFilesToCompact * this.store.getColumnFamilyDescriptor().getBlocksize();<a name="line.387"></a> +<span class="sourceLineNo">388</span> try {<a name="line.388"></a> +<span class="sourceLineNo">389</span> do {<a name="line.389"></a> +<span class="sourceLineNo">390</span> hasMore = scanner.next(cells, scannerContext);<a name="line.390"></a> +<span class="sourceLineNo">391</span> if (LOG.isDebugEnabled()) {<a name="line.391"></a> +<span class="sourceLineNo">392</span> now = EnvironmentEdgeManager.currentTime();<a name="line.392"></a> +<span class="sourceLineNo">393</span> }<a name="line.393"></a> +<span class="sourceLineNo">394</span> // output to writer:<a name="line.394"></a> +<span class="sourceLineNo">395</span> Cell lastCleanCell = null;<a name="line.395"></a> +<span class="sourceLineNo">396</span> long lastCleanCellSeqId = 0;<a name="line.396"></a> +<span class="sourceLineNo">397</span> for (Cell c : cells) {<a name="line.397"></a> +<span class="sourceLineNo">398</span> if (cleanSeqId && c.getSequenceId() <= smallestReadPoint) {<a name="line.398"></a> +<span class="sourceLineNo">399</span> lastCleanCell = c;<a name="line.399"></a> +<span class="sourceLineNo">400</span> lastCleanCellSeqId = c.getSequenceId();<a name="line.400"></a> +<span class="sourceLineNo">401</span> PrivateCellUtil.setSequenceId(c, 0);<a name="line.401"></a> +<span class="sourceLineNo">402</span> } else {<a name="line.402"></a> +<span class="sourceLineNo">403</span> lastCleanCell = null;<a name="line.403"></a> +<span class="sourceLineNo">404</span> lastCleanCellSeqId = 0;<a name="line.404"></a> +<span class="sourceLineNo">405</span> }<a name="line.405"></a> +<span class="sourceLineNo">406</span> writer.append(c);<a name="line.406"></a> +<span class="sourceLineNo">407</span> int len = KeyValueUtil.length(c);<a name="line.407"></a> +<span class="sourceLineNo">408</span> ++progress.currentCompactedKVs;<a name="line.408"></a> +<span class="sourceLineNo">409</span> progress.totalCompactedSize += len;<a name="line.409"></a> +<span class="sourceLineNo">410</span> bytesWrittenProgressForShippedCall += len;<a name="line.410"></a> +<span class="sourceLineNo">411</span> if (LOG.isDebugEnabled()) {<a name="line.411"></a> +<span class="sourceLineNo">412</span> bytesWrittenProgressForLog += len;<a name="line.412"></a> +<span class="sourceLineNo">413</span> }<a name="line.413"></a> +<span class="sourceLineNo">414</span> throughputController.control(compactionName, len);<a name="line.414"></a> +<span class="sourceLineNo">415</span> // check periodically to see if a system stop is requested<a name="line.415"></a> +<span class="sourceLineNo">416</span> if (closeCheckSizeLimit > 0) {<a name="line.416"></a> +<span class="sourceLineNo">417</span> bytesWrittenProgressForCloseCheck += len;<a name="line.417"></a> +<span class="sourceLineNo">418</span> if (bytesWrittenProgressForCloseCheck > closeCheckSizeLimit) {<a name="line.418"></a> +<span class="sourceLineNo">419</span> bytesWrittenProgressForCloseCheck = 0;<a name="line.419"></a> +<span class="sourceLineNo">420</span> if (!store.areWritesEnabled()) {<a name="line.420"></a> +<span class="sourceLineNo">421</span> progress.cancel();<a name="line.421"></a> +<span class="sourceLineNo">422</span> return false;<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> }<a name="line.425"></a> +<span class="sourceLineNo">426</span> if (kvs != null && bytesWrittenProgressForShippedCall > shippedCallSizeLimit) {<a name="line.426"></a> +<span class="sourceLineNo">427</span> if (lastCleanCell != null) {<a name="line.427"></a> +<span class="sourceLineNo">428</span> // HBASE-16931, set back sequence id to avoid affecting scan order unexpectedly.<a name="line.428"></a> +<span class="sourceLineNo">429</span> // ShipperListener will do a clone of the last cells it refer, so need to set back<a name="line.429"></a> +<span class="sourceLineNo">430</span> // sequence id before ShipperListener.beforeShipped<a name="line.430"></a> +<span class="sourceLineNo">431</span> PrivateCellUtil.setSequenceId(lastCleanCell, lastCleanCellSeqId);<a name="line.431"></a> +<span class="sourceLineNo">432</span> }<a name="line.432"></a> +<span class="sourceLineNo">433</span> // Clone the cells that are in the writer so that they are freed of references,<a name="line.433"></a> +<span class="sourceLineNo">434</span> // if they are holding any.<a name="line.434"></a> +<span class="sourceLineNo">435</span> ((ShipperListener)writer).beforeShipped();<a name="line.435"></a> +<span class="sourceLineNo">436</span> // The SHARED block references, being read for compaction, will be kept in prevBlocks<a name="line.436"></a> +<span class="sourceLineNo">437</span> // list(See HFileScannerImpl#prevBlocks). In case of scan flow, after each set of cells<a name="line.437"></a> +<span class="sourceLineNo">438</span> // being returned to client, we will call shipped() which can clear this list. Here by<a name="line.438"></a> +<span class="sourceLineNo">439</span> // we are doing the similar thing. In between the compaction (after every N cells<a name="line.439"></a> +<span class="sourceLineNo">440</span> // written with collective size of 'shippedCallSizeLimit') we will call shipped which<a name="line.440"></a> +<span class="sourceLineNo">441</span> // may clear prevBlocks list.<a name="line.441"></a> +<span class="sourceLineNo">442</span> kvs.shipped();<a name="line.442"></a> +<span class="sourceLineNo">443</span> bytesWrittenProgressForShippedCall = 0;<a name="line.443"></a> +<span class="sourceLineNo">444</span> }<a name="line.444"></a> +<span class="sourceLineNo">445</span> }<a name="line.445"></a> +<span class="sourceLineNo">446</span> if (lastCleanCell != null) {<a name="line.446"></a> +<span class="sourceLineNo">447</span> // HBASE-16931, set back sequence id to avoid affecting scan order unexpectedly<a name="line.447"></a> +<span class="sourceLineNo">448</span> PrivateCellUtil.setSequenceId(lastCleanCell, lastCleanCellSeqId);<a name="line.448"></a> <span class="sourceLineNo">449</span> }<a name="line.449"></a> -<span class="sourceLineNo">450</span> if (lastCleanCell != null) {<a name="line.450"></a> -<span class="sourceLineNo">451</span> // HBASE-16931, set back sequence id to avoid affecting scan order unexpectedly<a name="line.451"></a> -<span class="sourceLineNo">452</span> PrivateCellUtil.setSequenceId(lastCleanCell, lastCleanCellSeqId);<a name="line.452"></a> -<span class="sourceLineNo">453</span> }<a name="line.453"></a> -<span class="sourceLineNo">454</span> // Log the progress of long running compactions every minute if<a name="line.454"></a> -<span class="sourceLineNo">455</span> // logging at DEBUG level<a name="line.455"></a> -<span class="sourceLineNo">456</span> if (LOG.isDebugEnabled()) {<a name="line.456"></a> -<span class="sourceLineNo">457</span> if ((now - lastMillis) >= COMPACTION_PROGRESS_LOG_INTERVAL) {<a name="line.457"></a> -<span class="sourceLineNo">458</span> LOG.debug("Compaction progress: "<a name="line.458"></a> -<span class="sourceLineNo">459</span> + compactionName<a name="line.459"></a> -<span class="sourceLineNo">460</span> + " "<a name="line.460"></a> -<span class="sourceLineNo">461</span> + progress<a name="line.461"></a> -<span class="sourceLineNo">462</span> + String.format(", rate=%.2f kB/sec", (bytesWrittenProgressForLog / 1024.0)<a name="line.462"></a> -<span class="sourceLineNo">463</span> / ((now - lastMillis) / 1000.0)) + ", throughputController is "<a name="line.463"></a> -<span class="sourceLineNo">464</span> + throughputController);<a name="line.464"></a> -<span class="sourceLineNo">465</span> lastMillis = now;<a name="line.465"></a> -<span class="sourceLineNo">466</span> bytesWrittenProgressForLog = 0;<a name="line.466"></a> -<span class="sourceLineNo">467</span> }<a name="line.467"></a> -<span class="sourceLineNo">468</span> }<a name="line.468"></a> -<span class="sourceLineNo">469</span> cells.clear();<a name="line.469"></a> -<span class="sourceLineNo">470</span> } while (hasMore);<a name="line.470"></a> -<span class="sourceLineNo">471</span> } catch (InterruptedException e) {<a name="line.471"></a> -<span class="sourceLineNo">472</span> progress.cancel();<a name="line.472"></a> -<span class="sourceLineNo">473</span> throw new InterruptedIOException("Interrupted while control throughput of compacting "<a name="line.473"></a> -<span class="sourceLineNo">474</span> + compactionName);<a name="line.474"></a> -<span class="sourceLineNo">475</span> } finally {<a name="line.475"></a> -<span class="sourceLineNo">476</span> throughputController.finish(compactionName);<a name="line.476"></a> -<span class="sourceLineNo">477</span> }<a name="line.477"></a> -<span class="sourceLineNo">478</span> progress.complete();<a name="line.478"></a> -<span class="sourceLineNo">479</span> return true;<a name="line.479"></a> -<span class="sourceLineNo">480</span> }<a name="line.480"></a> -<span class="sourceLineNo">481</span><a name="line.481"></a> -<span class="sourceLineNo">482</span> /**<a name="line.482"></a> -<span class="sourceLineNo">483</span> * @param store store<a name="line.483"></a> -<span class="sourceLineNo">484</span> * @param scanners Store file scanners.<a name="line.484"></a> -<span class="sourceLineNo">485</span> * @param scanType Scan type.<a name="line.485"></a> -<span class="sourceLineNo">486</span> * @param smallestReadPoint Smallest MVCC read point.<a name="line.486"></a> -<span class="sourceLineNo">487</span> * @param earliestPutTs Earliest put across all files.<a name="line.487"></a> -<span class="sourceLineNo">488</span> * @return A compaction scanner.<a name="line.488"></a> -<span class="sourceLineNo">489</span> */<a name="line.489"></a> -<span class="sourceLineNo">490</span> protected InternalScanner createScanner(HStore store, ScanInfo scanInfo,<a name="line.490"></a> -<span class="sourceLineNo">491</span> List<StoreFileScanner> scanners, ScanType scanType, long smallestReadPoint,<a name="line.491"></a> -<span class="sourceLineNo">492</span> long earliestPutTs) throws IOException {<a name="line.492"></a> -<span class="sourceLineNo">493</span> return new StoreScanner(store, scanInfo, scanners, scanType, smallestReadPoint, earliestPutTs);<a name="line.493"></a> -<span class="sourceLineNo">494</span> }<a name="line.494"></a> -<span class="sourceLineNo">495</span><a name="line.495"></a> -<span class="sourceLineNo">496</span> /**<a name="line.496"></a> -<span class="sourceLineNo">497</span> * @param store The store.<a name="line.497"></a> -<span class="sourceLineNo">498</span> * @param scanners Store file scanners.<a name="line.498"></a> -<span class="sourceLineNo">499</span> * @param smallestReadPoint Smallest MVCC read point.<a name="line.499"></a> -<span class="sourceLineNo">500</span> * @param earliestPutTs Earliest put across all files.<a name="line.500"></a> -<span class="sourceLineNo">501</span> * @param dropDeletesFromRow Drop deletes starting with this row, inclusive. Can be null.<a name="line.501"></a> -<span class="sourceLineNo">502</span> * @param dropDeletesToRow Drop deletes ending with this row, exclusive. Can be null.<a name="line.502"></a> -<span class="sourceLineNo">503</span> * @return A compaction scanner.<a name="line.503"></a> -<span class="sourceLineNo">504</span> */<a name="line.504"></a> -<span class="sourceLineNo">505</span> protected InternalScanner createScanner(HStore store, ScanInfo scanInfo,<a name="line.505"></a> -<span class="sourceLineNo">506</span> List<StoreFileScanner> scanners, long smallestReadPoint, long earliestPutTs,<a name="line.506"></a> -<span class="sourceLineNo">507</span> byte[] dropDeletesFromRow, byte[] dropDeletesToRow) throws IOException {<a name="line.507"></a> -<span class="sourceLineNo">508</span> return new StoreScanner(store, scanInfo, scanners, smallestReadPoint, earliestPutTs,<a name="line.508"></a> -<span class="sourceLineNo">509</span> dropDeletesFromRow, dropDeletesToRow);<a name="line.509"></a> -<span class="sourceLineNo">510</span> }<a name="line.510"></a> -<span class="sourceLineNo">511</span>}<a name="line.511"></a> +<span class="sourceLineNo">450</span> // Log the progress of long running compactions every minute if<a name="line.450"></a> +<span class="sourceLineNo">451</span> // logging at DEBUG level<a name="line.451"></a> +<span class="sourceLineNo">452</span> if (LOG.isDebugEnabled()) {<a name="line.452"></a> +<span class="sourceLineNo">453</span> if ((now - lastMillis) >= COMPACTION_PROGRESS_LOG_INTERVAL) {<a name="line.453"></a> +<span class="sourceLineNo">454</span> LOG.debug("Compaction progress: "<a name="line.454"></a> +<span class="sourceLineNo">455</span> + compactionName<a name="line.455"></a> +<span class="sourceLineNo">456</span> + " "<a name="line.456"></a> +<span class="sourceLineNo">457</span> + progress<a name="line.457"></a> +<span class="sourceLineNo">458</span> + String.format(", rate=%.2f kB/sec", (bytesWrittenProgressForLog / 1024.0)<a name="line.458"></a> +<span class="sourceLineNo">459</span> / ((now - lastMillis) / 1000.0)) + ", throughputController is "<a name="line.459"></a> +<span class="sourceLineNo">460</span> + throughputController);<a name="line.460"></a> +<span class="sourceLineNo">461</span> lastMillis = now;<a name="line.461"></a> +<span class="sourceLineNo">462</span> bytesWrittenProgressForLog = 0;<a name="line.462"></a> +<span class="sourceLineNo">463</span> }<a name="line.463"></a> +<span class="sourceLineNo">464</span> }<a name="line.464"></a> +<span class="sourceLineNo">465</span> cells.clear();<a name="line.465"></a> +<span class="sourceLineNo">466</span> } while (hasMore);<a name="line.466"></a> +<span class="sourceLineNo">467</span> } catch (InterruptedException e) {<a name="line.467"></a> +<span class="sourceLineNo">468</span> progress.cancel();<a name="line.468"></a> +<span class="sourceLineNo">469</span> throw new InterruptedIOException("Interrupted while control throughput of compacting "<a name="line.469"></a> +<span class="sourceLineNo">470</span> + compactionName);<a name="line.470"></a> +<span class="sourceLineNo">471</span> } finally {<a name="line.471"></a> +<span class="sourceLineNo">472</span> throughputController.finish(compactionName);<a name="line.472"></a> +<span class="sourceLineNo">473</span> }<a name="line.473"></a> +<span class="sourceLineNo">474</span> progress.complete();<a name="line.474"></a> +<span class="sourceLineNo">475</span> return true;<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> /**<a name="line.478"></a> +<span class="sourceLineNo">479</span> * @param store store<a name="line.479"></a> +<span class="sourceLineNo">480</span> * @param scanners Store file scanners.<a name="line.480"></a> +<span class="sourceLineNo">481</span> * @param scanType Scan type.<a name="line.481"></a> +<span class="sourceLineNo">482</span> * @param smallestReadPoint Smallest MVCC read point.<a name="line.482"></a> +<span class="sourceLineNo">483</span> * @param earliestPutTs Earliest put across all files.<a name="line.483"></a> +<span class="sourceLineNo">484</span> * @return A compaction scanner.<a name="line.484"></a> +<span class="sourceLineNo">485</span> */<a name="line.485"></a> +<span class="sourceLineNo">486</span> protected InternalScanner createScanner(HStore store, ScanInfo scanInfo,<a name="line.486"></a> +<span class="sourceLineNo">487</span> List<StoreFileScanner> scanners, ScanType scanType, long smallestReadPoint,<a name="line.487"></a> +<span class="sourceLineNo">488</span> long earliestPutTs) throws IOException {<a name="line.488"></a> +<span class="sourceLineNo">489</span> return new StoreScanner(store, scanInfo, scanners, scanType, smallestReadPoint, earliestPutTs);<a name="line.489"></a> +<span class="sourceLineNo">490</span> }<a name="line.490"></a> +<span class="sourceLineNo">491</span><a name="line.491"></a> +<span class="sourceLineNo">492</span> /**<a name="line.492"></a> +<span class="sourceLineNo">493</span> * @param store The store.<a name="line.493"></a> +<span class="sourceLineNo">494</span> * @param scanners Store file scanners.<a name="line.494"></a> +<span class="sourceLineNo">495</span> * @param smallestReadPoint Smallest MVCC read point.<a name="line.495"></a> +<span class="sourceLineNo">496</span> * @param earliestPutTs Earliest put across all files.<a name="line.496"></a> +<span class="sourceLineNo">497</span> * @param dropDeletesFromRow Drop deletes starting with this row, inclusive. Can be null.<a name="line.497"></a> +<span class="sourceLineNo">498</span> * @param dropDeletesToRow Drop deletes ending with this row, exclusive. Can be null.<a name="line.498"></a> +<span class="sourceLineNo">499</span> * @return A compaction scanner.<a name="line.499"></a> +<span class="sourceLineNo">500</span> */<a name="line.500"></a> +<span class="sourceLineNo">501</span> protected InternalScanner createScanner(HStore store, ScanInfo scanInfo,<a name="line.501"></a> +<span class="sourceLineNo">502</span> List<StoreFileScanner> scanners, long smallestReadPoint, long earliestPutTs,<a name="line.502"></a> +<span class="sourceLineNo">503</span> byte[] dropDeletesFromRow, byte[] dropDeletesToRow) throws IOException {<a name="line.503"></a> +<span class="sourceLineNo">504</span> return new StoreScanner(store, scanInfo, scanners, smallestReadPoint, earliestPutTs,<a name="line.504"></a> +<span class="sourceLineNo">505</span> dropDeletesFromRow, dropDeletesToRow);<a name="line.505"></a> +<span class="sourceLineNo">506</span> }<a name="line.506"></a> +<span class="sourceLineNo">507</span>}<a name="line.507"></a>
