Repository: accumulo
Updated Branches:
  refs/heads/1.4.5-SNAPSHOT 461910148 -> 1258b4e03
  refs/heads/1.5.2-SNAPSHOT 4a05a17de -> 334f7a724
  refs/heads/1.6.0-SNAPSHOT 3ce0bfda3 -> d3bd456e7
  refs/heads/master 3c935b129 -> 4cfde769c


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/1.4.5-SNAPSHOT
Commit: 1258b4e0352e99d70b34b1131a4cfa5f90afd6ff
Parents: 4619101
Author: Bill Havanki <bhava...@cloudera.com>
Authored: Thu Mar 27 17:19:24 2014 -0400
Committer: Bill Havanki <bhava...@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;

Reply via email to