From 0ed16c73327bcf7c3019bae1dfb851ef4dbf41ce Mon Sep 17 00:00:00 2001 From: Yuhao Bi Date: Thu, 28 May 2015 20:32:28 +0800 Subject: [PATCH] HBASE-13776 - set illegal versions for HColumnDescriptor did not throw IllegalArgumentException Should check minVersions <= maxVersions in HMaster. --- .../src/main/java/org/apache/hadoop/hbase/master/HMaster.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java index a8f2bca..f5b86ae 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java @@ -1467,6 +1467,15 @@ public class HMaster extends HRegionServer implements MasterServices, Server { } // max versions already being checked + // HBASE-13776 - set illegal versions for HColumnDescriptor + // did not throw IllegalArgumentException + // check minVersions <= maxVerions + if (hcd.getMinVersions() > hcd.getMaxVersions()) { + String message = "Min versions for column family " + hcd.getNameAsString() + + " must be less than the Max versions."; + warnOrThrowExceptionForFailure(logWarn, CONF_KEY, message, null); + } + // check replication scope if (hcd.getScope() < 0) { String message = "Replication scope for column family " -- 1.9.1