You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by to...@apache.org on 2010/06/22 22:56:18 UTC
svn commit: r957029 - in /hbase/trunk: CHANGES.txt
src/main/java/org/apache/hadoop/hbase/util/FSUtils.java
src/main/resources/hbase-webapps/master/master.jsp
Author: todd
Date: Tue Jun 22 20:56:18 2010
New Revision: 957029
URL: http://svn.apache.org/viewvc?rev=957029&view=rev
Log:
HBASE-2762. Add warning to master if running without append enabled
Modified:
hbase/trunk/CHANGES.txt
hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java
hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp
Modified: hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=957029&r1=957028&r2=957029&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Tue Jun 22 20:56:18 2010
@@ -722,6 +722,7 @@ Release 0.21.0 - Unreleased
HBASE-2683 Make it obvious in the documentation that ZooKeeper needs
permanent storage
HBASE-2764 Force all Chore tasks to have a thread name
+ HBASE-2762 Add warning to master if running without append enabled
NEW FEATURES
HBASE-1961 HBase EC2 scripts
Modified: hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java?rev=957029&r1=957028&r2=957029&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java (original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/util/FSUtils.java Tue Jun 22 20:56:18 2010
@@ -601,6 +601,7 @@ public class FSUtils {
public static void recoverFileLease(final FileSystem fs, final Path p, Configuration conf)
throws IOException{
if (!isAppendSupported(conf)) {
+ LOG.warn("Running on HDFS without append enabled may result in data loss");
return;
}
// lease recovery not needed for local file system case.
Modified: hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp
URL: http://svn.apache.org/viewvc/hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp?rev=957029&r1=957028&r2=957029&view=diff
==============================================================================
--- hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp (original)
+++ hbase/trunk/src/main/resources/hbase-webapps/master/master.jsp Tue Jun 22 20:56:18 2010
@@ -3,6 +3,7 @@
import="org.apache.hadoop.conf.Configuration"
import="org.apache.hadoop.hbase.util.Bytes"
import="org.apache.hadoop.hbase.util.JvmVersion"
+ import="org.apache.hadoop.hbase.util.FSUtils"
import="org.apache.hadoop.hbase.master.HMaster"
import="org.apache.hadoop.hbase.HConstants"
import="org.apache.hadoop.hbase.master.MetaRegion"
@@ -38,6 +39,7 @@
<h1 id="page_title">Master: <%=master.getMasterAddress().getHostname()%>:<%=master.getMasterAddress().getPort()%></h1>
<p id="links_menu"><a href="/logs/">Local logs</a>, <a href="/stacks">Thread Dump</a>, <a href="/logLevel">Log Level</a></p>
+<!-- Various warnings that cluster admins should be aware of -->
<% if (JvmVersion.isBadJvmVersion()) { %>
<div class="warning">
Your current JVM version <%= System.getProperty("java.version") %> is known to be
@@ -46,6 +48,14 @@
for details.
</div>
<% } %>
+<% if (!FSUtils.isAppendSupported(conf)) { %>
+ <div class="warning">
+ You are currently running the HMaster without HDFS append support enabled.
+ This may result in data loss.
+ Please see the <a href="http://wiki.apache.org/hadoop/Hbase/HdfsSyncSupport">HBase wiki</a>
+ for details.
+ </div>
+<% } %>
<hr id="head_rule" />