You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by bh...@apache.org on 2014/03/27 22:35:46 UTC
[04/11] git commit: ACCUMULO-2524 Check for null args to
ProblemReport constructors
ACCUMULO-2524 Check for null args to ProblemReport constructors
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/1258b4e0
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/1258b4e0
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/1258b4e0
Branch: refs/heads/master
Commit: 1258b4e0352e99d70b34b1131a4cfa5f90afd6ff
Parents: 4619101
Author: Bill Havanki <bh...@cloudera.com>
Authored: Thu Mar 27 17:19:24 2014 -0400
Committer: Bill Havanki <bh...@cloudera.com>
Committed: Thu Mar 27 17:19:24 2014 -0400
----------------------------------------------------------------------
.../java/org/apache/accumulo/server/problems/ProblemReport.java | 3 +++
1 file changed, 3 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/accumulo/blob/1258b4e0/src/server/src/main/java/org/apache/accumulo/server/problems/ProblemReport.java
----------------------------------------------------------------------
diff --git a/src/server/src/main/java/org/apache/accumulo/server/problems/ProblemReport.java b/src/server/src/main/java/org/apache/accumulo/server/problems/ProblemReport.java
index 06959d0..bc8f8ac 100644
--- a/src/server/src/main/java/org/apache/accumulo/server/problems/ProblemReport.java
+++ b/src/server/src/main/java/org/apache/accumulo/server/problems/ProblemReport.java
@@ -29,6 +29,7 @@ import org.apache.accumulo.core.Constants;
import org.apache.accumulo.core.data.Key;
import org.apache.accumulo.core.data.Mutation;
import org.apache.accumulo.core.data.Value;
+import org.apache.accumulo.core.util.ArgumentChecker;
import org.apache.accumulo.core.util.Encoding;
import org.apache.accumulo.core.zookeeper.ZooUtil;
import org.apache.accumulo.core.zookeeper.ZooUtil.NodeExistsPolicy;
@@ -48,6 +49,7 @@ public class ProblemReport {
private long creationTime;
public ProblemReport(String table, ProblemType problemType, String resource, String server, Throwable e) {
+ ArgumentChecker.notNull(table, problemType, resource);
this.tableName = table;
this.problemType = problemType;
@@ -74,6 +76,7 @@ public class ProblemReport {
}
private ProblemReport(String table, ProblemType problemType, String resource, byte enc[]) throws IOException {
+ ArgumentChecker.notNull(table, problemType, resource);
this.tableName = table;
this.problemType = problemType;
this.resource = resource;