You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by gi...@apache.org on 2018/11/05 14:53:03 UTC
[04/13] hbase-site git commit: Published site at
c8574ba3c52274ed5a93e46f7af30dd8b46fb878.
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/74ea8168/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALProcedure.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALProcedure.html b/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALProcedure.html
index d98263b..18f1cc8 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALProcedure.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALProcedure.html
@@ -26,150 +26,143 @@
<span class="sourceLineNo">018</span>package org.apache.hadoop.hbase.master.replication;<a name="line.18"></a>
<span class="sourceLineNo">019</span><a name="line.19"></a>
<span class="sourceLineNo">020</span>import java.io.IOException;<a name="line.20"></a>
-<span class="sourceLineNo">021</span>import java.util.ArrayList;<a name="line.21"></a>
-<span class="sourceLineNo">022</span>import java.util.List;<a name="line.22"></a>
-<span class="sourceLineNo">023</span><a name="line.23"></a>
-<span class="sourceLineNo">024</span>import org.apache.hadoop.hbase.ServerName;<a name="line.24"></a>
-<span class="sourceLineNo">025</span>import org.apache.hadoop.hbase.master.procedure.MasterProcedureEnv;<a name="line.25"></a>
-<span class="sourceLineNo">026</span>import org.apache.hadoop.hbase.procedure2.ProcedureStateSerializer;<a name="line.26"></a>
-<span class="sourceLineNo">027</span>import org.apache.hadoop.hbase.procedure2.ProcedureSuspendedException;<a name="line.27"></a>
-<span class="sourceLineNo">028</span>import org.apache.hadoop.hbase.procedure2.ProcedureUtil;<a name="line.28"></a>
-<span class="sourceLineNo">029</span>import org.apache.hadoop.hbase.replication.ReplicationException;<a name="line.29"></a>
-<span class="sourceLineNo">030</span>import org.apache.yetus.audience.InterfaceAudience;<a name="line.30"></a>
-<span class="sourceLineNo">031</span>import org.slf4j.Logger;<a name="line.31"></a>
-<span class="sourceLineNo">032</span>import org.slf4j.LoggerFactory;<a name="line.32"></a>
-<span class="sourceLineNo">033</span><a name="line.33"></a>
-<span class="sourceLineNo">034</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.SyncReplicationReplayWALState;<a name="line.34"></a>
-<span class="sourceLineNo">035</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.SyncReplicationReplayWALStateData;<a name="line.35"></a>
-<span class="sourceLineNo">036</span><a name="line.36"></a>
-<span class="sourceLineNo">037</span>@InterfaceAudience.Private<a name="line.37"></a>
-<span class="sourceLineNo">038</span>public class SyncReplicationReplayWALProcedure<a name="line.38"></a>
-<span class="sourceLineNo">039</span> extends AbstractPeerNoLockProcedure<SyncReplicationReplayWALState> {<a name="line.39"></a>
-<span class="sourceLineNo">040</span><a name="line.40"></a>
-<span class="sourceLineNo">041</span> private static final Logger LOG =<a name="line.41"></a>
-<span class="sourceLineNo">042</span> LoggerFactory.getLogger(SyncReplicationReplayWALProcedure.class);<a name="line.42"></a>
+<span class="sourceLineNo">021</span>import java.util.List;<a name="line.21"></a>
+<span class="sourceLineNo">022</span>import org.apache.hadoop.hbase.ServerName;<a name="line.22"></a>
+<span class="sourceLineNo">023</span>import org.apache.hadoop.hbase.master.procedure.MasterProcedureEnv;<a name="line.23"></a>
+<span class="sourceLineNo">024</span>import org.apache.hadoop.hbase.procedure2.ProcedureStateSerializer;<a name="line.24"></a>
+<span class="sourceLineNo">025</span>import org.apache.hadoop.hbase.procedure2.ProcedureSuspendedException;<a name="line.25"></a>
+<span class="sourceLineNo">026</span>import org.apache.hadoop.hbase.procedure2.ProcedureUtil;<a name="line.26"></a>
+<span class="sourceLineNo">027</span>import org.apache.yetus.audience.InterfaceAudience;<a name="line.27"></a>
+<span class="sourceLineNo">028</span>import org.slf4j.Logger;<a name="line.28"></a>
+<span class="sourceLineNo">029</span>import org.slf4j.LoggerFactory;<a name="line.29"></a>
+<span class="sourceLineNo">030</span><a name="line.30"></a>
+<span class="sourceLineNo">031</span>import org.apache.hadoop.hbase.shaded.protobuf.ProtobufUtil;<a name="line.31"></a>
+<span class="sourceLineNo">032</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.SyncReplicationReplayWALState;<a name="line.32"></a>
+<span class="sourceLineNo">033</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.SyncReplicationReplayWALStateData;<a name="line.33"></a>
+<span class="sourceLineNo">034</span><a name="line.34"></a>
+<span class="sourceLineNo">035</span>/**<a name="line.35"></a>
+<span class="sourceLineNo">036</span> * The procedure for replaying a set of remote wals. It will get an available region server and<a name="line.36"></a>
+<span class="sourceLineNo">037</span> * schedule a {@link SyncReplicationReplayWALRemoteProcedure} to actually send the request to region<a name="line.37"></a>
+<span class="sourceLineNo">038</span> * server.<a name="line.38"></a>
+<span class="sourceLineNo">039</span> */<a name="line.39"></a>
+<span class="sourceLineNo">040</span>@InterfaceAudience.Private<a name="line.40"></a>
+<span class="sourceLineNo">041</span>public class SyncReplicationReplayWALProcedure<a name="line.41"></a>
+<span class="sourceLineNo">042</span> extends AbstractPeerNoLockProcedure<SyncReplicationReplayWALState> {<a name="line.42"></a>
<span class="sourceLineNo">043</span><a name="line.43"></a>
-<span class="sourceLineNo">044</span> private ServerName worker = null;<a name="line.44"></a>
-<span class="sourceLineNo">045</span><a name="line.45"></a>
-<span class="sourceLineNo">046</span> private List<String> wals;<a name="line.46"></a>
-<span class="sourceLineNo">047</span><a name="line.47"></a>
-<span class="sourceLineNo">048</span> public SyncReplicationReplayWALProcedure() {<a name="line.48"></a>
-<span class="sourceLineNo">049</span> }<a name="line.49"></a>
+<span class="sourceLineNo">044</span> private static final Logger LOG =<a name="line.44"></a>
+<span class="sourceLineNo">045</span> LoggerFactory.getLogger(SyncReplicationReplayWALProcedure.class);<a name="line.45"></a>
+<span class="sourceLineNo">046</span><a name="line.46"></a>
+<span class="sourceLineNo">047</span> private ServerName worker = null;<a name="line.47"></a>
+<span class="sourceLineNo">048</span><a name="line.48"></a>
+<span class="sourceLineNo">049</span> private List<String> wals;<a name="line.49"></a>
<span class="sourceLineNo">050</span><a name="line.50"></a>
-<span class="sourceLineNo">051</span> public SyncReplicationReplayWALProcedure(String peerId, List<String> wals) {<a name="line.51"></a>
-<span class="sourceLineNo">052</span> this.peerId = peerId;<a name="line.52"></a>
-<span class="sourceLineNo">053</span> this.wals = wals;<a name="line.53"></a>
-<span class="sourceLineNo">054</span> }<a name="line.54"></a>
-<span class="sourceLineNo">055</span><a name="line.55"></a>
-<span class="sourceLineNo">056</span> @Override<a name="line.56"></a>
-<span class="sourceLineNo">057</span> protected Flow executeFromState(MasterProcedureEnv env, SyncReplicationReplayWALState state)<a name="line.57"></a>
-<span class="sourceLineNo">058</span> throws ProcedureSuspendedException {<a name="line.58"></a>
-<span class="sourceLineNo">059</span> SyncReplicationReplayWALManager syncReplicationReplayWALManager =<a name="line.59"></a>
-<span class="sourceLineNo">060</span> env.getMasterServices().getSyncReplicationReplayWALManager();<a name="line.60"></a>
-<span class="sourceLineNo">061</span> switch (state) {<a name="line.61"></a>
-<span class="sourceLineNo">062</span> case ASSIGN_WORKER:<a name="line.62"></a>
-<span class="sourceLineNo">063</span> try {<a name="line.63"></a>
-<span class="sourceLineNo">064</span> worker = syncReplicationReplayWALManager.getPeerWorker(peerId);<a name="line.64"></a>
-<span class="sourceLineNo">065</span> } catch (ReplicationException e) {<a name="line.65"></a>
-<span class="sourceLineNo">066</span> long backoff = ProcedureUtil.getBackoffTimeMs(attempts);<a name="line.66"></a>
-<span class="sourceLineNo">067</span> LOG.warn("Failed to get worker to replay wals {} for peer id={}, sleep {} secs and retry",<a name="line.67"></a>
-<span class="sourceLineNo">068</span> wals, peerId, backoff / 1000, e);<a name="line.68"></a>
-<span class="sourceLineNo">069</span> throw suspend(backoff);<a name="line.69"></a>
-<span class="sourceLineNo">070</span> }<a name="line.70"></a>
-<span class="sourceLineNo">071</span> if (worker == null) {<a name="line.71"></a>
-<span class="sourceLineNo">072</span> long backoff = ProcedureUtil.getBackoffTimeMs(attempts);<a name="line.72"></a>
-<span class="sourceLineNo">073</span> LOG.info("No worker to replay wals {} for peer id={}, sleep {} secs and retry", wals,<a name="line.73"></a>
-<span class="sourceLineNo">074</span> peerId, backoff / 1000);<a name="line.74"></a>
-<span class="sourceLineNo">075</span> throw suspend(backoff);<a name="line.75"></a>
-<span class="sourceLineNo">076</span> }<a name="line.76"></a>
-<span class="sourceLineNo">077</span> attempts = 0;<a name="line.77"></a>
-<span class="sourceLineNo">078</span> setNextState(SyncReplicationReplayWALState.DISPATCH_WALS_TO_WORKER);<a name="line.78"></a>
-<span class="sourceLineNo">079</span> return Flow.HAS_MORE_STATE;<a name="line.79"></a>
-<span class="sourceLineNo">080</span> case DISPATCH_WALS_TO_WORKER:<a name="line.80"></a>
-<span class="sourceLineNo">081</span> addChildProcedure(new SyncReplicationReplayWALRemoteProcedure(peerId, wals, worker));<a name="line.81"></a>
-<span class="sourceLineNo">082</span> setNextState(SyncReplicationReplayWALState.RELEASE_WORKER);<a name="line.82"></a>
-<span class="sourceLineNo">083</span> return Flow.HAS_MORE_STATE;<a name="line.83"></a>
-<span class="sourceLineNo">084</span> case RELEASE_WORKER:<a name="line.84"></a>
-<span class="sourceLineNo">085</span> boolean finished = false;<a name="line.85"></a>
-<span class="sourceLineNo">086</span> try {<a name="line.86"></a>
-<span class="sourceLineNo">087</span> finished = syncReplicationReplayWALManager.isReplayWALFinished(wals.get(0));<a name="line.87"></a>
-<span class="sourceLineNo">088</span> } catch (IOException e) {<a name="line.88"></a>
-<span class="sourceLineNo">089</span> long backoff = ProcedureUtil.getBackoffTimeMs(attempts);<a name="line.89"></a>
-<span class="sourceLineNo">090</span> LOG.warn("Failed to check whether replay wals {} finished for peer id={}" +<a name="line.90"></a>
-<span class="sourceLineNo">091</span> ", sleep {} secs and retry",<a name="line.91"></a>
-<span class="sourceLineNo">092</span> wals, peerId, backoff / 1000, e);<a name="line.92"></a>
-<span class="sourceLineNo">093</span> throw suspend(backoff);<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> syncReplicationReplayWALManager.removePeerWorker(peerId, worker);<a name="line.96"></a>
-<span class="sourceLineNo">097</span> } catch (ReplicationException e) {<a name="line.97"></a>
-<span class="sourceLineNo">098</span> long backoff = ProcedureUtil.getBackoffTimeMs(attempts);<a name="line.98"></a>
-<span class="sourceLineNo">099</span> LOG.warn("Failed to remove worker {} for peer id={}, sleep {} secs and retry", worker,<a name="line.99"></a>
-<span class="sourceLineNo">100</span> peerId, backoff / 1000, e);<a name="line.100"></a>
-<span class="sourceLineNo">101</span> throw suspend(backoff);<a name="line.101"></a>
-<span class="sourceLineNo">102</span> }<a name="line.102"></a>
-<span class="sourceLineNo">103</span> attempts = 0;<a name="line.103"></a>
-<span class="sourceLineNo">104</span> if (!finished) {<a name="line.104"></a>
-<span class="sourceLineNo">105</span> LOG.warn("Failed to replay wals {} for peer id={}, retry", wals, peerId);<a name="line.105"></a>
-<span class="sourceLineNo">106</span> setNextState(SyncReplicationReplayWALState.ASSIGN_WORKER);<a name="line.106"></a>
-<span class="sourceLineNo">107</span> return Flow.HAS_MORE_STATE;<a name="line.107"></a>
-<span class="sourceLineNo">108</span> }<a name="line.108"></a>
-<span class="sourceLineNo">109</span> return Flow.NO_MORE_STATE;<a name="line.109"></a>
-<span class="sourceLineNo">110</span> default:<a name="line.110"></a>
-<span class="sourceLineNo">111</span> throw new UnsupportedOperationException("unhandled state=" + state);<a name="line.111"></a>
-<span class="sourceLineNo">112</span> }<a name="line.112"></a>
+<span class="sourceLineNo">051</span> public SyncReplicationReplayWALProcedure() {<a name="line.51"></a>
+<span class="sourceLineNo">052</span> }<a name="line.52"></a>
+<span class="sourceLineNo">053</span><a name="line.53"></a>
+<span class="sourceLineNo">054</span> public SyncReplicationReplayWALProcedure(String peerId, List<String> wals) {<a name="line.54"></a>
+<span class="sourceLineNo">055</span> this.peerId = peerId;<a name="line.55"></a>
+<span class="sourceLineNo">056</span> this.wals = wals;<a name="line.56"></a>
+<span class="sourceLineNo">057</span> }<a name="line.57"></a>
+<span class="sourceLineNo">058</span><a name="line.58"></a>
+<span class="sourceLineNo">059</span> @Override<a name="line.59"></a>
+<span class="sourceLineNo">060</span> protected Flow executeFromState(MasterProcedureEnv env, SyncReplicationReplayWALState state)<a name="line.60"></a>
+<span class="sourceLineNo">061</span> throws ProcedureSuspendedException {<a name="line.61"></a>
+<span class="sourceLineNo">062</span> SyncReplicationReplayWALManager syncReplicationReplayWALManager =<a name="line.62"></a>
+<span class="sourceLineNo">063</span> env.getMasterServices().getSyncReplicationReplayWALManager();<a name="line.63"></a>
+<span class="sourceLineNo">064</span> switch (state) {<a name="line.64"></a>
+<span class="sourceLineNo">065</span> case ASSIGN_WORKER:<a name="line.65"></a>
+<span class="sourceLineNo">066</span> worker = syncReplicationReplayWALManager.acquirePeerWorker(peerId, this);<a name="line.66"></a>
+<span class="sourceLineNo">067</span> setNextState(SyncReplicationReplayWALState.DISPATCH_WALS_TO_WORKER);<a name="line.67"></a>
+<span class="sourceLineNo">068</span> return Flow.HAS_MORE_STATE;<a name="line.68"></a>
+<span class="sourceLineNo">069</span> case DISPATCH_WALS_TO_WORKER:<a name="line.69"></a>
+<span class="sourceLineNo">070</span> addChildProcedure(new SyncReplicationReplayWALRemoteProcedure(peerId, wals, worker));<a name="line.70"></a>
+<span class="sourceLineNo">071</span> setNextState(SyncReplicationReplayWALState.RELEASE_WORKER);<a name="line.71"></a>
+<span class="sourceLineNo">072</span> return Flow.HAS_MORE_STATE;<a name="line.72"></a>
+<span class="sourceLineNo">073</span> case RELEASE_WORKER:<a name="line.73"></a>
+<span class="sourceLineNo">074</span> boolean finished = false;<a name="line.74"></a>
+<span class="sourceLineNo">075</span> try {<a name="line.75"></a>
+<span class="sourceLineNo">076</span> finished = syncReplicationReplayWALManager.isReplayWALFinished(wals.get(0));<a name="line.76"></a>
+<span class="sourceLineNo">077</span> } catch (IOException e) {<a name="line.77"></a>
+<span class="sourceLineNo">078</span> long backoff = ProcedureUtil.getBackoffTimeMs(attempts);<a name="line.78"></a>
+<span class="sourceLineNo">079</span> LOG.warn("Failed to check whether replay wals {} finished for peer id={}" +<a name="line.79"></a>
+<span class="sourceLineNo">080</span> ", sleep {} secs and retry", wals, peerId, backoff / 1000, e);<a name="line.80"></a>
+<span class="sourceLineNo">081</span> throw suspend(backoff);<a name="line.81"></a>
+<span class="sourceLineNo">082</span> }<a name="line.82"></a>
+<span class="sourceLineNo">083</span> syncReplicationReplayWALManager.releasePeerWorker(peerId, worker,<a name="line.83"></a>
+<span class="sourceLineNo">084</span> env.getProcedureScheduler());<a name="line.84"></a>
+<span class="sourceLineNo">085</span> if (!finished) {<a name="line.85"></a>
+<span class="sourceLineNo">086</span> LOG.warn("Failed to replay wals {} for peer id={}, retry", wals, peerId);<a name="line.86"></a>
+<span class="sourceLineNo">087</span> setNextState(SyncReplicationReplayWALState.ASSIGN_WORKER);<a name="line.87"></a>
+<span class="sourceLineNo">088</span> return Flow.HAS_MORE_STATE;<a name="line.88"></a>
+<span class="sourceLineNo">089</span> }<a name="line.89"></a>
+<span class="sourceLineNo">090</span> return Flow.NO_MORE_STATE;<a name="line.90"></a>
+<span class="sourceLineNo">091</span> default:<a name="line.91"></a>
+<span class="sourceLineNo">092</span> throw new UnsupportedOperationException("unhandled state=" + state);<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><a name="line.95"></a>
+<span class="sourceLineNo">096</span> @Override<a name="line.96"></a>
+<span class="sourceLineNo">097</span> protected void rollbackState(MasterProcedureEnv env, SyncReplicationReplayWALState state)<a name="line.97"></a>
+<span class="sourceLineNo">098</span> throws IOException, InterruptedException {<a name="line.98"></a>
+<span class="sourceLineNo">099</span> if (state == getInitialState()) {<a name="line.99"></a>
+<span class="sourceLineNo">100</span> return;<a name="line.100"></a>
+<span class="sourceLineNo">101</span> }<a name="line.101"></a>
+<span class="sourceLineNo">102</span> throw new UnsupportedOperationException();<a name="line.102"></a>
+<span class="sourceLineNo">103</span> }<a name="line.103"></a>
+<span class="sourceLineNo">104</span><a name="line.104"></a>
+<span class="sourceLineNo">105</span> @Override<a name="line.105"></a>
+<span class="sourceLineNo">106</span> protected SyncReplicationReplayWALState getState(int state) {<a name="line.106"></a>
+<span class="sourceLineNo">107</span> return SyncReplicationReplayWALState.forNumber(state);<a name="line.107"></a>
+<span class="sourceLineNo">108</span> }<a name="line.108"></a>
+<span class="sourceLineNo">109</span><a name="line.109"></a>
+<span class="sourceLineNo">110</span> @Override<a name="line.110"></a>
+<span class="sourceLineNo">111</span> protected int getStateId(SyncReplicationReplayWALState state) {<a name="line.111"></a>
+<span class="sourceLineNo">112</span> return state.getNumber();<a name="line.112"></a>
<span class="sourceLineNo">113</span> }<a name="line.113"></a>
<span class="sourceLineNo">114</span><a name="line.114"></a>
<span class="sourceLineNo">115</span> @Override<a name="line.115"></a>
-<span class="sourceLineNo">116</span> protected void rollbackState(MasterProcedureEnv env,<a name="line.116"></a>
-<span class="sourceLineNo">117</span> SyncReplicationReplayWALState state)<a name="line.117"></a>
-<span class="sourceLineNo">118</span> throws IOException, InterruptedException {<a name="line.118"></a>
-<span class="sourceLineNo">119</span> if (state == getInitialState()) {<a name="line.119"></a>
-<span class="sourceLineNo">120</span> return;<a name="line.120"></a>
-<span class="sourceLineNo">121</span> }<a name="line.121"></a>
-<span class="sourceLineNo">122</span> throw new UnsupportedOperationException();<a name="line.122"></a>
-<span class="sourceLineNo">123</span> }<a name="line.123"></a>
-<span class="sourceLineNo">124</span><a name="line.124"></a>
-<span class="sourceLineNo">125</span> @Override<a name="line.125"></a>
-<span class="sourceLineNo">126</span> protected SyncReplicationReplayWALState getState(int state) {<a name="line.126"></a>
-<span class="sourceLineNo">127</span> return SyncReplicationReplayWALState.forNumber(state);<a name="line.127"></a>
-<span class="sourceLineNo">128</span> }<a name="line.128"></a>
-<span class="sourceLineNo">129</span><a name="line.129"></a>
-<span class="sourceLineNo">130</span> @Override<a name="line.130"></a>
-<span class="sourceLineNo">131</span> protected int getStateId(<a name="line.131"></a>
-<span class="sourceLineNo">132</span> SyncReplicationReplayWALState state) {<a name="line.132"></a>
-<span class="sourceLineNo">133</span> return state.getNumber();<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> @Override<a name="line.136"></a>
-<span class="sourceLineNo">137</span> protected SyncReplicationReplayWALState getInitialState() {<a name="line.137"></a>
-<span class="sourceLineNo">138</span> return SyncReplicationReplayWALState.ASSIGN_WORKER;<a name="line.138"></a>
-<span class="sourceLineNo">139</span> }<a name="line.139"></a>
-<span class="sourceLineNo">140</span><a name="line.140"></a>
-<span class="sourceLineNo">141</span> @Override<a name="line.141"></a>
-<span class="sourceLineNo">142</span> protected void serializeStateData(ProcedureStateSerializer serializer)<a name="line.142"></a>
-<span class="sourceLineNo">143</span> throws IOException {<a name="line.143"></a>
-<span class="sourceLineNo">144</span> SyncReplicationReplayWALStateData.Builder builder =<a name="line.144"></a>
-<span class="sourceLineNo">145</span> SyncReplicationReplayWALStateData.newBuilder();<a name="line.145"></a>
-<span class="sourceLineNo">146</span> builder.setPeerId(peerId);<a name="line.146"></a>
-<span class="sourceLineNo">147</span> wals.stream().forEach(builder::addWal);<a name="line.147"></a>
-<span class="sourceLineNo">148</span> serializer.serialize(builder.build());<a name="line.148"></a>
-<span class="sourceLineNo">149</span> }<a name="line.149"></a>
-<span class="sourceLineNo">150</span><a name="line.150"></a>
-<span class="sourceLineNo">151</span> @Override<a name="line.151"></a>
-<span class="sourceLineNo">152</span> protected void deserializeStateData(ProcedureStateSerializer serializer) throws IOException {<a name="line.152"></a>
-<span class="sourceLineNo">153</span> SyncReplicationReplayWALStateData data =<a name="line.153"></a>
-<span class="sourceLineNo">154</span> serializer.deserialize(SyncReplicationReplayWALStateData.class);<a name="line.154"></a>
-<span class="sourceLineNo">155</span> peerId = data.getPeerId();<a name="line.155"></a>
-<span class="sourceLineNo">156</span> wals = new ArrayList<>();<a name="line.156"></a>
-<span class="sourceLineNo">157</span> data.getWalList().forEach(wals::add);<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> @Override<a name="line.160"></a>
-<span class="sourceLineNo">161</span> public PeerOperationType getPeerOperationType() {<a name="line.161"></a>
-<span class="sourceLineNo">162</span> return PeerOperationType.SYNC_REPLICATION_REPLAY_WAL;<a name="line.162"></a>
-<span class="sourceLineNo">163</span> }<a name="line.163"></a>
-<span class="sourceLineNo">164</span>}<a name="line.164"></a>
+<span class="sourceLineNo">116</span> protected SyncReplicationReplayWALState getInitialState() {<a name="line.116"></a>
+<span class="sourceLineNo">117</span> return SyncReplicationReplayWALState.ASSIGN_WORKER;<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> @Override<a name="line.120"></a>
+<span class="sourceLineNo">121</span> protected void serializeStateData(ProcedureStateSerializer serializer) throws IOException {<a name="line.121"></a>
+<span class="sourceLineNo">122</span> super.serializeStateData(serializer);<a name="line.122"></a>
+<span class="sourceLineNo">123</span> SyncReplicationReplayWALStateData.Builder builder =<a name="line.123"></a>
+<span class="sourceLineNo">124</span> SyncReplicationReplayWALStateData.newBuilder().setPeerId(peerId).addAllWal(wals);<a name="line.124"></a>
+<span class="sourceLineNo">125</span> if (worker != null) {<a name="line.125"></a>
+<span class="sourceLineNo">126</span> builder.setWorker(ProtobufUtil.toServerName(worker));<a name="line.126"></a>
+<span class="sourceLineNo">127</span> }<a name="line.127"></a>
+<span class="sourceLineNo">128</span> serializer.serialize(builder.build());<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> @Override<a name="line.131"></a>
+<span class="sourceLineNo">132</span> protected void deserializeStateData(ProcedureStateSerializer serializer) throws IOException {<a name="line.132"></a>
+<span class="sourceLineNo">133</span> super.deserializeStateData(serializer);<a name="line.133"></a>
+<span class="sourceLineNo">134</span> SyncReplicationReplayWALStateData data =<a name="line.134"></a>
+<span class="sourceLineNo">135</span> serializer.deserialize(SyncReplicationReplayWALStateData.class);<a name="line.135"></a>
+<span class="sourceLineNo">136</span> peerId = data.getPeerId();<a name="line.136"></a>
+<span class="sourceLineNo">137</span> wals = data.getWalList();<a name="line.137"></a>
+<span class="sourceLineNo">138</span> if (data.hasWorker()) {<a name="line.138"></a>
+<span class="sourceLineNo">139</span> worker = ProtobufUtil.toServerName(data.getWorker());<a name="line.139"></a>
+<span class="sourceLineNo">140</span> }<a name="line.140"></a>
+<span class="sourceLineNo">141</span> }<a name="line.141"></a>
+<span class="sourceLineNo">142</span><a name="line.142"></a>
+<span class="sourceLineNo">143</span> @Override<a name="line.143"></a>
+<span class="sourceLineNo">144</span> public PeerOperationType getPeerOperationType() {<a name="line.144"></a>
+<span class="sourceLineNo">145</span> return PeerOperationType.SYNC_REPLICATION_REPLAY_WAL;<a name="line.145"></a>
+<span class="sourceLineNo">146</span> }<a name="line.146"></a>
+<span class="sourceLineNo">147</span><a name="line.147"></a>
+<span class="sourceLineNo">148</span> @Override<a name="line.148"></a>
+<span class="sourceLineNo">149</span> protected void afterReplay(MasterProcedureEnv env) {<a name="line.149"></a>
+<span class="sourceLineNo">150</span> // If the procedure is not finished and the worker is not null, we should add it to the used<a name="line.150"></a>
+<span class="sourceLineNo">151</span> // worker set, to prevent the worker being used by others.<a name="line.151"></a>
+<span class="sourceLineNo">152</span> if (worker != null && !isFinished()) {<a name="line.152"></a>
+<span class="sourceLineNo">153</span> env.getMasterServices().getSyncReplicationReplayWALManager().addUsedPeerWorker(peerId,<a name="line.153"></a>
+<span class="sourceLineNo">154</span> worker);<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>}<a name="line.157"></a>
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/74ea8168/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALRemoteProcedure.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALRemoteProcedure.html b/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALRemoteProcedure.html
index e0343ff..afa0b72 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALRemoteProcedure.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/SyncReplicationReplayWALRemoteProcedure.html
@@ -51,151 +51,151 @@
<span class="sourceLineNo">043</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.ReplaySyncReplicationWALParameter;<a name="line.43"></a>
<span class="sourceLineNo">044</span>import org.apache.hadoop.hbase.shaded.protobuf.generated.MasterProcedureProtos.SyncReplicationReplayWALRemoteStateData;<a name="line.44"></a>
<span class="sourceLineNo">045</span><a name="line.45"></a>
-<span class="sourceLineNo">046</span>@InterfaceAudience.Private<a name="line.46"></a>
-<span class="sourceLineNo">047</span>public class SyncReplicationReplayWALRemoteProcedure extends Procedure<MasterProcedureEnv><a name="line.47"></a>
-<span class="sourceLineNo">048</span> implements RemoteProcedure<MasterProcedureEnv, ServerName>, PeerProcedureInterface {<a name="line.48"></a>
-<span class="sourceLineNo">049</span><a name="line.49"></a>
-<span class="sourceLineNo">050</span> private static final Logger LOG =<a name="line.50"></a>
-<span class="sourceLineNo">051</span> LoggerFactory.getLogger(SyncReplicationReplayWALRemoteProcedure.class);<a name="line.51"></a>
+<span class="sourceLineNo">046</span>/**<a name="line.46"></a>
+<span class="sourceLineNo">047</span> * A remote procedure which is used to send replaying remote wal work to region server.<a name="line.47"></a>
+<span class="sourceLineNo">048</span> */<a name="line.48"></a>
+<span class="sourceLineNo">049</span>@InterfaceAudience.Private<a name="line.49"></a>
+<span class="sourceLineNo">050</span>public class SyncReplicationReplayWALRemoteProcedure extends Procedure<MasterProcedureEnv><a name="line.50"></a>
+<span class="sourceLineNo">051</span> implements RemoteProcedure<MasterProcedureEnv, ServerName>, PeerProcedureInterface {<a name="line.51"></a>
<span class="sourceLineNo">052</span><a name="line.52"></a>
-<span class="sourceLineNo">053</span> private String peerId;<a name="line.53"></a>
-<span class="sourceLineNo">054</span><a name="line.54"></a>
-<span class="sourceLineNo">055</span> private ServerName targetServer;<a name="line.55"></a>
-<span class="sourceLineNo">056</span><a name="line.56"></a>
-<span class="sourceLineNo">057</span> private List<String> wals;<a name="line.57"></a>
-<span class="sourceLineNo">058</span><a name="line.58"></a>
-<span class="sourceLineNo">059</span> private boolean dispatched;<a name="line.59"></a>
-<span class="sourceLineNo">060</span><a name="line.60"></a>
-<span class="sourceLineNo">061</span> private ProcedureEvent<?> event;<a name="line.61"></a>
-<span class="sourceLineNo">062</span><a name="line.62"></a>
-<span class="sourceLineNo">063</span> private boolean succ;<a name="line.63"></a>
-<span class="sourceLineNo">064</span><a name="line.64"></a>
-<span class="sourceLineNo">065</span> public SyncReplicationReplayWALRemoteProcedure() {<a name="line.65"></a>
-<span class="sourceLineNo">066</span> }<a name="line.66"></a>
+<span class="sourceLineNo">053</span> private static final Logger LOG =<a name="line.53"></a>
+<span class="sourceLineNo">054</span> LoggerFactory.getLogger(SyncReplicationReplayWALRemoteProcedure.class);<a name="line.54"></a>
+<span class="sourceLineNo">055</span><a name="line.55"></a>
+<span class="sourceLineNo">056</span> private String peerId;<a name="line.56"></a>
+<span class="sourceLineNo">057</span><a name="line.57"></a>
+<span class="sourceLineNo">058</span> private ServerName targetServer;<a name="line.58"></a>
+<span class="sourceLineNo">059</span><a name="line.59"></a>
+<span class="sourceLineNo">060</span> private List<String> wals;<a name="line.60"></a>
+<span class="sourceLineNo">061</span><a name="line.61"></a>
+<span class="sourceLineNo">062</span> private boolean dispatched;<a name="line.62"></a>
+<span class="sourceLineNo">063</span><a name="line.63"></a>
+<span class="sourceLineNo">064</span> private ProcedureEvent<?> event;<a name="line.64"></a>
+<span class="sourceLineNo">065</span><a name="line.65"></a>
+<span class="sourceLineNo">066</span> private boolean succ;<a name="line.66"></a>
<span class="sourceLineNo">067</span><a name="line.67"></a>
-<span class="sourceLineNo">068</span> public SyncReplicationReplayWALRemoteProcedure(String peerId, List<String> wals,<a name="line.68"></a>
-<span class="sourceLineNo">069</span> ServerName targetServer) {<a name="line.69"></a>
-<span class="sourceLineNo">070</span> this.peerId = peerId;<a name="line.70"></a>
-<span class="sourceLineNo">071</span> this.wals = wals;<a name="line.71"></a>
-<span class="sourceLineNo">072</span> this.targetServer = targetServer;<a name="line.72"></a>
-<span class="sourceLineNo">073</span> }<a name="line.73"></a>
-<span class="sourceLineNo">074</span><a name="line.74"></a>
-<span class="sourceLineNo">075</span> @Override<a name="line.75"></a>
-<span class="sourceLineNo">076</span> public RemoteOperation remoteCallBuild(MasterProcedureEnv env, ServerName remote) {<a name="line.76"></a>
-<span class="sourceLineNo">077</span> ReplaySyncReplicationWALParameter.Builder builder =<a name="line.77"></a>
-<span class="sourceLineNo">078</span> ReplaySyncReplicationWALParameter.newBuilder();<a name="line.78"></a>
-<span class="sourceLineNo">079</span> builder.setPeerId(peerId);<a name="line.79"></a>
-<span class="sourceLineNo">080</span> wals.stream().forEach(builder::addWal);<a name="line.80"></a>
-<span class="sourceLineNo">081</span> return new ServerOperation(this, getProcId(), ReplaySyncReplicationWALCallable.class,<a name="line.81"></a>
-<span class="sourceLineNo">082</span> builder.build().toByteArray());<a name="line.82"></a>
-<span class="sourceLineNo">083</span> }<a name="line.83"></a>
-<span class="sourceLineNo">084</span><a name="line.84"></a>
-<span class="sourceLineNo">085</span> @Override<a name="line.85"></a>
-<span class="sourceLineNo">086</span> public void remoteCallFailed(MasterProcedureEnv env, ServerName remote, IOException exception) {<a name="line.86"></a>
-<span class="sourceLineNo">087</span> complete(env, exception);<a name="line.87"></a>
-<span class="sourceLineNo">088</span> }<a name="line.88"></a>
-<span class="sourceLineNo">089</span><a name="line.89"></a>
-<span class="sourceLineNo">090</span> @Override<a name="line.90"></a>
-<span class="sourceLineNo">091</span> public void remoteOperationCompleted(MasterProcedureEnv env) {<a name="line.91"></a>
-<span class="sourceLineNo">092</span> complete(env, null);<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> @Override<a name="line.95"></a>
-<span class="sourceLineNo">096</span> public void remoteOperationFailed(MasterProcedureEnv env, RemoteProcedureException error) {<a name="line.96"></a>
-<span class="sourceLineNo">097</span> complete(env, error);<a name="line.97"></a>
-<span class="sourceLineNo">098</span> }<a name="line.98"></a>
-<span class="sourceLineNo">099</span><a name="line.99"></a>
-<span class="sourceLineNo">100</span> private void complete(MasterProcedureEnv env, Throwable error) {<a name="line.100"></a>
-<span class="sourceLineNo">101</span> if (event == null) {<a name="line.101"></a>
-<span class="sourceLineNo">102</span> LOG.warn("procedure event for {} is null, maybe the procedure is created when recovery",<a name="line.102"></a>
-<span class="sourceLineNo">103</span> getProcId());<a name="line.103"></a>
-<span class="sourceLineNo">104</span> return;<a name="line.104"></a>
-<span class="sourceLineNo">105</span> }<a name="line.105"></a>
-<span class="sourceLineNo">106</span> if (error != null) {<a name="line.106"></a>
-<span class="sourceLineNo">107</span> LOG.warn("Replay wals {} on {} failed for peer id={}", wals, targetServer, peerId, error);<a name="line.107"></a>
-<span class="sourceLineNo">108</span> this.succ = false;<a name="line.108"></a>
-<span class="sourceLineNo">109</span> } else {<a name="line.109"></a>
-<span class="sourceLineNo">110</span> truncateWALs(env);<a name="line.110"></a>
-<span class="sourceLineNo">111</span> LOG.info("Replay wals {} on {} succeed for peer id={}", wals, targetServer, peerId);<a name="line.111"></a>
-<span class="sourceLineNo">112</span> this.succ = true;<a name="line.112"></a>
-<span class="sourceLineNo">113</span> }<a name="line.113"></a>
-<span class="sourceLineNo">114</span> event.wake(env.getProcedureScheduler());<a name="line.114"></a>
-<span class="sourceLineNo">115</span> event = null;<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> * Only truncate wals one by one when task succeed. The parent procedure will check the first<a name="line.119"></a>
-<span class="sourceLineNo">120</span> * wal length to know whether this task succeed.<a name="line.120"></a>
-<span class="sourceLineNo">121</span> */<a name="line.121"></a>
-<span class="sourceLineNo">122</span> private void truncateWALs(MasterProcedureEnv env) {<a name="line.122"></a>
-<span class="sourceLineNo">123</span> String firstWal = wals.get(0);<a name="line.123"></a>
-<span class="sourceLineNo">124</span> try {<a name="line.124"></a>
-<span class="sourceLineNo">125</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(firstWal);<a name="line.125"></a>
-<span class="sourceLineNo">126</span> } catch (IOException e) {<a name="line.126"></a>
-<span class="sourceLineNo">127</span> // As it is idempotent to rerun this task. Just ignore this exception and return.<a name="line.127"></a>
-<span class="sourceLineNo">128</span> LOG.warn("Failed to truncate wal {} for peer id={}", firstWal, peerId, e);<a name="line.128"></a>
-<span class="sourceLineNo">129</span> return;<a name="line.129"></a>
-<span class="sourceLineNo">130</span> }<a name="line.130"></a>
-<span class="sourceLineNo">131</span> for (int i = 1; i < wals.size(); i++) {<a name="line.131"></a>
-<span class="sourceLineNo">132</span> String wal = wals.get(i);<a name="line.132"></a>
-<span class="sourceLineNo">133</span> try {<a name="line.133"></a>
-<span class="sourceLineNo">134</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(wal);<a name="line.134"></a>
-<span class="sourceLineNo">135</span> } catch (IOException e1) {<a name="line.135"></a>
-<span class="sourceLineNo">136</span> try {<a name="line.136"></a>
-<span class="sourceLineNo">137</span> // retry<a name="line.137"></a>
-<span class="sourceLineNo">138</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(wal);<a name="line.138"></a>
-<span class="sourceLineNo">139</span> } catch (IOException e2) {<a name="line.139"></a>
-<span class="sourceLineNo">140</span> // As the parent procedure only check the first wal length. Just ignore this exception.<a name="line.140"></a>
-<span class="sourceLineNo">141</span> LOG.warn("Failed to truncate wal {} for peer id={}", wal, peerId, e2);<a name="line.141"></a>
-<span class="sourceLineNo">142</span> }<a name="line.142"></a>
-<span class="sourceLineNo">143</span> }<a name="line.143"></a>
-<span class="sourceLineNo">144</span> }<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> protected Procedure<MasterProcedureEnv>[] execute(MasterProcedureEnv env)<a name="line.148"></a>
-<span class="sourceLineNo">149</span> throws ProcedureYieldException, ProcedureSuspendedException, InterruptedException {<a name="line.149"></a>
-<span class="sourceLineNo">150</span> if (dispatched) {<a name="line.150"></a>
-<span class="sourceLineNo">151</span> if (succ) {<a name="line.151"></a>
-<span class="sourceLineNo">152</span> return null;<a name="line.152"></a>
-<span class="sourceLineNo">153</span> }<a name="line.153"></a>
-<span class="sourceLineNo">154</span> // retry<a name="line.154"></a>
-<span class="sourceLineNo">155</span> dispatched = false;<a name="line.155"></a>
-<span class="sourceLineNo">156</span> }<a name="line.156"></a>
-<span class="sourceLineNo">157</span><a name="line.157"></a>
-<span class="sourceLineNo">158</span> // Dispatch task to target server<a name="line.158"></a>
-<span class="sourceLineNo">159</span> try {<a name="line.159"></a>
-<span class="sourceLineNo">160</span> env.getRemoteDispatcher().addOperationToNode(targetServer, this);<a name="line.160"></a>
-<span class="sourceLineNo">161</span> } catch (FailedRemoteDispatchException e) {<a name="line.161"></a>
-<span class="sourceLineNo">162</span> LOG.warn(<a name="line.162"></a>
-<span class="sourceLineNo">163</span> "Can not add remote operation for replay wals {} on {} for peer id={}, "<a name="line.163"></a>
-<span class="sourceLineNo">164</span> + "this usually because the server is already dead",<a name="line.164"></a>
-<span class="sourceLineNo">165</span> wals, targetServer, peerId);<a name="line.165"></a>
-<span class="sourceLineNo">166</span> // Return directly and the parent procedure will assign a new worker to replay wals<a name="line.166"></a>
-<span class="sourceLineNo">167</span> return null;<a name="line.167"></a>
-<span class="sourceLineNo">168</span> }<a name="line.168"></a>
-<span class="sourceLineNo">169</span> dispatched = true;<a name="line.169"></a>
-<span class="sourceLineNo">170</span> event = new ProcedureEvent<>(this);<a name="line.170"></a>
-<span class="sourceLineNo">171</span> event.suspendIfNotReady(this);<a name="line.171"></a>
-<span class="sourceLineNo">172</span> throw new ProcedureSuspendedException();<a name="line.172"></a>
-<span class="sourceLineNo">173</span> }<a name="line.173"></a>
-<span class="sourceLineNo">174</span><a name="line.174"></a>
-<span class="sourceLineNo">175</span> @Override<a name="line.175"></a>
-<span class="sourceLineNo">176</span> protected void rollback(MasterProcedureEnv env) throws IOException, InterruptedException {<a name="line.176"></a>
-<span class="sourceLineNo">177</span> throw new UnsupportedOperationException();<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">180</span> @Override<a name="line.180"></a>
-<span class="sourceLineNo">181</span> protected boolean abort(MasterProcedureEnv env) {<a name="line.181"></a>
-<span class="sourceLineNo">182</span> return false;<a name="line.182"></a>
-<span class="sourceLineNo">183</span> }<a name="line.183"></a>
-<span class="sourceLineNo">184</span><a name="line.184"></a>
-<span class="sourceLineNo">185</span> @Override<a name="line.185"></a>
-<span class="sourceLineNo">186</span> protected void serializeStateData(ProcedureStateSerializer serializer)<a name="line.186"></a>
-<span class="sourceLineNo">187</span> throws IOException {<a name="line.187"></a>
+<span class="sourceLineNo">068</span> public SyncReplicationReplayWALRemoteProcedure() {<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> public SyncReplicationReplayWALRemoteProcedure(String peerId, List<String> wals,<a name="line.71"></a>
+<span class="sourceLineNo">072</span> ServerName targetServer) {<a name="line.72"></a>
+<span class="sourceLineNo">073</span> this.peerId = peerId;<a name="line.73"></a>
+<span class="sourceLineNo">074</span> this.wals = wals;<a name="line.74"></a>
+<span class="sourceLineNo">075</span> this.targetServer = targetServer;<a name="line.75"></a>
+<span class="sourceLineNo">076</span> }<a name="line.76"></a>
+<span class="sourceLineNo">077</span><a name="line.77"></a>
+<span class="sourceLineNo">078</span> @Override<a name="line.78"></a>
+<span class="sourceLineNo">079</span> public RemoteOperation remoteCallBuild(MasterProcedureEnv env, ServerName remote) {<a name="line.79"></a>
+<span class="sourceLineNo">080</span> ReplaySyncReplicationWALParameter.Builder builder =<a name="line.80"></a>
+<span class="sourceLineNo">081</span> ReplaySyncReplicationWALParameter.newBuilder();<a name="line.81"></a>
+<span class="sourceLineNo">082</span> builder.setPeerId(peerId);<a name="line.82"></a>
+<span class="sourceLineNo">083</span> wals.stream().forEach(builder::addWal);<a name="line.83"></a>
+<span class="sourceLineNo">084</span> return new ServerOperation(this, getProcId(), ReplaySyncReplicationWALCallable.class,<a name="line.84"></a>
+<span class="sourceLineNo">085</span> builder.build().toByteArray());<a name="line.85"></a>
+<span class="sourceLineNo">086</span> }<a name="line.86"></a>
+<span class="sourceLineNo">087</span><a name="line.87"></a>
+<span class="sourceLineNo">088</span> @Override<a name="line.88"></a>
+<span class="sourceLineNo">089</span> public void remoteCallFailed(MasterProcedureEnv env, ServerName remote, IOException exception) {<a name="line.89"></a>
+<span class="sourceLineNo">090</span> complete(env, exception);<a name="line.90"></a>
+<span class="sourceLineNo">091</span> }<a name="line.91"></a>
+<span class="sourceLineNo">092</span><a name="line.92"></a>
+<span class="sourceLineNo">093</span> @Override<a name="line.93"></a>
+<span class="sourceLineNo">094</span> public void remoteOperationCompleted(MasterProcedureEnv env) {<a name="line.94"></a>
+<span class="sourceLineNo">095</span> complete(env, null);<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> @Override<a name="line.98"></a>
+<span class="sourceLineNo">099</span> public void remoteOperationFailed(MasterProcedureEnv env, RemoteProcedureException error) {<a name="line.99"></a>
+<span class="sourceLineNo">100</span> complete(env, error);<a name="line.100"></a>
+<span class="sourceLineNo">101</span> }<a name="line.101"></a>
+<span class="sourceLineNo">102</span><a name="line.102"></a>
+<span class="sourceLineNo">103</span> private void complete(MasterProcedureEnv env, Throwable error) {<a name="line.103"></a>
+<span class="sourceLineNo">104</span> if (event == null) {<a name="line.104"></a>
+<span class="sourceLineNo">105</span> LOG.warn("procedure event for {} is null, maybe the procedure is created when recovery",<a name="line.105"></a>
+<span class="sourceLineNo">106</span> getProcId());<a name="line.106"></a>
+<span class="sourceLineNo">107</span> return;<a name="line.107"></a>
+<span class="sourceLineNo">108</span> }<a name="line.108"></a>
+<span class="sourceLineNo">109</span> if (error != null) {<a name="line.109"></a>
+<span class="sourceLineNo">110</span> LOG.warn("Replay wals {} on {} failed for peer id={}", wals, targetServer, peerId, error);<a name="line.110"></a>
+<span class="sourceLineNo">111</span> this.succ = false;<a name="line.111"></a>
+<span class="sourceLineNo">112</span> } else {<a name="line.112"></a>
+<span class="sourceLineNo">113</span> truncateWALs(env);<a name="line.113"></a>
+<span class="sourceLineNo">114</span> LOG.info("Replay wals {} on {} succeed for peer id={}", wals, targetServer, peerId);<a name="line.114"></a>
+<span class="sourceLineNo">115</span> this.succ = true;<a name="line.115"></a>
+<span class="sourceLineNo">116</span> }<a name="line.116"></a>
+<span class="sourceLineNo">117</span> event.wake(env.getProcedureScheduler());<a name="line.117"></a>
+<span class="sourceLineNo">118</span> event = null;<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">121</span> /**<a name="line.121"></a>
+<span class="sourceLineNo">122</span> * Only truncate wals one by one when task succeed. The parent procedure will check the first wal<a name="line.122"></a>
+<span class="sourceLineNo">123</span> * length to know whether this task succeed.<a name="line.123"></a>
+<span class="sourceLineNo">124</span> */<a name="line.124"></a>
+<span class="sourceLineNo">125</span> private void truncateWALs(MasterProcedureEnv env) {<a name="line.125"></a>
+<span class="sourceLineNo">126</span> String firstWal = wals.get(0);<a name="line.126"></a>
+<span class="sourceLineNo">127</span> try {<a name="line.127"></a>
+<span class="sourceLineNo">128</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(firstWal);<a name="line.128"></a>
+<span class="sourceLineNo">129</span> } catch (IOException e) {<a name="line.129"></a>
+<span class="sourceLineNo">130</span> // As it is idempotent to rerun this task. Just ignore this exception and return.<a name="line.130"></a>
+<span class="sourceLineNo">131</span> LOG.warn("Failed to truncate wal {} for peer id={}", firstWal, peerId, e);<a name="line.131"></a>
+<span class="sourceLineNo">132</span> return;<a name="line.132"></a>
+<span class="sourceLineNo">133</span> }<a name="line.133"></a>
+<span class="sourceLineNo">134</span> for (int i = 1; i < wals.size(); i++) {<a name="line.134"></a>
+<span class="sourceLineNo">135</span> String wal = wals.get(i);<a name="line.135"></a>
+<span class="sourceLineNo">136</span> try {<a name="line.136"></a>
+<span class="sourceLineNo">137</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(wal);<a name="line.137"></a>
+<span class="sourceLineNo">138</span> } catch (IOException e1) {<a name="line.138"></a>
+<span class="sourceLineNo">139</span> try {<a name="line.139"></a>
+<span class="sourceLineNo">140</span> // retry<a name="line.140"></a>
+<span class="sourceLineNo">141</span> env.getMasterServices().getSyncReplicationReplayWALManager().finishReplayWAL(wal);<a name="line.141"></a>
+<span class="sourceLineNo">142</span> } catch (IOException e2) {<a name="line.142"></a>
+<span class="sourceLineNo">143</span> // As the parent procedure only check the first wal length. Just ignore this exception.<a name="line.143"></a>
+<span class="sourceLineNo">144</span> LOG.warn("Failed to truncate wal {} for peer id={}", wal, peerId, e2);<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> }<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> protected Procedure<MasterProcedureEnv>[] execute(MasterProcedureEnv env)<a name="line.151"></a>
+<span class="sourceLineNo">152</span> throws ProcedureYieldException, ProcedureSuspendedException, InterruptedException {<a name="line.152"></a>
+<span class="sourceLineNo">153</span> if (dispatched) {<a name="line.153"></a>
+<span class="sourceLineNo">154</span> if (succ) {<a name="line.154"></a>
+<span class="sourceLineNo">155</span> return null;<a name="line.155"></a>
+<span class="sourceLineNo">156</span> }<a name="line.156"></a>
+<span class="sourceLineNo">157</span> // retry<a name="line.157"></a>
+<span class="sourceLineNo">158</span> dispatched = false;<a name="line.158"></a>
+<span class="sourceLineNo">159</span> }<a name="line.159"></a>
+<span class="sourceLineNo">160</span><a name="line.160"></a>
+<span class="sourceLineNo">161</span> // Dispatch task to target server<a name="line.161"></a>
+<span class="sourceLineNo">162</span> try {<a name="line.162"></a>
+<span class="sourceLineNo">163</span> env.getRemoteDispatcher().addOperationToNode(targetServer, this);<a name="line.163"></a>
+<span class="sourceLineNo">164</span> } catch (FailedRemoteDispatchException e) {<a name="line.164"></a>
+<span class="sourceLineNo">165</span> LOG.warn("Can not add remote operation for replay wals {} on {} for peer id={}, " +<a name="line.165"></a>
+<span class="sourceLineNo">166</span> "this usually because the server is already dead", wals, targetServer, peerId);<a name="line.166"></a>
+<span class="sourceLineNo">167</span> // Return directly and the parent procedure will assign a new worker to replay wals<a name="line.167"></a>
+<span class="sourceLineNo">168</span> return null;<a name="line.168"></a>
+<span class="sourceLineNo">169</span> }<a name="line.169"></a>
+<span class="sourceLineNo">170</span> dispatched = true;<a name="line.170"></a>
+<span class="sourceLineNo">171</span> event = new ProcedureEvent<>(this);<a name="line.171"></a>
+<span class="sourceLineNo">172</span> event.suspendIfNotReady(this);<a name="line.172"></a>
+<span class="sourceLineNo">173</span> throw new ProcedureSuspendedException();<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> @Override<a name="line.176"></a>
+<span class="sourceLineNo">177</span> protected void rollback(MasterProcedureEnv env) throws IOException, InterruptedException {<a name="line.177"></a>
+<span class="sourceLineNo">178</span> throw new UnsupportedOperationException();<a name="line.178"></a>
+<span class="sourceLineNo">179</span> }<a name="line.179"></a>
+<span class="sourceLineNo">180</span><a name="line.180"></a>
+<span class="sourceLineNo">181</span> @Override<a name="line.181"></a>
+<span class="sourceLineNo">182</span> protected boolean abort(MasterProcedureEnv env) {<a name="line.182"></a>
+<span class="sourceLineNo">183</span> return false;<a name="line.183"></a>
+<span class="sourceLineNo">184</span> }<a name="line.184"></a>
+<span class="sourceLineNo">185</span><a name="line.185"></a>
+<span class="sourceLineNo">186</span> @Override<a name="line.186"></a>
+<span class="sourceLineNo">187</span> protected void serializeStateData(ProcedureStateSerializer serializer) throws IOException {<a name="line.187"></a>
<span class="sourceLineNo">188</span> SyncReplicationReplayWALRemoteStateData.Builder builder =<a name="line.188"></a>
-<span class="sourceLineNo">189</span> SyncReplicationReplayWALRemoteStateData.newBuilder().setPeerId(peerId)<a name="line.189"></a>
-<span class="sourceLineNo">190</span> .setTargetServer(ProtobufUtil.toServerName(targetServer));<a name="line.190"></a>
+<span class="sourceLineNo">189</span> SyncReplicationReplayWALRemoteStateData.newBuilder().setPeerId(peerId)<a name="line.189"></a>
+<span class="sourceLineNo">190</span> .setTargetServer(ProtobufUtil.toServerName(targetServer));<a name="line.190"></a>
<span class="sourceLineNo">191</span> wals.stream().forEach(builder::addWal);<a name="line.191"></a>
<span class="sourceLineNo">192</span> serializer.serialize(builder.build());<a name="line.192"></a>
<span class="sourceLineNo">193</span> }<a name="line.193"></a>
@@ -203,7 +203,7 @@
<span class="sourceLineNo">195</span> @Override<a name="line.195"></a>
<span class="sourceLineNo">196</span> protected void deserializeStateData(ProcedureStateSerializer serializer) throws IOException {<a name="line.196"></a>
<span class="sourceLineNo">197</span> SyncReplicationReplayWALRemoteStateData data =<a name="line.197"></a>
-<span class="sourceLineNo">198</span> serializer.deserialize(SyncReplicationReplayWALRemoteStateData.class);<a name="line.198"></a>
+<span class="sourceLineNo">198</span> serializer.deserialize(SyncReplicationReplayWALRemoteStateData.class);<a name="line.198"></a>
<span class="sourceLineNo">199</span> peerId = data.getPeerId();<a name="line.199"></a>
<span class="sourceLineNo">200</span> wals = new ArrayList<>();<a name="line.200"></a>
<span class="sourceLineNo">201</span> data.getWalList().forEach(wals::add);<a name="line.201"></a>
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/74ea8168/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/ZKSyncReplicationReplayWALWorkerStorage.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/ZKSyncReplicationReplayWALWorkerStorage.html b/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/ZKSyncReplicationReplayWALWorkerStorage.html
deleted file mode 100644
index 8a83f03..0000000
--- a/devapidocs/src-html/org/apache/hadoop/hbase/master/replication/ZKSyncReplicationReplayWALWorkerStorage.html
+++ /dev/null
@@ -1,180 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
-<html lang="en">
-<head>
-<title>Source code</title>
-<link rel="stylesheet" type="text/css" href="../../../../../../../stylesheet.css" title="Style">
-</head>
-<body>
-<div class="sourceContainer">
-<pre><span class="sourceLineNo">001</span>/**<a name="line.1"></a>
-<span class="sourceLineNo">002</span> * Licensed to the Apache Software Foundation (ASF) under one<a name="line.2"></a>
-<span class="sourceLineNo">003</span> * or more contributor license agreements. See the NOTICE file<a name="line.3"></a>
-<span class="sourceLineNo">004</span> * distributed with this work for additional information<a name="line.4"></a>
-<span class="sourceLineNo">005</span> * regarding copyright ownership. The ASF licenses this file<a name="line.5"></a>
-<span class="sourceLineNo">006</span> * to you under the Apache License, Version 2.0 (the<a name="line.6"></a>
-<span class="sourceLineNo">007</span> * "License"); you may not use this file except in compliance<a name="line.7"></a>
-<span class="sourceLineNo">008</span> * with the License. You may obtain a copy of the License at<a name="line.8"></a>
-<span class="sourceLineNo">009</span> *<a name="line.9"></a>
-<span class="sourceLineNo">010</span> * http://www.apache.org/licenses/LICENSE-2.0<a name="line.10"></a>
-<span class="sourceLineNo">011</span> *<a name="line.11"></a>
-<span class="sourceLineNo">012</span> * Unless required by applicable law or agreed to in writing, software<a name="line.12"></a>
-<span class="sourceLineNo">013</span> * distributed under the License is distributed on an "AS IS" BASIS,<a name="line.13"></a>
-<span class="sourceLineNo">014</span> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.<a name="line.14"></a>
-<span class="sourceLineNo">015</span> * See the License for the specific language governing permissions and<a name="line.15"></a>
-<span class="sourceLineNo">016</span> * limitations under the License.<a name="line.16"></a>
-<span class="sourceLineNo">017</span> */<a name="line.17"></a>
-<span class="sourceLineNo">018</span>package org.apache.hadoop.hbase.master.replication;<a name="line.18"></a>
-<span class="sourceLineNo">019</span><a name="line.19"></a>
-<span class="sourceLineNo">020</span>import java.util.HashSet;<a name="line.20"></a>
-<span class="sourceLineNo">021</span>import java.util.List;<a name="line.21"></a>
-<span class="sourceLineNo">022</span>import java.util.Set;<a name="line.22"></a>
-<span class="sourceLineNo">023</span>import java.util.stream.Collectors;<a name="line.23"></a>
-<span class="sourceLineNo">024</span><a name="line.24"></a>
-<span class="sourceLineNo">025</span>import org.apache.hadoop.conf.Configuration;<a name="line.25"></a>
-<span class="sourceLineNo">026</span>import org.apache.hadoop.hbase.ServerName;<a name="line.26"></a>
-<span class="sourceLineNo">027</span>import org.apache.hadoop.hbase.replication.ReplicationException;<a name="line.27"></a>
-<span class="sourceLineNo">028</span>import org.apache.hadoop.hbase.replication.ZKReplicationStorageBase;<a name="line.28"></a>
-<span class="sourceLineNo">029</span>import org.apache.hadoop.hbase.zookeeper.ZKUtil;<a name="line.29"></a>
-<span class="sourceLineNo">030</span>import org.apache.hadoop.hbase.zookeeper.ZKWatcher;<a name="line.30"></a>
-<span class="sourceLineNo">031</span>import org.apache.hadoop.hbase.zookeeper.ZNodePaths;<a name="line.31"></a>
-<span class="sourceLineNo">032</span>import org.apache.yetus.audience.InterfaceAudience;<a name="line.32"></a>
-<span class="sourceLineNo">033</span>import org.apache.zookeeper.KeeperException;<a name="line.33"></a>
-<span class="sourceLineNo">034</span><a name="line.34"></a>
-<span class="sourceLineNo">035</span>@InterfaceAudience.Private<a name="line.35"></a>
-<span class="sourceLineNo">036</span>public class ZKSyncReplicationReplayWALWorkerStorage extends ZKReplicationStorageBase {<a name="line.36"></a>
-<span class="sourceLineNo">037</span><a name="line.37"></a>
-<span class="sourceLineNo">038</span> public static final String WORKERS_ZNODE = "zookeeper.znode.sync.replication.replaywal.workers";<a name="line.38"></a>
-<span class="sourceLineNo">039</span><a name="line.39"></a>
-<span class="sourceLineNo">040</span> public static final String WORKERS_ZNODE_DEFAULT = "replaywal-workers";<a name="line.40"></a>
-<span class="sourceLineNo">041</span><a name="line.41"></a>
-<span class="sourceLineNo">042</span> /**<a name="line.42"></a>
-<span class="sourceLineNo">043</span> * The name of the znode that contains a list of workers to replay wal.<a name="line.43"></a>
-<span class="sourceLineNo">044</span> */<a name="line.44"></a>
-<span class="sourceLineNo">045</span> private final String workersZNode;<a name="line.45"></a>
-<span class="sourceLineNo">046</span><a name="line.46"></a>
-<span class="sourceLineNo">047</span> public ZKSyncReplicationReplayWALWorkerStorage(ZKWatcher zookeeper, Configuration conf) {<a name="line.47"></a>
-<span class="sourceLineNo">048</span> super(zookeeper, conf);<a name="line.48"></a>
-<span class="sourceLineNo">049</span> String workersZNodeName = conf.get(WORKERS_ZNODE, WORKERS_ZNODE_DEFAULT);<a name="line.49"></a>
-<span class="sourceLineNo">050</span> workersZNode = ZNodePaths.joinZNode(replicationZNode, workersZNodeName);<a name="line.50"></a>
-<span class="sourceLineNo">051</span> }<a name="line.51"></a>
-<span class="sourceLineNo">052</span><a name="line.52"></a>
-<span class="sourceLineNo">053</span> private String getPeerNode(String peerId) {<a name="line.53"></a>
-<span class="sourceLineNo">054</span> return ZNodePaths.joinZNode(workersZNode, peerId);<a name="line.54"></a>
-<span class="sourceLineNo">055</span> }<a name="line.55"></a>
-<span class="sourceLineNo">056</span><a name="line.56"></a>
-<span class="sourceLineNo">057</span> public void addPeer(String peerId) throws ReplicationException {<a name="line.57"></a>
-<span class="sourceLineNo">058</span> try {<a name="line.58"></a>
-<span class="sourceLineNo">059</span> ZKUtil.createWithParents(zookeeper, getPeerNode(peerId));<a name="line.59"></a>
-<span class="sourceLineNo">060</span> } catch (KeeperException e) {<a name="line.60"></a>
-<span class="sourceLineNo">061</span> throw new ReplicationException(<a name="line.61"></a>
-<span class="sourceLineNo">062</span> "Failed to add peer id=" + peerId + " to replaywal-workers storage", e);<a name="line.62"></a>
-<span class="sourceLineNo">063</span> }<a name="line.63"></a>
-<span class="sourceLineNo">064</span> }<a name="line.64"></a>
-<span class="sourceLineNo">065</span><a name="line.65"></a>
-<span class="sourceLineNo">066</span> public void removePeer(String peerId) throws ReplicationException {<a name="line.66"></a>
-<span class="sourceLineNo">067</span> try {<a name="line.67"></a>
-<span class="sourceLineNo">068</span> ZKUtil.deleteNodeRecursively(zookeeper, getPeerNode(peerId));<a name="line.68"></a>
-<span class="sourceLineNo">069</span> } catch (KeeperException e) {<a name="line.69"></a>
-<span class="sourceLineNo">070</span> throw new ReplicationException(<a name="line.70"></a>
-<span class="sourceLineNo">071</span> "Failed to remove peer id=" + peerId + " to replaywal-workers storage", e);<a name="line.71"></a>
-<span class="sourceLineNo">072</span> }<a name="line.72"></a>
-<span class="sourceLineNo">073</span> }<a name="line.73"></a>
-<span class="sourceLineNo">074</span><a name="line.74"></a>
-<span class="sourceLineNo">075</span> private String getPeerWorkerNode(String peerId, ServerName worker) {<a name="line.75"></a>
-<span class="sourceLineNo">076</span> return ZNodePaths.joinZNode(getPeerNode(peerId), worker.getServerName());<a name="line.76"></a>
-<span class="sourceLineNo">077</span> }<a name="line.77"></a>
-<span class="sourceLineNo">078</span><a name="line.78"></a>
-<span class="sourceLineNo">079</span> public void addPeerWorker(String peerId, ServerName worker) throws ReplicationException {<a name="line.79"></a>
-<span class="sourceLineNo">080</span> try {<a name="line.80"></a>
-<span class="sourceLineNo">081</span> ZKUtil.createWithParents(zookeeper, getPeerWorkerNode(peerId, worker));<a name="line.81"></a>
-<span class="sourceLineNo">082</span> } catch (KeeperException e) {<a name="line.82"></a>
-<span class="sourceLineNo">083</span> throw new ReplicationException("Failed to add worker=" + worker + " for peer id=" + peerId,<a name="line.83"></a>
-<span class="sourceLineNo">084</span> e);<a name="line.84"></a>
-<span class="sourceLineNo">085</span> }<a name="line.85"></a>
-<span class="sourceLineNo">086</span> }<a name="line.86"></a>
-<span class="sourceLineNo">087</span><a name="line.87"></a>
-<span class="sourceLineNo">088</span> public void removePeerWorker(String peerId, ServerName worker) throws ReplicationException {<a name="line.88"></a>
-<span class="sourceLineNo">089</span> try {<a name="line.89"></a>
-<span class="sourceLineNo">090</span> ZKUtil.deleteNode(zookeeper, getPeerWorkerNode(peerId, worker));<a name="line.90"></a>
-<span class="sourceLineNo">091</span> } catch (KeeperException e) {<a name="line.91"></a>
-<span class="sourceLineNo">092</span> throw new ReplicationException("Failed to remove worker=" + worker + " for peer id=" + peerId,<a name="line.92"></a>
-<span class="sourceLineNo">093</span> e);<a name="line.93"></a>
-<span class="sourceLineNo">094</span> }<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> public Set<ServerName> getPeerWorkers(String peerId) throws ReplicationException {<a name="line.97"></a>
-<span class="sourceLineNo">098</span> try {<a name="line.98"></a>
-<span class="sourceLineNo">099</span> List<String> children = ZKUtil.listChildrenNoWatch(zookeeper, getPeerNode(peerId));<a name="line.99"></a>
-<span class="sourceLineNo">100</span> if (children == null) {<a name="line.100"></a>
-<span class="sourceLineNo">101</span> return new HashSet<>();<a name="line.101"></a>
-<span class="sourceLineNo">102</span> }<a name="line.102"></a>
-<span class="sourceLineNo">103</span> return children.stream().map(ServerName::valueOf).collect(Collectors.toSet());<a name="line.103"></a>
-<span class="sourceLineNo">104</span> } catch (KeeperException e) {<a name="line.104"></a>
-<span class="sourceLineNo">105</span> throw new ReplicationException("Failed to list workers for peer id=" + peerId, e);<a name="line.105"></a>
-<span class="sourceLineNo">106</span> }<a name="line.106"></a>
-<span class="sourceLineNo">107</span> }<a name="line.107"></a>
-<span class="sourceLineNo">108</span>}<a name="line.108"></a>
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-</pre>
-</div>
-</body>
-</html>