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 2011/02/09 20:16:16 UTC
svn commit: r1069040 - in /hbase/branches/0.90: CHANGES.txt pom.xml
src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java
src/main/resources/hbase-default.xml
src/main/xslt/configuration_to_docbook_section.xsl
Author: todd
Date: Wed Feb 9 19:16:16 2011
New Revision: 1069040
URL: http://svn.apache.org/viewvc?rev=1069040&view=rev
Log:
HBASE-3517. Store build version in hbase-default.xml and verify at runtime
Modified:
hbase/branches/0.90/CHANGES.txt
hbase/branches/0.90/pom.xml
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java
hbase/branches/0.90/src/main/resources/hbase-default.xml
hbase/branches/0.90/src/main/xslt/configuration_to_docbook_section.xsl
Modified: hbase/branches/0.90/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1069040&r1=1069039&r2=1069040&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Wed Feb 9 19:16:16 2011
@@ -31,6 +31,7 @@ Release 0.90.1 - Unreleased
aborts. Instead, should just cancel region open.
HBASE-3515 [replication] ReplicationSource can miss a log after RS comes
out of GC
+ HBASE-3517 Store build version in hbase-default.xml and verify at runtime
IMPROVEMENTS
Modified: hbase/branches/0.90/pom.xml
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/pom.xml?rev=1069040&r1=1069039&r2=1069040&view=diff
==============================================================================
--- hbase/branches/0.90/pom.xml (original)
+++ hbase/branches/0.90/pom.xml Wed Feb 9 19:16:16 2011
@@ -386,6 +386,18 @@
<goal>run</goal>
</goals>
</execution>
+ <execution>
+ <phase>process-resources</phase>
+ <configuration>
+ <tasks>
+ <replace file="${project.build.outputDirectory}/hbase-default.xml"
+ token="@@@VERSION@@@" value="${project.version}" />
+ </tasks>
+ </configuration>
+ <goals>
+ <goal>run</goal>
+ </goals>
+ </execution>
</executions>
</plugin>
<plugin>
Modified: hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java?rev=1069040&r1=1069039&r2=1069040&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java (original)
+++ hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/HBaseConfiguration.java Wed Feb 9 19:16:16 2011
@@ -24,6 +24,7 @@ import java.util.Map.Entry;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
import org.apache.hadoop.conf.Configuration;
+import org.apache.hadoop.hbase.util.VersionInfo;
/**
* Adds HBase configuration files to a Configuration
@@ -58,9 +59,21 @@ public class HBaseConfiguration extends
}
}
+ private static void checkDefaultsVersion(Configuration conf) {
+ String defaultsVersion = conf.get("hbase.defaults.for.version");
+ String thisVersion = VersionInfo.getVersion();
+ if (!thisVersion.equals(defaultsVersion)) {
+ throw new RuntimeException(
+ "hbase-default.xml file seems to be for and old version of HBase (" +
+ defaultsVersion + "), this version is " + thisVersion);
+ }
+ }
+
public static Configuration addHbaseResources(Configuration conf) {
conf.addResource("hbase-default.xml");
conf.addResource("hbase-site.xml");
+
+ checkDefaultsVersion(conf);
return conf;
}
Modified: hbase/branches/0.90/src/main/resources/hbase-default.xml
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/resources/hbase-default.xml?rev=1069040&r1=1069039&r2=1069040&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/resources/hbase-default.xml (original)
+++ hbase/branches/0.90/src/main/resources/hbase-default.xml Wed Feb 9 19:16:16 2011
@@ -579,4 +579,14 @@
true: Only the GET method is permitted.
</description>
</property>
+
+ <property skipInDoc="true">
+ <name>hbase.defaults.for.version</name>
+ <value>@@@VERSION@@@</value>
+ <description>
+ This defaults file was compiled for version @@@VERSION@@@. This variable is used
+ to make sure that a user doesn't have an old version of hbase-default.xml on the
+ classpath.
+ </description>
+ </property>
</configuration>
Modified: hbase/branches/0.90/src/main/xslt/configuration_to_docbook_section.xsl
URL: http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/xslt/configuration_to_docbook_section.xsl?rev=1069040&r1=1069039&r2=1069040&view=diff
==============================================================================
--- hbase/branches/0.90/src/main/xslt/configuration_to_docbook_section.xsl (original)
+++ hbase/branches/0.90/src/main/xslt/configuration_to_docbook_section.xsl Wed Feb 9 19:16:16 2011
@@ -45,6 +45,7 @@ This documentation is generated using th
</para>
<xsl:for-each select="property">
+<xsl:if test="not(@skipInDoc)">
<glossentry>
<xsl:attribute name="id">
<xsl:value-of select="name" />
@@ -57,6 +58,7 @@ This documentation is generated using th
<para>Default: <varname><xsl:value-of select="value"/></varname></para>
</glossdef>
</glossentry>
+</xsl:if>
</xsl:for-each>
</glossary>