helix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hu...@apache.org
Subject [helix] 23/44: TASK: Fix String formatting issue
Date Sat, 25 May 2019 01:19:57 GMT
This is an automated email from the ASF dual-hosted git repository.

hulee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/helix.git

commit a749afa0cfcb52fdf7be970017f3c2a452e2e819
Author: Hunter Lee <hulee@linkedin.com>
AuthorDate: Tue Apr 16 13:21:33 2019 -0700

    TASK: Fix String formatting issue
    
    For integers, you must use %d, not %f.
    
    RB=1633161
    BUG=HELIX-1794
    G=helix-reviewers
    A=jxue
    
    Signed-off-by: Hunter Lee <hulee@linkedin.com>
---
 helix-core/src/main/java/org/apache/helix/task/TaskDriver.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/helix-core/src/main/java/org/apache/helix/task/TaskDriver.java b/helix-core/src/main/java/org/apache/helix/task/TaskDriver.java
index 114b6a2..fb756e8 100644
--- a/helix-core/src/main/java/org/apache/helix/task/TaskDriver.java
+++ b/helix-core/src/main/java/org/apache/helix/task/TaskDriver.java
@@ -426,7 +426,7 @@ public class TaskDriver {
       Set<String> allNodes = jobDag.getAllNodes();
       if (capacity > 0 && allNodes.size() + jobConfigs.size() >= capacity)
{
         throw new IllegalStateException(
-            String.format("Queue %s already reaches its max capacity %f, failed to add %s",
queue,
+            String.format("Queue %s already reaches its max capacity %d, failed to add %s",
queue,
                 capacity, jobs.toString()));
       }
 


Mime
View raw message