Review Board 1.7.22


GIRAPH-128: RPC port from BasicRPCCommunications should be only a starting port, and retried

Review Request #3596 - Created Jan. 23, 2012 and updated

Avery Ching
trunk
GIRAPH-128
Reviewers
giraph
giraph
Simple handling of port collisions on the same machine while preserving debugability from the port number alone.  Round up the max number of workers to the next power of 10 and use it as a constant to increase the port number with.

Added a unit test to ensure it is working correctly.

Fixed 2 minor warnings on
src/main/java/org/apache/giraph/examples/MinimumIntCombiner.java
src/main/java/org/apache/giraph/examples/SimpleSumCombiner.java

of removing 'import java.util.List'.
Passed local and MR unittests.
Review request changed
Updated (Jan. 28, 2012, 3:01 a.m.)
Addressed the mockito suggestion.