accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bhava...@apache.org
Subject [04/11] git commit: ACCUMULO-2524 Check for null args to ProblemReport constructors
Date Thu, 27 Mar 2014 21:35:46 GMT
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 <bhavanki@cloudera.com>
Authored: Thu Mar 27 17:19:24 2014 -0400
Committer: Bill Havanki <bhavanki@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;


Mime
View raw message