http://git-wip-us.apache.org/repos/asf/hbase-site/blob/8b1eaec1/devapidocs/src-html/org/apache/hadoop/hbase/util/FSTableDescriptors.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/util/FSTableDescriptors.html 
b/devapidocs/src-html/org/apache/hadoop/hbase/util/FSTableDescriptors.html
index 7226d18..f065ddb 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/util/FSTableDescriptors.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/util/FSTableDescriptors.html
@@ -45,733 +45,736 @@
 <span class="sourceLineNo">037</span>import org.apache.hadoop.fs.FileSystem;<a 
name="line.37"></a>
 <span class="sourceLineNo">038</span>import org.apache.hadoop.fs.Path;<a 
name="line.38"></a>
 <span class="sourceLineNo">039</span>import org.apache.hadoop.fs.PathFilter;<a 
name="line.39"></a>
-<span class="sourceLineNo">040</span>import 
org.apache.yetus.audience.InterfaceAudience;<a name="line.40"></a>
-<span class="sourceLineNo">041</span>import org.slf4j.Logger;<a 
name="line.41"></a>
-<span class="sourceLineNo">042</span>import org.slf4j.LoggerFactory;<a 
name="line.42"></a>
-<span class="sourceLineNo">043</span>import 
org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder;<a 
name="line.43"></a>
-<span class="sourceLineNo">044</span>import 
org.apache.hadoop.hbase.client.TableDescriptor;<a name="line.44"></a>
-<span class="sourceLineNo">045</span>import 
org.apache.hadoop.hbase.client.TableDescriptorBuilder;<a name="line.45"></a>
-<span class="sourceLineNo">046</span>import 
org.apache.hadoop.hbase.Coprocessor;<a name="line.46"></a>
-<span class="sourceLineNo">047</span>import 
org.apache.hadoop.hbase.exceptions.DeserializationException;<a 
name="line.47"></a>
-<span class="sourceLineNo">048</span>import 
org.apache.hadoop.hbase.HConstants;<a name="line.48"></a>
-<span class="sourceLineNo">049</span>import 
org.apache.hadoop.hbase.regionserver.BloomType;<a name="line.49"></a>
-<span class="sourceLineNo">050</span>import 
org.apache.hbase.thirdparty.com.google.common.annotations.VisibleForTesting;<a 
name="line.50"></a>
-<span class="sourceLineNo">051</span>import 
org.apache.hbase.thirdparty.com.google.common.primitives.Ints;<a 
name="line.51"></a>
-<span class="sourceLineNo">052</span>import 
org.apache.hadoop.hbase.TableDescriptors;<a name="line.52"></a>
-<span class="sourceLineNo">053</span>import 
org.apache.hadoop.hbase.TableInfoMissingException;<a name="line.53"></a>
-<span class="sourceLineNo">054</span>import 
org.apache.hadoop.hbase.TableName;<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> * Implementation of {@link 
TableDescriptors} that reads descriptors from the<a name="line.57"></a>
-<span class="sourceLineNo">058</span> * passed filesystem.  It expects 
descriptors to be in a file in the<a name="line.58"></a>
-<span class="sourceLineNo">059</span> * {@link #TABLEINFO_DIR} subdir of the 
table's directory in FS.  Can be read-only<a name="line.59"></a>
-<span class="sourceLineNo">060</span> *  -- i.e. does not modify the 
filesystem or can be read and write.<a name="line.60"></a>
-<span class="sourceLineNo">061</span> *<a name="line.61"></a>
-<span class="sourceLineNo">062</span> * &lt;p&gt;Also has utility for keeping 
up the table descriptors tableinfo file.<a name="line.62"></a>
-<span class="sourceLineNo">063</span> * The table schema file is kept in the 
{@link #TABLEINFO_DIR} subdir<a name="line.63"></a>
-<span class="sourceLineNo">064</span> * of the table directory in the 
filesystem.<a name="line.64"></a>
-<span class="sourceLineNo">065</span> * It has a {@link 
#TABLEINFO_FILE_PREFIX} and then a suffix that is the<a name="line.65"></a>
-<span class="sourceLineNo">066</span> * edit sequenceid: e.g. 
&lt;code&gt;.tableinfo.0000000003&lt;/code&gt;.  This sequenceid<a 
name="line.66"></a>
-<span class="sourceLineNo">067</span> * is always increasing.  It starts at 
zero.  The table schema file with the<a name="line.67"></a>
-<span class="sourceLineNo">068</span> * highest sequenceid has the most recent 
schema edit. Usually there is one file<a name="line.68"></a>
-<span class="sourceLineNo">069</span> * only, the most recent but there may be 
short periods where there are more<a name="line.69"></a>
-<span class="sourceLineNo">070</span> * than one file. Old files are 
eventually cleaned.  Presumption is that there<a name="line.70"></a>
-<span class="sourceLineNo">071</span> * will not be lots of concurrent clients 
making table schema edits.  If so,<a name="line.71"></a>
-<span class="sourceLineNo">072</span> * the below needs a bit of a reworking 
and perhaps some supporting api in hdfs.<a name="line.72"></a>
-<span class="sourceLineNo">073</span> */<a name="line.73"></a>
-<span class="sourceLineNo">074</span>@InterfaceAudience.Private<a 
name="line.74"></a>
-<span class="sourceLineNo">075</span>public class FSTableDescriptors 
implements TableDescriptors {<a name="line.75"></a>
-<span class="sourceLineNo">076</span>  private static final Logger LOG = 
LoggerFactory.getLogger(FSTableDescriptors.class);<a name="line.76"></a>
-<span class="sourceLineNo">077</span>  private final FileSystem fs;<a 
name="line.77"></a>
-<span class="sourceLineNo">078</span>  private final Path rootdir;<a 
name="line.78"></a>
-<span class="sourceLineNo">079</span>  private final boolean fsreadonly;<a 
name="line.79"></a>
-<span class="sourceLineNo">080</span>  private volatile boolean usecache;<a 
name="line.80"></a>
-<span class="sourceLineNo">081</span>  private volatile boolean fsvisited;<a 
name="line.81"></a>
-<span class="sourceLineNo">082</span><a name="line.82"></a>
-<span class="sourceLineNo">083</span>  @VisibleForTesting<a name="line.83"></a>
-<span class="sourceLineNo">084</span>  long cachehits = 0;<a 
name="line.84"></a>
+<span class="sourceLineNo">040</span>import 
org.apache.hadoop.hbase.client.CoprocessorDescriptorBuilder;<a 
name="line.40"></a>
+<span class="sourceLineNo">041</span>import 
org.apache.hadoop.hbase.coprocessor.MultiRowMutationEndpoint;<a 
name="line.41"></a>
+<span class="sourceLineNo">042</span>import 
org.apache.yetus.audience.InterfaceAudience;<a name="line.42"></a>
+<span class="sourceLineNo">043</span>import org.slf4j.Logger;<a 
name="line.43"></a>
+<span class="sourceLineNo">044</span>import org.slf4j.LoggerFactory;<a 
name="line.44"></a>
+<span class="sourceLineNo">045</span>import 
org.apache.hadoop.hbase.client.ColumnFamilyDescriptorBuilder;<a 
name="line.45"></a>
+<span class="sourceLineNo">046</span>import 
org.apache.hadoop.hbase.client.TableDescriptor;<a name="line.46"></a>
+<span class="sourceLineNo">047</span>import 
org.apache.hadoop.hbase.client.TableDescriptorBuilder;<a name="line.47"></a>
+<span class="sourceLineNo">048</span>import 
org.apache.hadoop.hbase.Coprocessor;<a name="line.48"></a>
+<span class="sourceLineNo">049</span>import 
org.apache.hadoop.hbase.exceptions.DeserializationException;<a 
name="line.49"></a>
+<span class="sourceLineNo">050</span>import 
org.apache.hadoop.hbase.HConstants;<a name="line.50"></a>
+<span class="sourceLineNo">051</span>import 
org.apache.hadoop.hbase.regionserver.BloomType;<a name="line.51"></a>
+<span class="sourceLineNo">052</span>import 
org.apache.hbase.thirdparty.com.google.common.annotations.VisibleForTesting;<a 
name="line.52"></a>
+<span class="sourceLineNo">053</span>import 
org.apache.hbase.thirdparty.com.google.common.primitives.Ints;<a 
name="line.53"></a>
+<span class="sourceLineNo">054</span>import 
org.apache.hadoop.hbase.TableDescriptors;<a name="line.54"></a>
+<span class="sourceLineNo">055</span>import 
org.apache.hadoop.hbase.TableInfoMissingException;<a name="line.55"></a>
+<span class="sourceLineNo">056</span>import 
org.apache.hadoop.hbase.TableName;<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> * Implementation of {@link 
TableDescriptors} that reads descriptors from the<a name="line.59"></a>
+<span class="sourceLineNo">060</span> * passed filesystem.  It expects 
descriptors to be in a file in the<a name="line.60"></a>
+<span class="sourceLineNo">061</span> * {@link #TABLEINFO_DIR} subdir of the 
table's directory in FS.  Can be read-only<a name="line.61"></a>
+<span class="sourceLineNo">062</span> *  -- i.e. does not modify the 
filesystem or can be read and write.<a name="line.62"></a>
+<span class="sourceLineNo">063</span> *<a name="line.63"></a>
+<span class="sourceLineNo">064</span> * &lt;p&gt;Also has utility for keeping 
up the table descriptors tableinfo file.<a name="line.64"></a>
+<span class="sourceLineNo">065</span> * The table schema file is kept in the 
{@link #TABLEINFO_DIR} subdir<a name="line.65"></a>
+<span class="sourceLineNo">066</span> * of the table directory in the 
filesystem.<a name="line.66"></a>
+<span class="sourceLineNo">067</span> * It has a {@link 
#TABLEINFO_FILE_PREFIX} and then a suffix that is the<a name="line.67"></a>
+<span class="sourceLineNo">068</span> * edit sequenceid: e.g. 
&lt;code&gt;.tableinfo.0000000003&lt;/code&gt;.  This sequenceid<a 
name="line.68"></a>
+<span class="sourceLineNo">069</span> * is always increasing.  It starts at 
zero.  The table schema file with the<a name="line.69"></a>
+<span class="sourceLineNo">070</span> * highest sequenceid has the most recent 
schema edit. Usually there is one file<a name="line.70"></a>
+<span class="sourceLineNo">071</span> * only, the most recent but there may be 
short periods where there are more<a name="line.71"></a>
+<span class="sourceLineNo">072</span> * than one file. Old files are 
eventually cleaned.  Presumption is that there<a name="line.72"></a>
+<span class="sourceLineNo">073</span> * will not be lots of concurrent clients 
making table schema edits.  If so,<a name="line.73"></a>
+<span class="sourceLineNo">074</span> * the below needs a bit of a reworking 
and perhaps some supporting api in hdfs.<a name="line.74"></a>
+<span class="sourceLineNo">075</span> */<a name="line.75"></a>
+<span class="sourceLineNo">076</span>@InterfaceAudience.Private<a 
name="line.76"></a>
+<span class="sourceLineNo">077</span>public class FSTableDescriptors 
implements TableDescriptors {<a name="line.77"></a>
+<span class="sourceLineNo">078</span>  private static final Logger LOG = 
LoggerFactory.getLogger(FSTableDescriptors.class);<a name="line.78"></a>
+<span class="sourceLineNo">079</span>  private final FileSystem fs;<a 
name="line.79"></a>
+<span class="sourceLineNo">080</span>  private final Path rootdir;<a 
name="line.80"></a>
+<span class="sourceLineNo">081</span>  private final boolean fsreadonly;<a 
name="line.81"></a>
+<span class="sourceLineNo">082</span>  private volatile boolean usecache;<a 
name="line.82"></a>
+<span class="sourceLineNo">083</span>  private volatile boolean fsvisited;<a 
name="line.83"></a>
+<span class="sourceLineNo">084</span><a name="line.84"></a>
 <span class="sourceLineNo">085</span>  @VisibleForTesting<a name="line.85"></a>
-<span class="sourceLineNo">086</span>  long invocations = 0;<a 
name="line.86"></a>
-<span class="sourceLineNo">087</span><a name="line.87"></a>
-<span class="sourceLineNo">088</span>  /**<a name="line.88"></a>
-<span class="sourceLineNo">089</span>   * The file name prefix used to store 
HTD in HDFS<a name="line.89"></a>
-<span class="sourceLineNo">090</span>   */<a name="line.90"></a>
-<span class="sourceLineNo">091</span>  public static final String 
TABLEINFO_FILE_PREFIX = ".tableinfo";<a name="line.91"></a>
-<span class="sourceLineNo">092</span>  public static final String 
TABLEINFO_DIR = ".tabledesc";<a name="line.92"></a>
-<span class="sourceLineNo">093</span>  public static final String TMP_DIR = 
".tmp";<a name="line.93"></a>
-<span class="sourceLineNo">094</span><a name="line.94"></a>
-<span class="sourceLineNo">095</span>  // This cache does not age out the old 
stuff.  Thinking is that the amount<a name="line.95"></a>
-<span class="sourceLineNo">096</span>  // of data we keep up in here is so 
small, no need to do occasional purge.<a name="line.96"></a>
-<span class="sourceLineNo">097</span>  // TODO.<a name="line.97"></a>
-<span class="sourceLineNo">098</span>  private final Map&lt;TableName, 
TableDescriptor&gt; cache = new ConcurrentHashMap&lt;&gt;();<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>   * Table descriptor for 
&lt;code&gt;hbase:meta&lt;/code&gt; catalog table<a name="line.101"></a>
-<span class="sourceLineNo">102</span>   */<a name="line.102"></a>
-<span class="sourceLineNo">103</span>  private final TableDescriptor 
metaTableDescriptor;<a name="line.103"></a>
-<span class="sourceLineNo">104</span><a name="line.104"></a>
-<span class="sourceLineNo">105</span>  /**<a name="line.105"></a>
-<span class="sourceLineNo">106</span>   * Construct a FSTableDescriptors 
instance using the hbase root dir of the given<a name="line.106"></a>
-<span class="sourceLineNo">107</span>   * conf and the filesystem where that 
root dir lives.<a name="line.107"></a>
-<span class="sourceLineNo">108</span>   * This instance can do write 
operations (is not read only).<a name="line.108"></a>
-<span class="sourceLineNo">109</span>   */<a name="line.109"></a>
-<span class="sourceLineNo">110</span>  public FSTableDescriptors(final 
Configuration conf) throws IOException {<a name="line.110"></a>
-<span class="sourceLineNo">111</span>    this(conf, 
FSUtils.getCurrentFileSystem(conf), FSUtils.getRootDir(conf));<a 
name="line.111"></a>
-<span class="sourceLineNo">112</span>  }<a name="line.112"></a>
-<span class="sourceLineNo">113</span><a name="line.113"></a>
-<span class="sourceLineNo">114</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs, final Path rootdir)<a 
name="line.114"></a>
-<span class="sourceLineNo">115</span>          throws IOException {<a 
name="line.115"></a>
-<span class="sourceLineNo">116</span>    this(conf, fs, rootdir, false, 
true);<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>  /**<a name="line.119"></a>
-<span class="sourceLineNo">120</span>   * @param fsreadonly True if we are 
read-only when it comes to filesystem<a name="line.120"></a>
-<span class="sourceLineNo">121</span>   *                   operations; i.e. 
on remove, we do not do delete in fs.<a name="line.121"></a>
-<span class="sourceLineNo">122</span>   */<a name="line.122"></a>
-<span class="sourceLineNo">123</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs,<a name="line.123"></a>
-<span class="sourceLineNo">124</span>                            final Path 
rootdir, final boolean fsreadonly, final boolean usecache) throws IOException 
{<a name="line.124"></a>
-<span class="sourceLineNo">125</span>    this(conf, fs, rootdir, fsreadonly, 
usecache, null);<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>  /**<a name="line.128"></a>
-<span class="sourceLineNo">129</span>   * @param fsreadonly True if we are 
read-only when it comes to filesystem<a name="line.129"></a>
-<span class="sourceLineNo">130</span>   *                   operations; i.e. 
on remove, we do not do delete in fs.<a name="line.130"></a>
-<span class="sourceLineNo">131</span>   * @param metaObserver Used by HMaster. 
It need to modify the META_REPLICAS_NUM for meta table descriptor.<a 
name="line.131"></a>
-<span class="sourceLineNo">132</span>   *                     see 
HMaster#finishActiveMasterInitialization<a name="line.132"></a>
-<span class="sourceLineNo">133</span>   *                     TODO: This is a 
workaround. Should remove this ugly code...<a name="line.133"></a>
-<span class="sourceLineNo">134</span>   */<a name="line.134"></a>
-<span class="sourceLineNo">135</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs,<a name="line.135"></a>
-<span class="sourceLineNo">136</span>                            final Path 
rootdir, final boolean fsreadonly, final boolean usecache,<a 
name="line.136"></a>
-<span class="sourceLineNo">137</span>                            
Function&lt;TableDescriptorBuilder, TableDescriptorBuilder&gt; metaObserver) 
throws IOException {<a name="line.137"></a>
-<span class="sourceLineNo">138</span>    this.fs = fs;<a name="line.138"></a>
-<span class="sourceLineNo">139</span>    this.rootdir = rootdir;<a 
name="line.139"></a>
-<span class="sourceLineNo">140</span>    this.fsreadonly = fsreadonly;<a 
name="line.140"></a>
-<span class="sourceLineNo">141</span>    this.usecache = usecache;<a 
name="line.141"></a>
-<span class="sourceLineNo">142</span>    this.metaTableDescriptor = 
metaObserver == null ? createMetaTableDescriptor(conf)<a name="line.142"></a>
-<span class="sourceLineNo">143</span>          : 
metaObserver.apply(createMetaTableDescriptorBuilder(conf)).build();<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>  @VisibleForTesting<a 
name="line.146"></a>
-<span class="sourceLineNo">147</span>  public static TableDescriptorBuilder 
createMetaTableDescriptorBuilder(final Configuration conf) throws IOException 
{<a name="line.147"></a>
-<span class="sourceLineNo">148</span>    // TODO We used to set CacheDataInL1 
for META table. When we have BucketCache in file mode, now<a 
name="line.148"></a>
-<span class="sourceLineNo">149</span>    // the META table data goes to File 
mode BC only. Test how that affect the system. If too much,<a 
name="line.149"></a>
-<span class="sourceLineNo">150</span>    // we have to rethink about adding 
back the setCacheDataInL1 for META table CFs.<a name="line.150"></a>
-<span class="sourceLineNo">151</span>    return 
TableDescriptorBuilder.newBuilder(TableName.META_TABLE_NAME)<a 
name="line.151"></a>
-<span class="sourceLineNo">152</span>            
.addColumnFamily(ColumnFamilyDescriptorBuilder.newBuilder(HConstants.CATALOG_FAMILY)<a
 name="line.152"></a>
-<span class="sourceLineNo">153</span>                    
.setMaxVersions(conf.getInt(HConstants.HBASE_META_VERSIONS,<a 
name="line.153"></a>
-<span class="sourceLineNo">154</span>                            
HConstants.DEFAULT_HBASE_META_VERSIONS))<a name="line.154"></a>
-<span class="sourceLineNo">155</span>                    .setInMemory(true)<a 
name="line.155"></a>
-<span class="sourceLineNo">156</span>                    
.setBlocksize(conf.getInt(HConstants.HBASE_META_BLOCK_SIZE,<a 
name="line.156"></a>
-<span class="sourceLineNo">157</span>                            
HConstants.DEFAULT_HBASE_META_BLOCK_SIZE))<a name="line.157"></a>
-<span class="sourceLineNo">158</span>                    
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.158"></a>
-<span class="sourceLineNo">159</span>                    // Disable blooms for 
meta.  Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.159"></a>
-<span class="sourceLineNo">160</span>                    
.setBloomFilterType(BloomType.NONE)<a name="line.160"></a>
-<span class="sourceLineNo">161</span>                    .build())<a 
name="line.161"></a>
-<span class="sourceLineNo">162</span>            
.addColumnFamily(ColumnFamilyDescriptorBuilder.newBuilder(HConstants.TABLE_FAMILY)<a
 name="line.162"></a>
-<span class="sourceLineNo">163</span>                    
.setMaxVersions(conf.getInt(HConstants.HBASE_META_VERSIONS,<a 
name="line.163"></a>
-<span class="sourceLineNo">164</span>                        
HConstants.DEFAULT_HBASE_META_VERSIONS))<a name="line.164"></a>
-<span class="sourceLineNo">165</span>                    .setInMemory(true)<a 
name="line.165"></a>
-<span class="sourceLineNo">166</span>                    .setBlocksize(8 * 
1024)<a name="line.166"></a>
-<span class="sourceLineNo">167</span>                    
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.167"></a>
-<span class="sourceLineNo">168</span>                    // Disable blooms for 
meta.  Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.168"></a>
-<span class="sourceLineNo">169</span>                    
.setBloomFilterType(BloomType.NONE)<a name="line.169"></a>
-<span class="sourceLineNo">170</span>                    .build())<a 
name="line.170"></a>
-<span class="sourceLineNo">171</span>            
.addColumnFamily(ColumnFamilyDescriptorBuilder<a name="line.171"></a>
-<span class="sourceLineNo">172</span>                    
.newBuilder(HConstants.REPLICATION_BARRIER_FAMILY)<a name="line.172"></a>
-<span class="sourceLineNo">173</span>                    
.setMaxVersions(HConstants.ALL_VERSIONS)<a name="line.173"></a>
-<span class="sourceLineNo">174</span>                    .setInMemory(true)<a 
name="line.174"></a>
-<span class="sourceLineNo">175</span>                    
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.175"></a>
-<span class="sourceLineNo">176</span>                    // Disable blooms for 
meta.  Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.176"></a>
-<span class="sourceLineNo">177</span>                    
.setBloomFilterType(BloomType.NONE)<a name="line.177"></a>
-<span class="sourceLineNo">178</span>                    .build())<a 
name="line.178"></a>
-<span class="sourceLineNo">179</span>            
.addCoprocessor("org.apache.hadoop.hbase.coprocessor.MultiRowMutationEndpoint",<a
 name="line.179"></a>
-<span class="sourceLineNo">180</span>                    null, 
Coprocessor.PRIORITY_SYSTEM, null);<a name="line.180"></a>
-<span class="sourceLineNo">181</span>  }<a name="line.181"></a>
-<span class="sourceLineNo">182</span><a name="line.182"></a>
-<span class="sourceLineNo">183</span>  @VisibleForTesting<a 
name="line.183"></a>
-<span class="sourceLineNo">184</span>  public static TableDescriptor 
createMetaTableDescriptor(final Configuration conf)<a name="line.184"></a>
-<span class="sourceLineNo">185</span>      throws IOException {<a 
name="line.185"></a>
-<span class="sourceLineNo">186</span>    return 
createMetaTableDescriptorBuilder(conf).build();<a name="line.186"></a>
-<span class="sourceLineNo">187</span>  }<a name="line.187"></a>
-<span class="sourceLineNo">188</span><a name="line.188"></a>
-<span class="sourceLineNo">189</span>  @Override<a name="line.189"></a>
-<span class="sourceLineNo">190</span>  public void setCacheOn() throws 
IOException {<a name="line.190"></a>
-<span class="sourceLineNo">191</span>    this.cache.clear();<a 
name="line.191"></a>
-<span class="sourceLineNo">192</span>    this.usecache = true;<a 
name="line.192"></a>
-<span class="sourceLineNo">193</span>  }<a name="line.193"></a>
-<span class="sourceLineNo">194</span><a name="line.194"></a>
-<span class="sourceLineNo">195</span>  @Override<a name="line.195"></a>
-<span class="sourceLineNo">196</span>  public void setCacheOff() throws 
IOException {<a name="line.196"></a>
-<span class="sourceLineNo">197</span>    this.usecache = false;<a 
name="line.197"></a>
-<span class="sourceLineNo">198</span>    this.cache.clear();<a 
name="line.198"></a>
-<span class="sourceLineNo">199</span>  }<a name="line.199"></a>
-<span class="sourceLineNo">200</span><a name="line.200"></a>
-<span class="sourceLineNo">201</span>  @VisibleForTesting<a 
name="line.201"></a>
-<span class="sourceLineNo">202</span>  public boolean isUsecache() {<a 
name="line.202"></a>
-<span class="sourceLineNo">203</span>    return this.usecache;<a 
name="line.203"></a>
-<span class="sourceLineNo">204</span>  }<a name="line.204"></a>
-<span class="sourceLineNo">205</span><a name="line.205"></a>
-<span class="sourceLineNo">206</span>  /**<a name="line.206"></a>
-<span class="sourceLineNo">207</span>   * Get the current table descriptor for 
the given table, or null if none exists.<a name="line.207"></a>
-<span class="sourceLineNo">208</span>   *<a name="line.208"></a>
-<span class="sourceLineNo">209</span>   * Uses a local cache of the descriptor 
but still checks the filesystem on each call<a name="line.209"></a>
-<span class="sourceLineNo">210</span>   * to see if a newer file has been 
created since the cached one was read.<a name="line.210"></a>
-<span class="sourceLineNo">211</span>   */<a name="line.211"></a>
-<span class="sourceLineNo">212</span>  @Override<a name="line.212"></a>
-<span class="sourceLineNo">213</span>  @Nullable<a name="line.213"></a>
-<span class="sourceLineNo">214</span>  public TableDescriptor get(final 
TableName tablename)<a name="line.214"></a>
-<span class="sourceLineNo">215</span>  throws IOException {<a 
name="line.215"></a>
-<span class="sourceLineNo">216</span>    invocations++;<a name="line.216"></a>
-<span class="sourceLineNo">217</span>    if 
(TableName.META_TABLE_NAME.equals(tablename)) {<a name="line.217"></a>
-<span class="sourceLineNo">218</span>      cachehits++;<a name="line.218"></a>
-<span class="sourceLineNo">219</span>      return metaTableDescriptor;<a 
name="line.219"></a>
-<span class="sourceLineNo">220</span>    }<a name="line.220"></a>
-<span class="sourceLineNo">221</span>    // hbase:meta is already handled. If 
some one tries to get the descriptor for<a name="line.221"></a>
-<span class="sourceLineNo">222</span>    // .logs, .oldlogs or .corrupt throw 
an exception.<a name="line.222"></a>
-<span class="sourceLineNo">223</span>    if 
(HConstants.HBASE_NON_USER_TABLE_DIRS.contains(tablename.getNameAsString())) 
{<a name="line.223"></a>
-<span class="sourceLineNo">224</span>       throw new IOException("No 
descriptor found for non table = " + tablename);<a name="line.224"></a>
-<span class="sourceLineNo">225</span>    }<a name="line.225"></a>
-<span class="sourceLineNo">226</span><a name="line.226"></a>
-<span class="sourceLineNo">227</span>    if (usecache) {<a name="line.227"></a>
-<span class="sourceLineNo">228</span>      // Look in cache of descriptors.<a 
name="line.228"></a>
-<span class="sourceLineNo">229</span>      TableDescriptor cachedtdm = 
this.cache.get(tablename);<a name="line.229"></a>
-<span class="sourceLineNo">230</span>      if (cachedtdm != null) {<a 
name="line.230"></a>
-<span class="sourceLineNo">231</span>        cachehits++;<a 
name="line.231"></a>
-<span class="sourceLineNo">232</span>        return cachedtdm;<a 
name="line.232"></a>
-<span class="sourceLineNo">233</span>      }<a name="line.233"></a>
-<span class="sourceLineNo">234</span>    }<a name="line.234"></a>
-<span class="sourceLineNo">235</span>    TableDescriptor tdmt = null;<a 
name="line.235"></a>
-<span class="sourceLineNo">236</span>    try {<a name="line.236"></a>
-<span class="sourceLineNo">237</span>      tdmt = getTableDescriptorFromFs(fs, 
rootdir, tablename);<a name="line.237"></a>
-<span class="sourceLineNo">238</span>    } catch (NullPointerException e) {<a 
name="line.238"></a>
-<span class="sourceLineNo">239</span>      LOG.debug("Exception during 
readTableDecriptor. Current table name = "<a name="line.239"></a>
-<span class="sourceLineNo">240</span>          + tablename, e);<a 
name="line.240"></a>
-<span class="sourceLineNo">241</span>    } catch (TableInfoMissingException e) 
{<a name="line.241"></a>
-<span class="sourceLineNo">242</span>      // ignore. This is regular 
operation<a name="line.242"></a>
-<span class="sourceLineNo">243</span>    } catch (IOException ioe) {<a 
name="line.243"></a>
-<span class="sourceLineNo">244</span>      LOG.debug("Exception during 
readTableDecriptor. Current table name = "<a name="line.244"></a>
-<span class="sourceLineNo">245</span>          + tablename, ioe);<a 
name="line.245"></a>
-<span class="sourceLineNo">246</span>    }<a name="line.246"></a>
-<span class="sourceLineNo">247</span>    // last HTD written wins<a 
name="line.247"></a>
-<span class="sourceLineNo">248</span>    if (usecache &amp;&amp; tdmt != null) 
{<a name="line.248"></a>
-<span class="sourceLineNo">249</span>      this.cache.put(tablename, tdmt);<a 
name="line.249"></a>
-<span class="sourceLineNo">250</span>    }<a name="line.250"></a>
-<span class="sourceLineNo">251</span><a name="line.251"></a>
-<span class="sourceLineNo">252</span>    return tdmt;<a name="line.252"></a>
-<span class="sourceLineNo">253</span>  }<a name="line.253"></a>
+<span class="sourceLineNo">086</span>  long cachehits = 0;<a 
name="line.86"></a>
+<span class="sourceLineNo">087</span>  @VisibleForTesting<a name="line.87"></a>
+<span class="sourceLineNo">088</span>  long invocations = 0;<a 
name="line.88"></a>
+<span class="sourceLineNo">089</span><a name="line.89"></a>
+<span class="sourceLineNo">090</span>  /**<a name="line.90"></a>
+<span class="sourceLineNo">091</span>   * The file name prefix used to store 
HTD in HDFS<a name="line.91"></a>
+<span class="sourceLineNo">092</span>   */<a name="line.92"></a>
+<span class="sourceLineNo">093</span>  public static final String 
TABLEINFO_FILE_PREFIX = ".tableinfo";<a name="line.93"></a>
+<span class="sourceLineNo">094</span>  public static final String 
TABLEINFO_DIR = ".tabledesc";<a name="line.94"></a>
+<span class="sourceLineNo">095</span>  public static final String TMP_DIR = 
".tmp";<a name="line.95"></a>
+<span class="sourceLineNo">096</span><a name="line.96"></a>
+<span class="sourceLineNo">097</span>  // This cache does not age out the old 
stuff.  Thinking is that the amount<a name="line.97"></a>
+<span class="sourceLineNo">098</span>  // of data we keep up in here is so 
small, no need to do occasional purge.<a name="line.98"></a>
+<span class="sourceLineNo">099</span>  // TODO.<a name="line.99"></a>
+<span class="sourceLineNo">100</span>  private final Map&lt;TableName, 
TableDescriptor&gt; cache = new ConcurrentHashMap&lt;&gt;();<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>   * Table descriptor for 
&lt;code&gt;hbase:meta&lt;/code&gt; catalog table<a name="line.103"></a>
+<span class="sourceLineNo">104</span>   */<a name="line.104"></a>
+<span class="sourceLineNo">105</span>  private final TableDescriptor 
metaTableDescriptor;<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>   * Construct a FSTableDescriptors 
instance using the hbase root dir of the given<a name="line.108"></a>
+<span class="sourceLineNo">109</span>   * conf and the filesystem where that 
root dir lives.<a name="line.109"></a>
+<span class="sourceLineNo">110</span>   * This instance can do write 
operations (is not read only).<a name="line.110"></a>
+<span class="sourceLineNo">111</span>   */<a name="line.111"></a>
+<span class="sourceLineNo">112</span>  public FSTableDescriptors(final 
Configuration conf) throws IOException {<a name="line.112"></a>
+<span class="sourceLineNo">113</span>    this(conf, 
FSUtils.getCurrentFileSystem(conf), FSUtils.getRootDir(conf));<a 
name="line.113"></a>
+<span class="sourceLineNo">114</span>  }<a name="line.114"></a>
+<span class="sourceLineNo">115</span><a name="line.115"></a>
+<span class="sourceLineNo">116</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs, final Path rootdir)<a 
name="line.116"></a>
+<span class="sourceLineNo">117</span>          throws IOException {<a 
name="line.117"></a>
+<span class="sourceLineNo">118</span>    this(conf, fs, rootdir, false, 
true);<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>   * @param fsreadonly True if we are 
read-only when it comes to filesystem<a name="line.122"></a>
+<span class="sourceLineNo">123</span>   *                   operations; i.e. 
on remove, we do not do delete in fs.<a name="line.123"></a>
+<span class="sourceLineNo">124</span>   */<a name="line.124"></a>
+<span class="sourceLineNo">125</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs,<a name="line.125"></a>
+<span class="sourceLineNo">126</span>                            final Path 
rootdir, final boolean fsreadonly, final boolean usecache) throws IOException 
{<a name="line.126"></a>
+<span class="sourceLineNo">127</span>    this(conf, fs, rootdir, fsreadonly, 
usecache, null);<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>  /**<a name="line.130"></a>
+<span class="sourceLineNo">131</span>   * @param fsreadonly True if we are 
read-only when it comes to filesystem<a name="line.131"></a>
+<span class="sourceLineNo">132</span>   *                   operations; i.e. 
on remove, we do not do delete in fs.<a name="line.132"></a>
+<span class="sourceLineNo">133</span>   * @param metaObserver Used by HMaster. 
It need to modify the META_REPLICAS_NUM for meta table descriptor.<a 
name="line.133"></a>
+<span class="sourceLineNo">134</span>   *                     see 
HMaster#finishActiveMasterInitialization<a name="line.134"></a>
+<span class="sourceLineNo">135</span>   *                     TODO: This is a 
workaround. Should remove this ugly code...<a name="line.135"></a>
+<span class="sourceLineNo">136</span>   */<a name="line.136"></a>
+<span class="sourceLineNo">137</span>  public FSTableDescriptors(final 
Configuration conf, final FileSystem fs,<a name="line.137"></a>
+<span class="sourceLineNo">138</span>                            final Path 
rootdir, final boolean fsreadonly, final boolean usecache,<a 
name="line.138"></a>
+<span class="sourceLineNo">139</span>                            
Function&lt;TableDescriptorBuilder, TableDescriptorBuilder&gt; metaObserver) 
throws IOException {<a name="line.139"></a>
+<span class="sourceLineNo">140</span>    this.fs = fs;<a name="line.140"></a>
+<span class="sourceLineNo">141</span>    this.rootdir = rootdir;<a 
name="line.141"></a>
+<span class="sourceLineNo">142</span>    this.fsreadonly = fsreadonly;<a 
name="line.142"></a>
+<span class="sourceLineNo">143</span>    this.usecache = usecache;<a 
name="line.143"></a>
+<span class="sourceLineNo">144</span>    this.metaTableDescriptor = 
metaObserver == null ? createMetaTableDescriptor(conf)<a name="line.144"></a>
+<span class="sourceLineNo">145</span>          : 
metaObserver.apply(createMetaTableDescriptorBuilder(conf)).build();<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>  @VisibleForTesting<a 
name="line.148"></a>
+<span class="sourceLineNo">149</span>  public static TableDescriptorBuilder 
createMetaTableDescriptorBuilder(final Configuration conf) throws IOException 
{<a name="line.149"></a>
+<span class="sourceLineNo">150</span>    // TODO We used to set CacheDataInL1 
for META table. When we have BucketCache in file mode, now<a 
name="line.150"></a>
+<span class="sourceLineNo">151</span>    // the META table data goes to File 
mode BC only. Test how that affect the system. If too much,<a 
name="line.151"></a>
+<span class="sourceLineNo">152</span>    // we have to rethink about adding 
back the setCacheDataInL1 for META table CFs.<a name="line.152"></a>
+<span class="sourceLineNo">153</span>    return 
TableDescriptorBuilder.newBuilder(TableName.META_TABLE_NAME)<a 
name="line.153"></a>
+<span class="sourceLineNo">154</span>      
.setColumnFamily(ColumnFamilyDescriptorBuilder.newBuilder(HConstants.CATALOG_FAMILY)<a
 name="line.154"></a>
+<span class="sourceLineNo">155</span>        
.setMaxVersions(conf.getInt(HConstants.HBASE_META_VERSIONS,<a 
name="line.155"></a>
+<span class="sourceLineNo">156</span>                
HConstants.DEFAULT_HBASE_META_VERSIONS))<a name="line.156"></a>
+<span class="sourceLineNo">157</span>        .setInMemory(true)<a 
name="line.157"></a>
+<span class="sourceLineNo">158</span>        
.setBlocksize(conf.getInt(HConstants.HBASE_META_BLOCK_SIZE,<a 
name="line.158"></a>
+<span class="sourceLineNo">159</span>                
HConstants.DEFAULT_HBASE_META_BLOCK_SIZE))<a name="line.159"></a>
+<span class="sourceLineNo">160</span>        
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.160"></a>
+<span class="sourceLineNo">161</span>        // Disable blooms for meta.  
Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.161"></a>
+<span class="sourceLineNo">162</span>        
.setBloomFilterType(BloomType.NONE)<a name="line.162"></a>
+<span class="sourceLineNo">163</span>        .build())<a name="line.163"></a>
+<span class="sourceLineNo">164</span>      
.setColumnFamily(ColumnFamilyDescriptorBuilder.newBuilder(HConstants.TABLE_FAMILY)<a
 name="line.164"></a>
+<span class="sourceLineNo">165</span>        
.setMaxVersions(conf.getInt(HConstants.HBASE_META_VERSIONS,<a 
name="line.165"></a>
+<span class="sourceLineNo">166</span>            
HConstants.DEFAULT_HBASE_META_VERSIONS))<a name="line.166"></a>
+<span class="sourceLineNo">167</span>        .setInMemory(true)<a 
name="line.167"></a>
+<span class="sourceLineNo">168</span>        .setBlocksize(8 * 1024)<a 
name="line.168"></a>
+<span class="sourceLineNo">169</span>        
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.169"></a>
+<span class="sourceLineNo">170</span>        // Disable blooms for meta.  
Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.170"></a>
+<span class="sourceLineNo">171</span>        
.setBloomFilterType(BloomType.NONE)<a name="line.171"></a>
+<span class="sourceLineNo">172</span>        .build())<a name="line.172"></a>
+<span class="sourceLineNo">173</span>      
.setColumnFamily(ColumnFamilyDescriptorBuilder<a name="line.173"></a>
+<span class="sourceLineNo">174</span>        
.newBuilder(HConstants.REPLICATION_BARRIER_FAMILY)<a name="line.174"></a>
+<span class="sourceLineNo">175</span>        
.setMaxVersions(HConstants.ALL_VERSIONS)<a name="line.175"></a>
+<span class="sourceLineNo">176</span>        .setInMemory(true)<a 
name="line.176"></a>
+<span class="sourceLineNo">177</span>        
.setScope(HConstants.REPLICATION_SCOPE_LOCAL)<a name="line.177"></a>
+<span class="sourceLineNo">178</span>        // Disable blooms for meta.  
Needs work.  Seems to mess w/ getClosestOrBefore.<a name="line.178"></a>
+<span class="sourceLineNo">179</span>        
.setBloomFilterType(BloomType.NONE)<a name="line.179"></a>
+<span class="sourceLineNo">180</span>        .build())<a name="line.180"></a>
+<span class="sourceLineNo">181</span>      
.setCoprocessor(CoprocessorDescriptorBuilder.newBuilder(<a name="line.181"></a>
+<span class="sourceLineNo">182</span>        
MultiRowMutationEndpoint.class.getName())<a name="line.182"></a>
+<span class="sourceLineNo">183</span>        
.setPriority(Coprocessor.PRIORITY_SYSTEM).build());<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>  @VisibleForTesting<a 
name="line.186"></a>
+<span class="sourceLineNo">187</span>  public static TableDescriptor 
createMetaTableDescriptor(final Configuration conf)<a name="line.187"></a>
+<span class="sourceLineNo">188</span>      throws IOException {<a 
name="line.188"></a>
+<span class="sourceLineNo">189</span>    return 
createMetaTableDescriptorBuilder(conf).build();<a name="line.189"></a>
+<span class="sourceLineNo">190</span>  }<a name="line.190"></a>
+<span class="sourceLineNo">191</span><a name="line.191"></a>
+<span class="sourceLineNo">192</span>  @Override<a name="line.192"></a>
+<span class="sourceLineNo">193</span>  public void setCacheOn() throws 
IOException {<a name="line.193"></a>
+<span class="sourceLineNo">194</span>    this.cache.clear();<a 
name="line.194"></a>
+<span class="sourceLineNo">195</span>    this.usecache = true;<a 
name="line.195"></a>
+<span class="sourceLineNo">196</span>  }<a name="line.196"></a>
+<span class="sourceLineNo">197</span><a name="line.197"></a>
+<span class="sourceLineNo">198</span>  @Override<a name="line.198"></a>
+<span class="sourceLineNo">199</span>  public void setCacheOff() throws 
IOException {<a name="line.199"></a>
+<span class="sourceLineNo">200</span>    this.usecache = false;<a 
name="line.200"></a>
+<span class="sourceLineNo">201</span>    this.cache.clear();<a 
name="line.201"></a>
+<span class="sourceLineNo">202</span>  }<a name="line.202"></a>
+<span class="sourceLineNo">203</span><a name="line.203"></a>
+<span class="sourceLineNo">204</span>  @VisibleForTesting<a 
name="line.204"></a>
+<span class="sourceLineNo">205</span>  public boolean isUsecache() {<a 
name="line.205"></a>
+<span class="sourceLineNo">206</span>    return this.usecache;<a 
name="line.206"></a>
+<span class="sourceLineNo">207</span>  }<a name="line.207"></a>
+<span class="sourceLineNo">208</span><a name="line.208"></a>
+<span class="sourceLineNo">209</span>  /**<a name="line.209"></a>
+<span class="sourceLineNo">210</span>   * Get the current table descriptor for 
the given table, or null if none exists.<a name="line.210"></a>
+<span class="sourceLineNo">211</span>   *<a name="line.211"></a>
+<span class="sourceLineNo">212</span>   * Uses a local cache of the descriptor 
but still checks the filesystem on each call<a name="line.212"></a>
+<span class="sourceLineNo">213</span>   * to see if a newer file has been 
created since the cached one was read.<a name="line.213"></a>
+<span class="sourceLineNo">214</span>   */<a name="line.214"></a>
+<span class="sourceLineNo">215</span>  @Override<a name="line.215"></a>
+<span class="sourceLineNo">216</span>  @Nullable<a name="line.216"></a>
+<span class="sourceLineNo">217</span>  public TableDescriptor get(final 
TableName tablename)<a name="line.217"></a>
+<span class="sourceLineNo">218</span>  throws IOException {<a 
name="line.218"></a>
+<span class="sourceLineNo">219</span>    invocations++;<a name="line.219"></a>
+<span class="sourceLineNo">220</span>    if 
(TableName.META_TABLE_NAME.equals(tablename)) {<a name="line.220"></a>
+<span class="sourceLineNo">221</span>      cachehits++;<a name="line.221"></a>
+<span class="sourceLineNo">222</span>      return metaTableDescriptor;<a 
name="line.222"></a>
+<span class="sourceLineNo">223</span>    }<a name="line.223"></a>
+<span class="sourceLineNo">224</span>    // hbase:meta is already handled. If 
some one tries to get the descriptor for<a name="line.224"></a>
+<span class="sourceLineNo">225</span>    // .logs, .oldlogs or .corrupt throw 
an exception.<a name="line.225"></a>
+<span class="sourceLineNo">226</span>    if 
(HConstants.HBASE_NON_USER_TABLE_DIRS.contains(tablename.getNameAsString())) 
{<a name="line.226"></a>
+<span class="sourceLineNo">227</span>       throw new IOException("No 
descriptor found for non table = " + tablename);<a name="line.227"></a>
+<span class="sourceLineNo">228</span>    }<a name="line.228"></a>
+<span class="sourceLineNo">229</span><a name="line.229"></a>
+<span class="sourceLineNo">230</span>    if (usecache) {<a name="line.230"></a>
+<span class="sourceLineNo">231</span>      // Look in cache of descriptors.<a 
name="line.231"></a>
+<span class="sourceLineNo">232</span>      TableDescriptor cachedtdm = 
this.cache.get(tablename);<a name="line.232"></a>
+<span class="sourceLineNo">233</span>      if (cachedtdm != null) {<a 
name="line.233"></a>
+<span class="sourceLineNo">234</span>        cachehits++;<a 
name="line.234"></a>
+<span class="sourceLineNo">235</span>        return cachedtdm;<a 
name="line.235"></a>
+<span class="sourceLineNo">236</span>      }<a name="line.236"></a>
+<span class="sourceLineNo">237</span>    }<a name="line.237"></a>
+<span class="sourceLineNo">238</span>    TableDescriptor tdmt = null;<a 
name="line.238"></a>
+<span class="sourceLineNo">239</span>    try {<a name="line.239"></a>
+<span class="sourceLineNo">240</span>      tdmt = getTableDescriptorFromFs(fs, 
rootdir, tablename);<a name="line.240"></a>
+<span class="sourceLineNo">241</span>    } catch (NullPointerException e) {<a 
name="line.241"></a>
+<span class="sourceLineNo">242</span>      LOG.debug("Exception during 
readTableDecriptor. Current table name = "<a name="line.242"></a>
+<span class="sourceLineNo">243</span>          + tablename, e);<a 
name="line.243"></a>
+<span class="sourceLineNo">244</span>    } catch (TableInfoMissingException e) 
{<a name="line.244"></a>
+<span class="sourceLineNo">245</span>      // ignore. This is regular 
operation<a name="line.245"></a>
+<span class="sourceLineNo">246</span>    } catch (IOException ioe) {<a 
name="line.246"></a>
+<span class="sourceLineNo">247</span>      LOG.debug("Exception during 
readTableDecriptor. Current table name = "<a name="line.247"></a>
+<span class="sourceLineNo">248</span>          + tablename, ioe);<a 
name="line.248"></a>
+<span class="sourceLineNo">249</span>    }<a name="line.249"></a>
+<span class="sourceLineNo">250</span>    // last HTD written wins<a 
name="line.250"></a>
+<span class="sourceLineNo">251</span>    if (usecache &amp;&amp; tdmt != null) 
{<a name="line.251"></a>
+<span class="sourceLineNo">252</span>      this.cache.put(tablename, tdmt);<a 
name="line.252"></a>
+<span class="sourceLineNo">253</span>    }<a name="line.253"></a>
 <span class="sourceLineNo">254</span><a name="line.254"></a>
-<span class="sourceLineNo">255</span>  /**<a name="line.255"></a>
-<span class="sourceLineNo">256</span>   * Returns a map from table name to 
table descriptor for all tables.<a name="line.256"></a>
-<span class="sourceLineNo">257</span>   */<a name="line.257"></a>
-<span class="sourceLineNo">258</span>  @Override<a name="line.258"></a>
-<span class="sourceLineNo">259</span>  public Map&lt;String, 
TableDescriptor&gt; getAll()<a name="line.259"></a>
-<span class="sourceLineNo">260</span>  throws IOException {<a 
name="line.260"></a>
-<span class="sourceLineNo">261</span>    Map&lt;String, TableDescriptor&gt; 
tds = new TreeMap&lt;&gt;();<a name="line.261"></a>
-<span class="sourceLineNo">262</span><a name="line.262"></a>
-<span class="sourceLineNo">263</span>    if (fsvisited &amp;&amp; usecache) 
{<a name="line.263"></a>
-<span class="sourceLineNo">264</span>      for (Map.Entry&lt;TableName, 
TableDescriptor&gt; entry: this.cache.entrySet()) {<a name="line.264"></a>
-<span class="sourceLineNo">265</span>        
tds.put(entry.getKey().toString(), entry.getValue());<a name="line.265"></a>
-<span class="sourceLineNo">266</span>      }<a name="line.266"></a>
-<span class="sourceLineNo">267</span>      // add hbase:meta to the response<a 
name="line.267"></a>
-<span class="sourceLineNo">268</span>      
tds.put(this.metaTableDescriptor.getTableName().getNameAsString(), 
metaTableDescriptor);<a name="line.268"></a>
-<span class="sourceLineNo">269</span>    } else {<a name="line.269"></a>
-<span class="sourceLineNo">270</span>      LOG.trace("Fetching table 
descriptors from the filesystem.");<a name="line.270"></a>
-<span class="sourceLineNo">271</span>      boolean allvisited = true;<a 
name="line.271"></a>
-<span class="sourceLineNo">272</span>      for (Path d : 
FSUtils.getTableDirs(fs, rootdir)) {<a name="line.272"></a>
-<span class="sourceLineNo">273</span>        TableDescriptor htd = null;<a 
name="line.273"></a>
-<span class="sourceLineNo">274</span>        try {<a name="line.274"></a>
-<span class="sourceLineNo">275</span>          htd = 
get(FSUtils.getTableName(d));<a name="line.275"></a>
-<span class="sourceLineNo">276</span>        } catch (FileNotFoundException 
fnfe) {<a name="line.276"></a>
-<span class="sourceLineNo">277</span>          // inability of retrieving one 
HTD shouldn't stop getting the remaining<a name="line.277"></a>
-<span class="sourceLineNo">278</span>          LOG.warn("Trouble retrieving 
htd", fnfe);<a name="line.278"></a>
-<span class="sourceLineNo">279</span>        }<a name="line.279"></a>
-<span class="sourceLineNo">280</span>        if (htd == null) {<a 
name="line.280"></a>
-<span class="sourceLineNo">281</span>          allvisited = false;<a 
name="line.281"></a>
-<span class="sourceLineNo">282</span>          continue;<a name="line.282"></a>
-<span class="sourceLineNo">283</span>        } else {<a name="line.283"></a>
-<span class="sourceLineNo">284</span>          
tds.put(htd.getTableName().getNameAsString(), htd);<a name="line.284"></a>
-<span class="sourceLineNo">285</span>        }<a name="line.285"></a>
-<span class="sourceLineNo">286</span>        fsvisited = allvisited;<a 
name="line.286"></a>
-<span class="sourceLineNo">287</span>      }<a name="line.287"></a>
-<span class="sourceLineNo">288</span>    }<a name="line.288"></a>
-<span class="sourceLineNo">289</span>    return tds;<a name="line.289"></a>
-<span class="sourceLineNo">290</span>  }<a name="line.290"></a>
-<span class="sourceLineNo">291</span><a name="line.291"></a>
-<span class="sourceLineNo">292</span>  /**<a name="line.292"></a>
-<span class="sourceLineNo">293</span>    * Find descriptors by namespace.<a 
name="line.293"></a>
-<span class="sourceLineNo">294</span>    * @see 
#get(org.apache.hadoop.hbase.TableName)<a name="line.294"></a>
-<span class="sourceLineNo">295</span>    */<a name="line.295"></a>
-<span class="sourceLineNo">296</span>  @Override<a name="line.296"></a>
-<span class="sourceLineNo">297</span>  public Map&lt;String, 
TableDescriptor&gt; getByNamespace(String name)<a name="line.297"></a>
-<span class="sourceLineNo">298</span>  throws IOException {<a 
name="line.298"></a>
-<span class="sourceLineNo">299</span>    Map&lt;String, TableDescriptor&gt; 
htds = new TreeMap&lt;&gt;();<a name="line.299"></a>
-<span class="sourceLineNo">300</span>    List&lt;Path&gt; tableDirs =<a 
name="line.300"></a>
-<span class="sourceLineNo">301</span>        FSUtils.getLocalTableDirs(fs, 
FSUtils.getNamespaceDir(rootdir, name));<a name="line.301"></a>
-<span class="sourceLineNo">302</span>    for (Path d: tableDirs) {<a 
name="line.302"></a>
-<span class="sourceLineNo">303</span>      TableDescriptor htd = null;<a 
name="line.303"></a>
-<span class="sourceLineNo">304</span>      try {<a name="line.304"></a>
-<span class="sourceLineNo">305</span>        htd = 
get(FSUtils.getTableName(d));<a name="line.305"></a>
-<span class="sourceLineNo">306</span>      } catch (FileNotFoundException 
fnfe) {<a name="line.306"></a>
-<span class="sourceLineNo">307</span>        // inability of retrieving one 
HTD shouldn't stop getting the remaining<a name="line.307"></a>
-<span class="sourceLineNo">308</span>        LOG.warn("Trouble retrieving 
htd", fnfe);<a name="line.308"></a>
-<span class="sourceLineNo">309</span>      }<a name="line.309"></a>
-<span class="sourceLineNo">310</span>      if (htd == null) continue;<a 
name="line.310"></a>
-<span class="sourceLineNo">311</span>      
htds.put(FSUtils.getTableName(d).getNameAsString(), htd);<a name="line.311"></a>
-<span class="sourceLineNo">312</span>    }<a name="line.312"></a>
-<span class="sourceLineNo">313</span>    return htds;<a name="line.313"></a>
-<span class="sourceLineNo">314</span>  }<a name="line.314"></a>
-<span class="sourceLineNo">315</span><a name="line.315"></a>
-<span class="sourceLineNo">316</span>  /**<a name="line.316"></a>
-<span class="sourceLineNo">317</span>   * Adds (or updates) the table 
descriptor to the FileSystem<a name="line.317"></a>
-<span class="sourceLineNo">318</span>   * and updates the local cache with 
it.<a name="line.318"></a>
-<span class="sourceLineNo">319</span>   */<a name="line.319"></a>
-<span class="sourceLineNo">320</span>  @Override<a name="line.320"></a>
-<span class="sourceLineNo">321</span>  public void add(TableDescriptor htd) 
throws IOException {<a name="line.321"></a>
-<span class="sourceLineNo">322</span>    if (fsreadonly) {<a 
name="line.322"></a>
-<span class="sourceLineNo">323</span>      throw new 
NotImplementedException("Cannot add a table descriptor - in read only mode");<a 
name="line.323"></a>
-<span class="sourceLineNo">324</span>    }<a name="line.324"></a>
-<span class="sourceLineNo">325</span>    TableName tableName = 
htd.getTableName();<a name="line.325"></a>
-<span class="sourceLineNo">326</span>    if 
(TableName.META_TABLE_NAME.equals(tableName)) {<a name="line.326"></a>
-<span class="sourceLineNo">327</span>      throw new 
NotImplementedException(HConstants.NOT_IMPLEMENTED);<a name="line.327"></a>
-<span class="sourceLineNo">328</span>    }<a name="line.328"></a>
-<span class="sourceLineNo">329</span>    if 
(HConstants.HBASE_NON_USER_TABLE_DIRS.contains(tableName.getNameAsString())) 
{<a name="line.329"></a>
-<span class="sourceLineNo">330</span>      throw new 
NotImplementedException(<a name="line.330"></a>
-<span class="sourceLineNo">331</span>          "Cannot add a table descriptor 
for a reserved subdirectory name: "<a name="line.331"></a>
-<span class="sourceLineNo">332</span>              + 
htd.getTableName().getNameAsString());<a name="line.332"></a>
-<span class="sourceLineNo">333</span>    }<a name="line.333"></a>
-<span class="sourceLineNo">334</span>    updateTableDescriptor(htd);<a 
name="line.334"></a>
-<span class="sourceLineNo">335</span>  }<a name="line.335"></a>
-<span class="sourceLineNo">336</span><a name="line.336"></a>
-<span class="sourceLineNo">337</span>  /**<a name="line.337"></a>
-<span class="sourceLineNo">338</span>   * Removes the table descriptor from 
the local cache and returns it.<a name="line.338"></a>
-<span class="sourceLineNo">339</span>   * If not in read only mode, it also 
deletes the entire table directory(!)<a name="line.339"></a>
-<span class="sourceLineNo">340</span>   * from the FileSystem.<a 
name="line.340"></a>
-<span class="sourceLineNo">341</span>   */<a name="line.341"></a>
-<span class="sourceLineNo">342</span>  @Override<a name="line.342"></a>
-<span class="sourceLineNo">343</span>  public TableDescriptor remove(final 
TableName tablename)<a name="line.343"></a>
-<span class="sourceLineNo">344</span>  throws IOException {<a 
name="line.344"></a>
-<span class="sourceLineNo">345</span>    if (fsreadonly) {<a 
name="line.345"></a>
-<span class="sourceLineNo">346</span>      throw new 
NotImplementedException("Cannot remove a table descriptor - in read only 
mode");<a name="line.346"></a>
-<span class="sourceLineNo">347</span>    }<a name="line.347"></a>
-<span class="sourceLineNo">348</span>    Path tabledir = 
getTableDir(tablename);<a name="line.348"></a>
-<span class="sourceLineNo">349</span>    if (this.fs.exists(tabledir)) {<a 
name="line.349"></a>
-<span class="sourceLineNo">350</span>      if (!this.fs.delete(tabledir, 
true)) {<a name="line.350"></a>
-<span class="sourceLineNo">351</span>        throw new IOException("Failed 
delete of " + tabledir.toString());<a name="line.351"></a>
-<span class="sourceLineNo">352</span>      }<a name="line.352"></a>
-<span class="sourceLineNo">353</span>    }<a name="line.353"></a>
-<span class="sourceLineNo">354</span>    TableDescriptor descriptor = 
this.cache.remove(tablename);<a name="line.354"></a>
-<span class="sourceLineNo">355</span>    return descriptor;<a 
name="line.355"></a>
-<span class="sourceLineNo">356</span>  }<a name="line.356"></a>
-<span class="sourceLineNo">357</span><a name="line.357"></a>
-<span class="sourceLineNo">358</span>  /**<a name="line.358"></a>
-<span class="sourceLineNo">359</span>   * Checks if a current table info file 
exists for the given table<a name="line.359"></a>
-<span class="sourceLineNo">360</span>   *<a name="line.360"></a>
-<span class="sourceLineNo">361</span>   * @param tableName name of table<a 
name="line.361"></a>
-<span class="sourceLineNo">362</span>   * @return true if exists<a 
name="line.362"></a>
-<span class="sourceLineNo">363</span>   * @throws IOException<a 
name="line.363"></a>
-<span class="sourceLineNo">364</span>   */<a name="line.364"></a>
-<span class="sourceLineNo">365</span>  public boolean 
isTableInfoExists(TableName tableName) throws IOException {<a 
name="line.365"></a>
-<span class="sourceLineNo">366</span>    return getTableInfoPath(tableName) != 
null;<a name="line.366"></a>
-<span class="sourceLineNo">367</span>  }<a name="line.367"></a>
-<span class="sourceLineNo">368</span><a name="line.368"></a>
-<span class="sourceLineNo">369</span>  /**<a name="line.369"></a>
-<span class="sourceLineNo">370</span>   * Find the most current table info 
file for the given table in the hbase root directory.<a name="line.370"></a>
-<span class="sourceLineNo">371</span>   * @return The file status of the 
current table info file or null if it does not exist<a name="line.371"></a>
-<span class="sourceLineNo">372</span>   */<a name="line.372"></a>
-<span class="sourceLineNo">373</span>  private FileStatus 
getTableInfoPath(final TableName tableName) throws IOException {<a 
name="line.373"></a>
-<span class="sourceLineNo">374</span>    Path tableDir = 
getTableDir(tableName);<a name="line.374"></a>
-<span class="sourceLineNo">375</span>    return getTableInfoPath(tableDir);<a 
name="line.375"></a>
-<span class="sourceLineNo">376</span>  }<a name="line.376"></a>
-<span class="sourceLineNo">377</span><a name="line.377"></a>
-<span class="sourceLineNo">378</span>  private FileStatus 
getTableInfoPath(Path tableDir)<a name="line.378"></a>
-<span class="sourceLineNo">379</span>  throws IOException {<a 
name="line.379"></a>
-<span class="sourceLineNo">380</span>    return getTableInfoPath(fs, tableDir, 
!fsreadonly);<a name="line.380"></a>
-<span class="sourceLineNo">381</span>  }<a name="line.381"></a>
-<span class="sourceLineNo">382</span><a name="line.382"></a>
-<span class="sourceLineNo">383</span>  /**<a name="line.383"></a>
-<span class="sourceLineNo">384</span>   * Find the most current table info 
file for the table located in the given table directory.<a name="line.384"></a>
-<span class="sourceLineNo">385</span>   *<a name="line.385"></a>
-<span class="sourceLineNo">386</span>   * Looks within the {@link 
#TABLEINFO_DIR} subdirectory of the given directory for any table info<a 
name="line.386"></a>
-<span class="sourceLineNo">387</span>   * files and takes the 'current' one - 
meaning the one with the highest sequence number if present<a 
name="line.387"></a>
-<span class="sourceLineNo">388</span>   * or no sequence number at all if none 
exist (for backward compatibility from before there<a name="line.388"></a>
-<span class="sourceLineNo">389</span>   * were sequence numbers).<a 
name="line.389"></a>
-<span class="sourceLineNo">390</span>   *<a name="line.390"></a>
-<span class="sourceLineNo">391</span>   * @return The file status of the 
current table info file or null if it does not exist<a name="line.391"></a>
-<span class="sourceLineNo">392</span>   * @throws IOException<a 
name="line.392"></a>
-<span class="sourceLineNo">393</span>   */<a name="line.393"></a>
-<span class="sourceLineNo">394</span>  public static FileStatus 
getTableInfoPath(FileSystem fs, Path tableDir)<a name="line.394"></a>
-<span class="sourceLineNo">395</span>  throws IOException {<a 
name="line.395"></a>
-<span class="sourceLineNo">396</span>    return getTableInfoPath(fs, tableDir, 
false);<a name="line.396"></a>
-<span class="sourceLineNo">397</span>  }<a name="line.397"></a>
-<span class="sourceLineNo">398</span><a name="line.398"></a>
-<span class="sourceLineNo">399</span>  /**<a name="line.399"></a>
-<span class="sourceLineNo">400</span>   * Find the most current table info 
file for the table in the given table directory.<a name="line.400"></a>
-<span class="sourceLineNo">401</span>   *<a name="line.401"></a>
-<span class="sourceLineNo">402</span>   * Looks within the {@link 
#TABLEINFO_DIR} subdirectory of the given directory for any table info<a 
name="line.402"></a>
-<span class="sourceLineNo">403</span>   * files and takes the 'current' one - 
meaning the one with the highest sequence number if<a name="line.403"></a>
-<span class="sourceLineNo">404</span>   * present or no sequence number at all 
if none exist (for backward compatibility from before<a name="line.404"></a>
-<span class="sourceLineNo">405</span>   * there were sequence numbers).<a 
name="line.405"></a>
-<span class="sourceLineNo">406</span>   * If there are multiple table info 
files found and removeOldFiles is true it also deletes the<a 
name="line.406"></a>
-<span class="sourceLineNo">407</span>   * older files.<a name="line.407"></a>
-<span class="sourceLineNo">408</span>   *<a name="line.408"></a>
-<span class="sourceLineNo">409</span>   * @return The file status of the 
current table info file or null if none exist<a name="line.409"></a>
-<span class="sourceLineNo">410</span>   * @throws IOException<a 
name="line.410"></a>
-<span class="sourceLineNo">411</span>   */<a name="line.411"></a>
-<span class="sourceLineNo">412</span>  private static FileStatus 
getTableInfoPath(FileSystem fs, Path tableDir, boolean removeOldFiles)<a 
name="line.412"></a>
-<span class="sourceLineNo">413</span>  throws IOException {<a 
name="line.413"></a>
-<span class="sourceLineNo">414</span>    Path tableInfoDir = new 
Path(tableDir, TABLEINFO_DIR);<a name="line.414"></a>
-<span class="sourceLineNo">415</span>    return getCurrentTableInfoStatus(fs, 
tableInfoDir, removeOldFiles);<a name="line.415"></a>
-<span class="sourceLineNo">416</span>  }<a name="line.416"></a>
-<span class="sourceLineNo">417</span><a name="line.417"></a>
-<span class="sourceLineNo">418</span>  /**<a name="line.418"></a>
-<span class="sourceLineNo">419</span>   * Find the most current table info 
file in the given directory<a name="line.419"></a>
-<span class="sourceLineNo">420</span>   *<a name="line.420"></a>
-<span class="sourceLineNo">421</span>   * Looks within the given directory for 
any table info files<a name="line.421"></a>
-<span class="sourceLineNo">422</span>   * and takes the 'current' one - 
meaning the one with the highest sequence number if present<a 
name="line.422"></a>
-<span class="sourceLineNo">423</span>   * or no sequence number at all if none 
exist (for backward compatibility from before there<a name="line.423"></a>
-<span class="sourceLineNo">424</span>   * were sequence numbers).<a 
name="line.424"></a>
-<span class="sourceLineNo">425</span>   * If there are multiple possible files 
found<a name="line.425"></a>
-<span class="sourceLineNo">426</span>   * and the we're not in read only mode 
it also deletes the older files.<a name="line.426"></a>
-<span class="sourceLineNo">427</span>   *<a name="line.427"></a>
-<span class="sourceLineNo">428</span>   * @return The file status of the 
current table info file or null if it does not exist<a name="line.428"></a>
-<span class="sourceLineNo">429</span>   * @throws IOException<a 
name="line.429"></a>
-<span class="sourceLineNo">430</span>   */<a name="line.430"></a>
-<span class="sourceLineNo">431</span>  // only visible for 
FSTableDescriptorMigrationToSubdir, can be removed with that<a 
name="line.431"></a>
-<span class="sourceLineNo">432</span>  static FileStatus 
getCurrentTableInfoStatus(FileSystem fs, Path dir, boolean removeOldFiles)<a 
name="line.432"></a>
-<span class="sourceLineNo">433</span>  throws IOException {<a 
name="line.433"></a>
-<span class="sourceLineNo">434</span>    FileStatus [] status = 
FSUtils.listStatus(fs, dir, TABLEINFO_PATHFILTER);<a name="line.434"></a>
-<span class="sourceLineNo">435</span>    if (status == null || status.length 
&lt; 1) return null;<a name="line.435"></a>
-<span class="sourceLineNo">436</span>    FileStatus mostCurrent = null;<a 
name="line.436"></a>
-<span class="sourceLineNo">437</span>    for (FileStatus file : status) {<a 
name="line.437"></a>
-<span class="sourceLineNo">438</span>      if (mostCurrent == null || 
TABLEINFO_FILESTATUS_COMPARATOR.compare(file, mostCurrent) &lt; 0) {<a 
name="line.438"></a>
-<span class="sourceLineNo">439</span>        mostCurrent = file;<a 
name="line.439"></a>
-<span class="sourceLineNo">440</span>      }<a name="line.440"></a>
-<span class="sourceLineNo">441</span>    }<a name="line.441"></a>
-<span class="sourceLineNo">442</span>    if (removeOldFiles &amp;&amp; 
status.length &gt; 1) {<a name="line.442"></a>
-<span class="sourceLineNo">443</span>      // Clean away old versions<a 
name="line.443"></a>
-<span class="sourceLineNo">444</span>      for (FileStatus file : status) {<a 
name="line.444"></a>
-<span class="sourceLineNo">445</span>        Path path = file.getPath();<a 
name="line.445"></a>
-<span class="sourceLineNo">446</span>        if (!file.equals(mostCurrent)) 
{<a name="line.446"></a>
-<span class="sourceLineNo">447</span>          if (!fs.delete(file.getPath(), 
false)) {<a name="line.447"></a>
-<span class="sourceLineNo">448</span>            LOG.warn("Failed cleanup of " 
+ path);<a name="line.448"></a>
-<span class="sourceLineNo">449</span>          } else {<a name="line.449"></a>
-<span class="sourceLineNo">450</span>            LOG.debug("Cleaned up old 
tableinfo file " + path);<a name="line.450"></a>
-<span class="sourceLineNo">451</span>          }<a name="line.451"></a>
-<span class="sourceLineNo">452</span>        }<a name="line.452"></a>
-<span class="sourceLineNo">453</span>      }<a name="line.453"></a>
-<span class="sourceLineNo">454</span>    }<a name="line.454"></a>
-<span class="sourceLineNo">455</span>    return mostCurrent;<a 
name="line.455"></a>
-<span class="sourceLineNo">456</span>  }<a name="line.456"></a>
-<span class="sourceLineNo">457</span><a name="line.457"></a>
-<span class="sourceLineNo">458</span>  /**<a name="line.458"></a>
-<span class="sourceLineNo">459</span>   * Compare {@link FileStatus} instances 
by {@link Path#getName()}. Returns in<a name="line.459"></a>
-<span class="sourceLineNo">460</span>   * reverse order.<a name="line.460"></a>
-<span class="sourceLineNo">461</span>   */<a name="line.461"></a>
-<span class="sourceLineNo">462</span>  @VisibleForTesting<a 
name="line.462"></a>
-<span class="sourceLineNo">463</span>  static final 
Comparator&lt;FileStatus&gt; TABLEINFO_FILESTATUS_COMPARATOR =<a 
name="line.463"></a>
-<span class="sourceLineNo">464</span>  new Comparator&lt;FileStatus&gt;() {<a 
name="line.464"></a>
-<span class="sourceLineNo">465</span>    @Override<a name="line.465"></a>
-<span class="sourceLineNo">466</span>    public int compare(FileStatus left, 
FileStatus right) {<a name="line.466"></a>
-<span class="sourceLineNo">467</span>      return right.compareTo(left);<a 
name="line.467"></a>
-<span class="sourceLineNo">468</span>    }};<a name="line.468"></a>
-<span class="sourceLineNo">469</span><a name="line.469"></a>
-<span class="sourceLineNo">470</span>  /**<a name="line.470"></a>
-<span class="sourceLineNo">471</span>   * Return the table directory in HDFS<a 
name="line.471"></a>
-<span class="sourceLineNo">472</span>   */<a name="line.472"></a>
-<span class="sourceLineNo">473</span>  @VisibleForTesting Path 
getTableDir(final TableName tableName) {<a name="line.473"></a>
-<span class="sourceLineNo">474</span>    return FSUtils.getTableDir(rootdir, 
tableName);<a name="line.474"></a>
-<span class="sourceLineNo">475</span>  }<a name="line.475"></a>
-<span class="sourceLineNo">476</span><a name="line.476"></a>
-<span class="sourceLineNo">477</span>  private static final PathFilter 
TABLEINFO_PATHFILTER = new PathFilter() {<a name="line.477"></a>
-<span class="sourceLineNo">478</span>    @Override<a name="line.478"></a>
-<span class="sourceLineNo">479</span>    public boolean accept(Path p) {<a 
name="line.479"></a>
-<span class="sourceLineNo">480</span>      // Accept any file that starts with 
TABLEINFO_NAME<a name="line.480"></a>
-<span class="sourceLineNo">481</span>      return 
p.getName().startsWith(TABLEINFO_FILE_PREFIX);<a name="line.481"></a>
-<span class="sourceLineNo">482</span>    }};<a name="line.482"></a>
-<span class="sourceLineNo">483</span><a name="line.483"></a>
-<span class="sourceLineNo">484</span>  /**<a name="line.484"></a>
-<span class="sourceLineNo">485</span>   * Width of the sequenceid that is a 
suffix on a tableinfo file.<a name="line.485"></a>
-<span class="sourceLineNo">486</span>   */<a name="line.486"></a>
-<span class="sourceLineNo">487</span>  @VisibleForTesting static final int 
WIDTH_OF_SEQUENCE_ID = 10;<a name="line.487"></a>
-<span class="sourceLineNo">488</span><a name="line.488"></a>
-<span class="sourceLineNo">489</span>  /*<a name="line.489"></a>
-<span class="sourceLineNo">490</span>   * @param number Number to use as 
suffix.<a name="line.490"></a>
-<span class="sourceLineNo">491</span>   * @return Returns zero-prefixed 
decimal version of passed<a name="line.491"></a>
-<span class="sourceLineNo">492</span>   * number (Does absolute in case number 
is negative).<a name="line.492"></a>
-<span class="sourceLineNo">493</span>   */<a name="line.493"></a>
-<span class="sourceLineNo">494</span>  private static String 
formatTableInfoSequenceId(final int number) {<a name="line.494"></a>
-<span class="sourceLineNo">495</span>    byte [] b = new 
byte[WIDTH_OF_SEQUENCE_ID];<a name="line.495"></a>
-<span class="sourceLineNo">496</span>    int d = Math.abs(number);<a 
name="line.496"></a>
-<span class="sourceLineNo">497</span>    for (int i = b.length - 1; i &gt;= 0; 
i--) {<a name="line.497"></a>
-<span class="sourceLineNo">498</span>      b[i] = (byte)((d % 10) + '0');<a 
name="line.498"></a>
-<span class="sourceLineNo">499</span>      d /= 10;<a name="line.499"></a>
-<span class="sourceLineNo">500</span>    }<a name="line.500"></a>
-<span class="sourceLineNo">501</span>    return Bytes.toString(b);<a 
name="line.501"></a>
-<span class="sourceLineNo">502</span>  }<a name="line.502"></a>
-<span class="sourceLineNo">503</span><a name="line.503"></a>
-<span class="sourceLineNo">504</span>  /**<a name="line.504"></a>
-<span class="sourceLineNo">505</span>   * Regex to eat up sequenceid suffix on 
a .tableinfo file.<a name="line.505"></a>
-<span class="sourceLineNo">506</span>   * Use regex because may encounter 
oldstyle .tableinfos where there is no<a name="line.506"></a>
-<span class="sourceLineNo">507</span>   * sequenceid on the end.<a 
name="line.507"></a>
-<span class="sourceLineNo">508</span>   */<a name="line.508"></a>
-<span class="sourceLineNo">509</span>  private static final Pattern 
TABLEINFO_FILE_REGEX =<a name="line.509"></a>
-<span class="sourceLineNo">510</span>    Pattern.compile(TABLEINFO_FILE_PREFIX 
+ "(\\.([0-9]{" + WIDTH_OF_SEQUENCE_ID + "}))?$");<a name="line.510"></a>
-<span class="sourceLineNo">511</span><a name="line.511"></a>
-<span class="sourceLineNo">512</span>  /**<a name="line.512"></a>
-<span class="sourceLineNo">513</span>   * @param p Path to a 
&lt;code&gt;.tableinfo&lt;/code&gt; file.<a name="line.513"></a>
-<span class="sourceLineNo">514</span>   * @return The current editid or 0 if 
none found.<a name="line.514"></a>
-<span class="sourceLineNo">515</span>   */<a name="line.515"></a>
-<span class="sourceLineNo">516</span>  @VisibleForTesting static int 
getTableInfoSequenceId(final Path p) {<a name="line.516"></a>
-<span class="sourceLineNo">517</span>    if (p == null) return 0;<a 
name="line.517"></a>
-<span class="sourceLineNo">518</span>    Matcher m = 
TABLEINFO_FILE_REGEX.matcher(p.getName());<a name="line.518"></a>
-<span class="sourceLineNo">519</span>    if (!m.matches()) throw new 
IllegalArgumentException(p.toString());<a name="line.519"></a>
-<span class="sourceLineNo">520</span>    String suffix = m.group(2);<a 
name="line.520"></a>
-<span class="sourceLineNo">521</span>    if (suffix == null || suffix.length() 
&lt;= 0) return 0;<a name="line.521"></a>
-<span class="sourceLineNo">522</span>    return 
Integer.parseInt(m.group(2));<a name="line.522"></a>
-<span class="sourceLineNo">523</span>  }<a name="line.523"></a>
-<span class="sourceLineNo">524</span><a name="line.524"></a>
-<span class="sourceLineNo">525</span>  /**<a name="line.525"></a>
-<span class="sourceLineNo">526</span>   * @param sequenceid<a 
name="line.526"></a>
-<span class="sourceLineNo">527</span>   * @return Name of tableinfo file.<a 
name="line.527"></a>
-<span class="sourceLineNo">528</span>   */<a name="line.528"></a>
-<span class="sourceLineNo">529</span>  @VisibleForTesting static String 
getTableInfoFileName(final int sequenceid) {<a name="line.529"></a>
-<span class="sourceLineNo">530</span>    return TABLEINFO_FILE_PREFIX + "." + 
formatTableInfoSequenceId(sequenceid);<a name="line.530"></a>
-<span class="sourceLineNo">531</span>  }<a name="line.531"></a>
-<span class="sourceLineNo">532</span><a name="line.532"></a>
-<span class="sourceLineNo">533</span>  /**<a name="line.533"></a>
-<span class="sourceLineNo">534</span>   * Returns the latest table descriptor 
for the given table directly from the file system<a name="line.534"></a>
-<span class="sourceLineNo">535</span>   * if it exists, bypassing the local 
cache.<a name="line.535"></a>
-<span class="sourceLineNo">536</span>   * Returns null if it's not found.<a 
name="line.536"></a>
-<span class="sourceLineNo">537</span>   */<a name="line.537"></a>
-<span class="sourceLineNo">538</span>  public static TableDescriptor 
getTableDescriptorFromFs(FileSystem fs,<a name="line.538"></a>
-<span class="sourceLineNo">539</span>      Path hbaseRootDir, TableName 
tableName) throws IOException {<a name="line.539"></a>
-<span class="sourceLineNo">540</span>    Path tableDir = 
FSUtils.getTableDir(hbaseRootDir, tableName);<a name="line.540"></a>
-<span class="sourceLineNo">541</span>    return getTableDescriptorFromFs(fs, 
tableDir);<a name="line.541"></a>
-<span class="sourceLineNo">542</span>  }<a name="line.542"></a>
-<span class="sourceLineNo">543</span><a name="line.543"></a>
-<span class="sourceLineNo">544</span>  /**<a name="line.544"></a>
-<span class="sourceLineNo">545</span>   * Returns the latest table descriptor 
for the table located at the given directory<a name="line.545"></a>
-<span class="sourceLineNo">546</span>   * directly from the file system if it 
exists.<a name="line.546"></a>
-<span class="sourceLineNo">547</span>   * @throws TableInfoMissingException if 
there is no descriptor<a name="line.547"></a>
-<span class="sourceLineNo">548</span>   */<a name="line.548"></a>
-<span class="sourceLineNo">549</span>  public static TableDescriptor 
getTableDescriptorFromFs(FileSystem fs, Path tableDir)<a name="line.549"></a>
-<span class="sourceLineNo">550</span>  throws IOException {<a 
name="line.550"></a>
-<span class="sourceLineNo">551</span>    FileStatus status = 
getTableInfoPath(fs, tableDir, false);<a name="line.551"></a>
-<span class="sourceLineNo">552</span>    if (status == null) {<a 
name="line.552"></a>
-<span class="sourceLineNo">553</span>      throw new 
TableInfoMissingException("No table descriptor file under " + tableDir);<a 
name="line.553"></a>
-<span class="sourceLineNo">554</span>    }<a name="line.554"></a>
-<span class="sourceLineNo">555</span>    return readTableDescriptor(fs, 
status);<a name="line.555"></a>
-<span class="sourceLineNo">556</span>  }<a name="line.556"></a>
-<span class="sourceLineNo">557</span><a name="line.557"></a>
-<span class="sourceLineNo">558</span>  private static TableDescriptor 
readTableDescriptor(FileSystem fs, FileStatus status)<a name="line.558"></a>
-<span class="sourceLineNo">559</span>      throws IOException {<a 
name="line.559"></a>
-<span class="sourceLineNo">560</span>    int len = 
Ints.checkedCast(status.getLen());<a name="line.560"></a>
-<span class="sourceLineNo">561</span>    byte [] content = new byte[len];<a 
name="line.561"></a>
-<span class="sourceLineNo">562</span>    FSDataInputStream fsDataInputStream = 
fs.open(status.getPath());<a name="line.562"></a>
-<span class="sourceLineNo">563</span>    try {<a name="line.563"></a>
-<span class="sourceLineNo">564</span>      
fsDataInputStream.readFully(content);<a name="line.564"></a>
-<span class="sourceLineNo">565</span>    } finally {<a name="line.565"></a>
-<span class="sourceLineNo">566</span>      fsDataInputStream.close();<a 
name="line.566"></a>
-<span class="sourceLineNo">567</span>    }<a name="line.567"></a>
-<span class="sourceLineNo">568</span>    TableDescriptor htd = null;<a 
name="line.568"></a>
-<span class="sourceLineNo">569</span>    try {<a name="line.569"></a>
-<span class="sourceLineNo">570</span>      htd = 
TableDescriptorBuilder.parseFrom(content);<a name="line.570"></a>
-<span class="sourceLineNo">571</span>    } catch (DeserializationException e) 
{<a name="line.571"></a>
-<span class="sourceLineNo">572</span>      throw new IOException("content=" + 
Bytes.toShort(content), e);<a name="line.572"></a>
-<span class="sourceLineNo">573</span>    }<a name="line.573"></a>
-<span class="sourceLineNo">574</span>    return htd;<a name="line.574"></a>
-<span class="sourceLineNo">575</span>  }<a name="line.575"></a>
-<span class="sourceLineNo">576</span><a name="line.576"></a>
-<span class="sourceLineNo">577</span>  /**<a name="line.577"></a>
-<span class="sourceLineNo">578</span>   * Update table descriptor on the file 
system<a name="line.578"></a>
-<span class="sourceLineNo">579</span>   * @throws IOException Thrown if failed 
update.<a name="line.579"></a>
-<span class="sourceLineNo">580</span>   * @throws NotImplementedException if 
in read only mode<a name="line.580"></a>
-<span class="sourceLineNo">581</span>   */<a name="line.581"></a>
-<span class="sourceLineNo">582</span>  @VisibleForTesting Path 
updateTableDescriptor(TableDescriptor td)<a name="line.582"></a>
-<span class="sourceLineNo">583</span>  throws IOException {<a 
name="line.583"></a>
-<span class="sourceLineNo">584</span>    if (fsreadonly) {<a 
name="line.584"></a>
-<span class="sourceLineNo">585</span>      throw new 
NotImplementedException("Cannot update a table descriptor - in read only 
mode");<a name="line.585"></a>
-<span class="sourceLineNo">586</span>    }<a name="line.586"></a>
-<span class="sourceLineNo">587</span>    TableName tableName = 
td.getTableName();<a name="line.587"></a>
-<span class="sourceLineNo">588</span>    Path tableDir = 
getTableDir(tableName);<a name="line.588"></a>
-<span class="sourceLineNo">589</span>    Path p = writeTableDescriptor(fs, td, 
tableDir, getTableInfoPath(tableDir));<a name="line.589"></a>
-<span class="sourceLineNo">590</span>    if (p == null) throw new 
IOException("Failed update");<a name="line.590"></a>
-<span class="sourceLineNo">591</span>    LOG.info("Updated tableinfo=" + p);<a 
name="line.591"></a>
-<span class="sourceLineNo">592</span>    if (usecache) {<a name="line.592"></a>
-<span class="sourceLineNo">593</span>      this.cache.put(td.getTableName(), 
td);<a name="line.593"></a>
-<span class="sourceLineNo">594</span>    }<a name="line.594"></a>
-<span class="sourceLineNo">595</span>    return p;<a name="line.595"></a>
-<span class="sourceLineNo">596</span>  }<a name="line.596"></a>
-<span class="sourceLineNo">597</span><a name="line.597"></a>
-<span class="sourceLineNo">598</span>  /**<a name="line.598"></a>
-<span class="sourceLineNo">599</span>   * Deletes all the table descriptor 
files from the file system.<a name="line.599"></a>
-<span class="sourceLineNo">600</span>   * Used in unit tests only.<a 
name="line.600"></a>
-<span class="sourceLineNo">601</span>   * @throws NotImplementedException if 
in read only mode<a name="line.601"></a>
-<span class="sourceLineNo">602</span>   */<a name="line.602"></a>
-<span class="sourceLineNo">603</span>  public void 
deleteTableDescriptorIfExists(TableName tableName) throws IOException {<a 
name="line.603"></a>
-<span class="sourceLineNo">604</span>    if (fsreadonly) {<a 
name="line.604"></a>
-<span class="sourceLineNo">605</span>      throw new 
NotImplementedException("Cannot delete a table descriptor - in read only 
mode");<a name="line.605"></a>
-<span class="sourceLineNo">606</span>    }<a name="line.606"></a>
-<span class="sourceLineNo">607</span><a name="line.607"></a>
-<span class="sourceLineNo">608</span>    Path tableDir = 
getTableDir(tableName);<a name="line.608"></a>
-<span class="sourceLineNo">609</span>    Path tableInfoDir = new 
Path(tableDir, TABLEINFO_DIR);<a name="line.609"></a>
-<span class="sourceLineNo">610</span>    deleteTableDescriptorFiles(fs, 
tableInfoDir, Integer.MAX_VALUE);<a name="line.610"></a>
-<span class="sourceLineNo">611</span>  }<a name="line.611"></a>
-<span class="sourceLineNo">612</span><a name="line.612"></a>
-<span class="sourceLineNo">613</span>  /**<a name="line.613"></a>
-<span class="sourceLineNo">614</span>   * Deletes files matching the table 
info file pattern within the given directory<a name="line.614"></a>
-<span class="sourceLineNo">615</span>   * whose sequenceId is at most the 
given max sequenceId.<a name="line.615"></a>
-<span class="sourceLineNo">616</span>   */<a name="line.616"></a>
-<span class="sourceLineNo">617</span>  private static void 
deleteTableDescriptorFiles(FileSystem fs, Path dir, int maxSequenceId)<a 
name="line.617"></a>
-<span class="sourceLineNo">618</span>  throws IOException {<a 
name="line.618"></a>
-<span class="sourceLineNo">619</span>    FileStatus [] status = 
FSUtils.listStatus(fs, dir, TABLEINFO_PATHFILTER);<a name="line.619"></a>
-<span class="sourceLineNo">620</span>    for (FileStatus file : status) {<a 
name="line.620"></a>
-<span class="sourceLineNo">621</span>      Path path = file.getPath();<a 
name="line.621"></a>
-<span class="sourceLineNo">622</span>      int sequenceId = 
getTableInfoSequenceId(path);<a name="line.622"></a>
-<span class="sourceLineNo">623</span>      if (sequenceId &lt;= maxSequenceId) 
{<a name="line.623"></a>
-<span class="sourceLineNo">624</span>        boolean success = 
FSUtils.delete(fs, path, false);<a name="line.624"></a>
-<span class="sourceLineNo">625</span>        if (success) {<a 
name="line.625"></a>
-<span class="sourceLineNo">626</span>          LOG.debug("Deleted " + path);<a 
name="line.626"></a>
-<span class="sourceLineNo">627</span>        } else {<a name="line.627"></a>
-<span class="sourceLineNo">628</span>          LOG.error("Failed to delete 
table descriptor at " + path);<a name="line.628"></a>
-<span class="sourceLineNo">629</span>        }<a name="line.629"></a>
-<span class="sourceLineNo">630</span>      }<a name="line.630"></a>
-<span class="sourceLineNo">631</span>    }<a name="line.631"></a>
-<span class="sourceLineNo">632</span>  }<a name="line.632"></a>
-<span class="sourceLineNo">633</span><a name="line.633"></a>
-<span class="sourceLineNo">634</span>  /**<a name="line.634"></a>
-<span class="sourceLineNo">635</span>   * Attempts to write a new table 
descriptor to the given table's directory.<a name="line.635"></a>
-<span class="sourceLineNo">636</span>   * It first writes it to the .tmp dir 
then uses an atomic rename to move it into place.<a name="line.636"></a>
-<span class="sourceLineNo">637</span>   * It begins at the currentSequenceId + 
1 and tries 10 times to find a new sequence number<a name="line.637"></a>
-<span class="sourceLineNo">638</span>   * not already in use.<a 
name="line.638"></a>
-<span class="sourceLineNo">639</span>   * Removes the current descriptor file 
if passed in.<a name="line.639"></a>
-<span class="sourceLineNo">640</span>   *<a name="line.640"></a>
-<span class="sourceLineNo">641</span>   * @return Descriptor file or null if 
we failed write.<a name="line.641"></a>
-<span class="sourceLineNo">642</span>   */<a name="line.642"></a>
-<span class="sourceLineNo">643</span>  private static Path 
writeTableDescriptor(final FileSystem fs,<a name="line.643"></a>
-<span class="sourceLineNo">644</span>    final TableDescriptor htd, final Path 
tableDir,<a name="line.644"></a>
-<span class="sourceLineNo">645</span>    final FileStatus 
currentDescriptorFile)<a name="line.645"></a>
-<span class="sourceLineNo">646</span>  throws IOException {<a 
name="line.646"></a>
-<span class="sourceLineNo">647</span>    // Get temporary dir into which we'll 
first write a file to avoid half-written file phenomenon.<a name="line.647"></a>
-<span class="sourceLineNo">648</span>    // This directory is never removed to 
avoid removing it out from under a concurrent writer.<a name="line.648"></a>
-<span class="sourceLineNo">649</span>    Path tmpTableDir = new Path(tableDir, 
TMP_DIR);<a name="line.649"></a>
-<span class="sourceLineNo">650</span>    Path tableInfoDir = new 
Path(tableDir, TABLEINFO_DIR);<a name="line.650"></a>
-<span class="sourceLineNo">651</span><a name="line.651"></a>
-<span class="sourceLineNo">652</span>    // What is current sequenceid?  We 
read the current sequenceid from<a name="line.652"></a>
-<span class="sourceLineNo">653</span>    // the current file.  After we read 
it, another thread could come in and<a name="line.653"></a>
-<span class="sourceLineNo">654</span>    // compete with us writing out next 
version of file.  The below retries<a name="line.654"></a>
-<span class="sourceLineNo">655</span>    // should help in this case some but 
its hard to do guarantees in face of<a name="line.655"></a>
-<span class="sourceLineNo">656</span>    // concurrent schema edits.<a 
name="line.656"></a>
-<span class="sourceLineNo">657</span>    int currentSequenceId = 
currentDescriptorFile == null ? 0 :<a name="line.657"></a>
-<span class="sourceLineNo">658</span>      
getTableInfoSequenceId(currentDescriptorFile.getPath());<a name="line.658"></a>
-<span class="sourceLineNo">659</span>    int newSequenceId = 
currentSequenceId;<a name="line.659"></a>
-<span class="sourceLineNo">660</span><a name="line.660"></a>
-<span class="sourceLineNo">661</span>    // Put arbitrary upperbound on how 
often we retry<a name="line.661"></a>
-<span class="sourceLineNo">662</span>    int retries = 10;<a 
name="line.662"></a>
-<span class="sourceLineNo">663</span>    int retrymax = currentSequenceId + 
retries;<a name="line.663"></a>
-<span class="sourceLineNo">664</span>    Path tableInfoDirPath = null;<a 
name="line.664"></a>
-<span class="sourceLineNo">665</span>    do {<a name="line.665"></a>
-<span class="sourceLineNo">666</span>      newSequenceId += 1;<a 
name="line.666"></a>
-<span class="sourceLineNo">667</span>      String filename = 
getTableInfoFileName(newSequenceId);<a name="line.667"></a>
-<span class="sourceLineNo">668</span>      Path tempPath = new 
Path(tmpTableDir, filename);<a name="line.668"></a>
-<span class="sourceLineNo">669</span>      if (fs.exists(tempPath)) {<a 
name="line.669"></a>
-<span class="sourceLineNo">670</span>        LOG.debug(tempPath + " exists; 
retrying up to " + retries + " times");<a name="line.670"></a>
-<span class="sourceLineNo">671</span>        continue;<a name="line.671"></a>
-<span class="sourceLineNo">672</span>      }<a name="line.672"></a>
-<span class="sourceLineNo">673</span>      tableInfoDirPath = new 
Path(tableInfoDir, filename);<a name="line.673"></a>
-<span class="sourceLineNo">674</span>      try {<a name="line.674"></a>
-<span class="sourceLineNo">675</span>        writeTD(fs, tempPath, htd);<a 
name="line.675"></a>
-<span class="sourceLineNo">676</span>        
fs.mkdirs(tableInfoDirPath.getParent());<a name="line.676"></a>
-<span class="sourceLineNo">677</span>        if (!fs.rename(tempPath, 
tableInfoDirPath)) {<a name="line.677"></a>
-<span class="sourceLineNo">678</span>          throw new IOException("Failed 
rename of " + tempPath + " to " + tableInfoDirPath);<a name="line.678"></a>
-<span class="sourceLineNo">679</span>        }<a name="line.679"></a>
-<span class="sourceLineNo">680</span>        LOG.debug("Wrote into " + 
tableInfoDirPath);<a name="line.680"></a>
-<span class="sourceLineNo">681</span>      } catch (IOException ioe) {<a 
name="line.681"></a>
-<span class="sourceLineNo">682</span>        // Presume clash of names or 
something; go around again.<a name="line.682"></a>
-<span class="sourceLineNo">683</span>        LOG.debug("Failed write and/or 
rename; retrying", ioe);<a name="line.683"></a>
-<span class="sourceLineNo">684</span>        if (!FSUtils.deleteDirectory(fs, 
tempPath)) {<a name="line.684"></a>
-<span class="sourceLineNo">685</span>          LOG.warn("Failed cleanup of " + 
tempPath);<a name="line.685"></a>
-<span class="sourceLineNo">686</span>        }<a name="line.686"></a>
-<span class="sourceLineNo">687</span>        tableInfoDirPath = null;<a 
name="line.687"></a>
-<span class="sourceLineNo">688</span>        continue;<a name="line.688"></a>
-<span class="sourceLineNo">689</span>      }<a name="line.689"></a>
-<span class="sourceLineNo">690</span>      break;<a name="line.690"></a>
-<span class="sourceLineNo">691</span>    } while (newSequenceId &lt; 
retrymax);<a name="line.691"></a>
-<span class="sourceLineNo">692</span>    if (tableInfoDirPath != null) {<a 
name="line.692"></a>
-<span class="sourceLineNo">693</span>      // if we succeeded, remove old 
table info files.<a name="line.693"></a>
-<span class="sourceLineNo">694</span>      deleteTableDescriptorFiles(fs, 
tableInfoDir, newSequenceId - 1);<a name="line.694"></a>
-<span class="sourceLineNo">695</span>    }<a name="line.695"></a>
-<span class="sourceLineNo">696</span>    return tableInfoDirPath;<a 
name="line.696"></a>
-<span class="sourceLineNo">697</span>  }<a name="line.697"></a>
-<span class="sourceLineNo">698</span><a name="line.698"></a>
-<span class="sourceLineNo">699</sp

<TRUNCATED>

Reply via email to