commit d83a0be4a27991dc9ecc5b230358e88211532181
author Xiaoyao Qian <qian13@illinois.edu> Mon Jan 28 21:36:36 2019 -0800
committer Ning Wang <nwang@twitter.com> Mon Jan 28 21:36:36 2019 -0800
tree b98bd85556d947db67583d261a7174c3d6794045
parent e82293bc7a1d6514cafbf00eaaae24cb05bbf4ec

Add log line to avoid ambiguity (#3177)

heron/packing/src/java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java

diff --git a/heron/packing/src/java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java b/heron/packing/src/java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java
index 1eef9a6..c48d6a2 100644
--- a/heron/packing/src/java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java
+++ b/heron/packing/src/java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java
@@ -284,9 +284,9 @@
       if (!containerResHint.equals(notSpecified)
           && usedRes.greaterThan(containerResHint.minus(containerResPadding))) {
         throw new PackingException(String.format("Invalid packing plan generated. "
-                + "Total instance %s in a container (%s) have exceeded "
+                + "Total instance %s in a container (%s) + padding(%s) have exceeded "
                 + "the container-level constraint of %s.",
-            resourceType, usedRes.toString(), containerResHint));
+            resourceType, usedRes.toString(), containerResPadding.toString(), containerResHint));
       }
 
       // calculate resource for the remaining unspecified instances if any