You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2018/03/14 22:06:17 UTC
[2/4] hbase git commit: HBASE-18864 (addendum) Fixed unit test
failure TestFromClientSide#testIllegalTableDescriptor
HBASE-18864 (addendum) Fixed unit test failure TestFromClientSide#testIllegalTableDescriptor
Signed-off-by: Andrew Purtell <ap...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/fa3d572d
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/fa3d572d
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/fa3d572d
Branch: refs/heads/branch-1.4
Commit: fa3d572d26673954dbf1ddd31b99ea577aaab753
Parents: 95d16a5
Author: Sakthi <ja...@cloudera.com>
Authored: Wed Mar 14 11:43:01 2018 -0700
Committer: Andrew Purtell <ap...@apache.org>
Committed: Wed Mar 14 13:57:20 2018 -0700
----------------------------------------------------------------------
.../apache/hadoop/hbase/client/TestFromClientSide.java | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/fa3d572d/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
index 6f73409..58ff20f 100644
--- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
+++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
@@ -5577,9 +5577,13 @@ public class TestFromClientSide {
checkTableIsIllegal(htd);
hcd.setMinVersions(3);
- hcd.setScope(-1);
- checkTableIsIllegal(htd);
- hcd.setScope(0);
+ try {
+ hcd.setScope(-1);
+ fail("Illegal value for setScope did not throw");
+ } catch (IllegalArgumentException e) {
+ // expected
+ hcd.setScope(0);
+ }
checkTableIsLegal(htd);
try {