Github user dlogothetis commented on a diff in the pull request:

    https://github.com/apache/giraph/pull/87#discussion_r225302450
  
    --- Diff: 
giraph-core/src/main/java/org/apache/giraph/edge/ByteArrayEdges.java ---
    @@ -94,6 +94,9 @@ public void add(Edge<I, E> edge) {
         } catch (IOException e) {
           throw new IllegalStateException("add: Failed to write to the new " +
               "byte array");
    +    } catch (NegativeArraySizeException negativeArraySizeException) {
    +      throw new IllegalStateException("add: Too many edges for a vertex, " 
+
    --- End diff --
    
    Could we add the ID of the vertex in the exception message? It may be 
useful for debugging purposes.


---

Reply via email to