Return-Path: 
 <commits-return-2484-apmail-hama-commits-archive=hama.apache.org@hama.apache.org>
X-Original-To: apmail-hama-commits-archive@www.apache.org
Delivered-To: apmail-hama-commits-archive@www.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 5A9B4E57A
	for <apmail-hama-commits-archive@www.apache.org>;
 Tue,  5 Feb 2013 08:09:30 +0000 (UTC)
Received: (qmail 73071 invoked by uid 500); 5 Feb 2013 08:09:30 -0000
Delivered-To: apmail-hama-commits-archive@hama.apache.org
Received: (qmail 73008 invoked by uid 500); 5 Feb 2013 08:09:29 -0000
Mailing-List: contact commits-help@hama.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:commits-help@hama.apache.org>
List-Unsubscribe: <mailto:commits-unsubscribe@hama.apache.org>
List-Post: <mailto:commits@hama.apache.org>
List-Id: <commits.hama.apache.org>
Reply-To: dev@hama.apache.org
Delivered-To: mailing list commits@hama.apache.org
Received: (qmail 72980 invoked by uid 99); 5 Feb 2013 08:09:28 -0000
Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Feb 2013 08:09:28 +0000
X-ASF-Spam-Status: No, hits=-2000.0 required=5.0
	tests=ALL_TRUSTED
X-Spam-Check-By: apache.org
Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4)
    by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 05 Feb 2013 08:09:27 +0000
Received: from eris.apache.org (localhost [127.0.0.1])
	by eris.apache.org (Postfix) with ESMTP id 31D6123889ED;
	Tue,  5 Feb 2013 08:09:08 +0000 (UTC)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1442490 - in
 /hama/trunk/graph/src/main/java/org/apache/hama/graph: GraphJobRunner.java
 VertexInputReader.java VerticesInfo.java
Date: Tue, 05 Feb 2013 08:09:08 -0000
To: commits@hama.apache.org
From: tommaso@apache.org
X-Mailer: svnmailer-1.0.8-patched
Message-Id: <20130205080908.31D6123889ED@eris.apache.org>
X-Virus-Checked: Checked by ClamAV on apache.org

Author: tommaso
Date: Tue Feb  5 08:09:07 2013
New Revision: 1442490

URL: http://svn.apache.org/viewvc?rev=1442490&view=rev
Log:
HAMA-572 - fixed some declaration redundancies

Modified:
    hama/trunk/graph/src/main/java/org/apache/hama/graph/GraphJobRunner.java
    hama/trunk/graph/src/main/java/org/apache/hama/graph/VertexInputReader.java
    hama/trunk/graph/src/main/java/org/apache/hama/graph/VerticesInfo.java

Modified: hama/trunk/graph/src/main/java/org/apache/hama/graph/GraphJobRunner.java
URL: http://svn.apache.org/viewvc/hama/trunk/graph/src/main/java/org/apache/hama/graph/GraphJobRunner.java?rev=1442490&r1=1442489&r2=1442490&view=diff
==============================================================================
--- hama/trunk/graph/src/main/java/org/apache/hama/graph/GraphJobRunner.java (original)
+++ hama/trunk/graph/src/main/java/org/apache/hama/graph/GraphJobRunner.java Tue Feb  5 08:09:07 2013
@@ -80,9 +80,6 @@ public final class GraphJobRunner<V exte
   private int maxIteration = -1;
   private long iteration;
 
-  private Class<V> vertexIdClass;
-  private Class<M> vertexValueClass;
-  private Class<E> edgeValueClass;
   private Class<Vertex<V, E, M>> vertexClass;
 
   private AggregationRunner<V, E, M> aggregationRunner;
@@ -232,13 +229,13 @@ public final class GraphJobRunner<V exte
     maxIteration = peer.getConfiguration().getInt("hama.graph.max.iteration",
         -1);
 
-    vertexIdClass = (Class<V>) conf.getClass(GraphJob.VERTEX_ID_CLASS_ATTR,
-        Text.class, Writable.class);
-    vertexValueClass = (Class<M>) conf.getClass(
-        GraphJob.VERTEX_VALUE_CLASS_ATTR, IntWritable.class, Writable.class);
-    edgeValueClass = (Class<E>) conf.getClass(
-        GraphJob.VERTEX_EDGE_VALUE_CLASS_ATTR, IntWritable.class,
-        Writable.class);
+      Class<V> vertexIdClass = (Class<V>) conf.getClass(GraphJob.VERTEX_ID_CLASS_ATTR,
+              Text.class, Writable.class);
+      Class<M> vertexValueClass = (Class<M>) conf.getClass(
+              GraphJob.VERTEX_VALUE_CLASS_ATTR, IntWritable.class, Writable.class);
+      Class<E> edgeValueClass = (Class<E>) conf.getClass(
+              GraphJob.VERTEX_EDGE_VALUE_CLASS_ATTR, IntWritable.class,
+              Writable.class);
     vertexClass = (Class<Vertex<V, E, M>>) conf.getClass(
         "hama.graph.vertex.class", Vertex.class);
 
@@ -281,7 +278,7 @@ public final class GraphJobRunner<V exte
     if (LOG.isDebugEnabled())
       LOG.debug("Vertex class: " + vertexClass);
 
-    KeyValuePair<Writable, Writable> next = null;
+    KeyValuePair<Writable, Writable> next;
     while ((next = peer.readNext()) != null) {
       Vertex<V, E, M> vertex = (Vertex<V, E, M>) next.getKey();
       vertex.runner = this;
@@ -327,7 +324,7 @@ public final class GraphJobRunner<V exte
     }
 
     peer.sync();
-    GraphJobMessage msg = null;
+    GraphJobMessage msg;
     while ((msg = peer.getCurrentMessage()) != null) {
       V vertexName = (V) msg.getVertexId();
 
@@ -342,7 +339,6 @@ public final class GraphJobRunner<V exte
       }
       newVertex.setup(conf);
       tmp.put(vertexName, newVertex);
-      newVertex = null;
     }
 
     for (Vertex<V, E, M> e : vertices) {
@@ -370,7 +366,7 @@ public final class GraphJobRunner<V exte
 
     peer.sync();
 
-    GraphJobMessage msg = null;
+    GraphJobMessage msg;
     while ((msg = peer.getCurrentMessage()) != null) {
       if (msg.isVerticesSizeMessage()) {
         numberVertices += msg.getVerticesSize().get();
@@ -392,7 +388,7 @@ public final class GraphJobRunner<V exte
   private Map<V, List<M>> parseMessages(
       BSPPeer<Writable, Writable, Writable, Writable, GraphJobMessage> peer)
       throws IOException {
-    GraphJobMessage msg = null;
+    GraphJobMessage msg;
     final Map<V, List<M>> msgMap = new HashMap<V, List<M>>();
     while ((msg = peer.getCurrentMessage()) != null) {
       // either this is a vertex message or a directive that must be read
@@ -511,10 +507,8 @@ public final class GraphJobRunner<V exte
    */
   public static <V extends Writable, E extends Writable, M extends Writable> Vertex<V, E, M> newVertexInstance(
       Class<?> vertexClass, Configuration conf) {
-    @SuppressWarnings("unchecked")
-    Vertex<V, E, M> vertex = (Vertex<V, E, M>) ReflectionUtils.newInstance(
+    return (Vertex<V, E, M>) ReflectionUtils.newInstance(
         vertexClass, conf);
-    return vertex;
   }
 
 }

Modified: hama/trunk/graph/src/main/java/org/apache/hama/graph/VertexInputReader.java
URL: http://svn.apache.org/viewvc/hama/trunk/graph/src/main/java/org/apache/hama/graph/VertexInputReader.java?rev=1442490&r1=1442489&r2=1442490&view=diff
==============================================================================
--- hama/trunk/graph/src/main/java/org/apache/hama/graph/VertexInputReader.java (original)
+++ hama/trunk/graph/src/main/java/org/apache/hama/graph/VertexInputReader.java Tue Feb  5 08:09:07 2013
@@ -35,7 +35,7 @@ public abstract class VertexInputReader<
 
   private static final Log LOG = LogFactory.getLog(VertexInputReader.class);
 
-  private KeyValuePair<Writable, Writable> outputRecord = new KeyValuePair<Writable, Writable>();
+  private final KeyValuePair<Writable, Writable> outputRecord = new KeyValuePair<Writable, Writable>();
 
   /**
    * Parses a given key and value into the given vertex. If returned true, the
@@ -51,7 +51,7 @@ public abstract class VertexInputReader<
       KeyValuePair<Writable, Writable> inputRecord, Configuration conf) {
     Class<Vertex<V, E, M>> vertexClass = (Class<Vertex<V, E, M>>) conf
         .getClass(GraphJob.VERTEX_CLASS_ATTR, Vertex.class);
-    boolean vertexCreation = true;
+    boolean vertexCreation;
     Vertex<V, E, M> vertex = GraphJobRunner
         .newVertexInstance(vertexClass, conf);
     try {

Modified: hama/trunk/graph/src/main/java/org/apache/hama/graph/VerticesInfo.java
URL: http://svn.apache.org/viewvc/hama/trunk/graph/src/main/java/org/apache/hama/graph/VerticesInfo.java?rev=1442490&r1=1442489&r2=1442490&view=diff
==============================================================================
--- hama/trunk/graph/src/main/java/org/apache/hama/graph/VerticesInfo.java (original)
+++ hama/trunk/graph/src/main/java/org/apache/hama/graph/VerticesInfo.java Tue Feb  5 08:09:07 2013
@@ -35,7 +35,7 @@ import org.apache.hadoop.io.Writable;
 public class VerticesInfo<V extends Writable, E extends Writable, M extends Writable>
     implements Iterable<Vertex<V, E, M>> {
 
-  private List<Vertex<V, E, M>> vertices = new ArrayList<Vertex<V, E, M>>(100);
+  private final List<Vertex<V, E, M>> vertices = new ArrayList<Vertex<V, E, M>>(100);
 
   public void addVertex(Vertex<V, E, M> vertex) {
     int i = 0;