diff --git a/hbase10/src/main/java/com/yahoo/ycsb/db/HBaseClient10.java b/hbase10/src/main/java/com/yahoo/ycsb/db/HBaseClient10.java index 12b13d999497016de79eb9f0f32dc46f84f36c99..d7088597f3245fc62881f9fd1096908ea8da3a58 100644 --- a/hbase10/src/main/java/com/yahoo/ycsb/db/HBaseClient10.java +++ b/hbase10/src/main/java/com/yahoo/ycsb/db/HBaseClient10.java @@ -15,9 +15,6 @@ package com.yahoo.ycsb.db; -import com.google.common.annotations.VisibleForTesting; -import com.google.common.base.Preconditions; - import com.yahoo.ycsb.ByteArrayByteIterator; import com.yahoo.ycsb.ByteIterator; import com.yahoo.ycsb.DBException; @@ -438,7 +435,7 @@ public class HBaseClient10 extends com.yahoo.ycsb.DB { try { if (clientSideBuffering) { - Preconditions.checkNotNull(bufferedMutator); + // removed Preconditions.checkNotNull, which throws NPE, in favor of NPE on next line bufferedMutator.mutate(p); } else { currentTable.put(p); @@ -506,7 +503,7 @@ public class HBaseClient10 extends com.yahoo.ycsb.DB { d.setDurability(durability); try { if (clientSideBuffering) { - Preconditions.checkNotNull(bufferedMutator); + // removed Preconditions.checkNotNull, which throws NPE, in favor of NPE on next line bufferedMutator.mutate(d); } else { currentTable.delete(d); @@ -521,7 +518,7 @@ public class HBaseClient10 extends com.yahoo.ycsb.DB { return Status.OK; } - @VisibleForTesting + // Only non-private for testing. void setConfiguration(final Configuration newConfig) { this.config = newConfig; }