[incubator-heron] branch master updated: Add log line to avoid ambiguity (#3177)

[nw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

nwang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git


The following commit(s) were added to refs/heads/master by this push:
     new d83a0be  Add log line to avoid ambiguity (#3177)
d83a0be is described below

commit d83a0be4a27991dc9ecc5b230358e88211532181
Author: Xiaoyao Qian <qi...@illinois.edu>
AuthorDate: Mon Jan 28 21:36:36 2019 -0800

    Add log line to avoid ambiguity (#3177)
---
 .../java/org/apache/heron/packing/roundrobin/RoundRobinPacking.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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 @@ public class RoundRobinPacking implements IPacking, IRepacking {
       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