You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/01/30 23:05:43 UTC

[GitHub] [accumulo] ctubbsii opened a new issue #1491: Broken ChaoticBalancerIT

ctubbsii opened a new issue #1491: Broken ChaoticBalancerIT
URL: https://github.com/apache/accumulo/issues/1491
 
 
   The ChaoticBalancerIT keeps failing. I found the following in the master logs. It seems the 
   
   ```
   2020-01-30 17:52:32,964 [master.Master] ERROR: Error processing table state for store Metadata Tablets
   java.lang.IllegalArgumentException: bound must be positive                                          
     at java.base/java.util.Random.nextInt(Random.java:388)                                            
     at org.apache.accumulo.server.master.balancer.ChaoticLoadBalancer.getAssignments(ChaoticLoadBalancer.java:90)
     at org.apache.accumulo.server.master.balancer.TableLoadBalancer.getAssignments(TableLoadBalancer.java:131)
     at org.apache.accumulo.master.TabletGroupWatcher.flushChanges(TabletGroupWatcher.java:875)        
     at org.apache.accumulo.master.TabletGroupWatcher.run(TabletGroupWatcher.java:361)  
   ```
   
   The code appears to fail here in ChaoticLoadBalanacer, when assigning `int index = r.nextInt(tServerArray.size());`
   
   ```java
       if (tServerArray.isEmpty()) {                                                                   
         // No tservers to assign to                                                                   
         return;                                                                                       
       }                                                                                               
                                                                                                       
       for (KeyExtent ke : unassigned.keySet()) {                                                      
         int index = r.nextInt(tServerArray.size());                                                   
         TServerInstance dest = tServerArray.get(index);
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [accumulo] ctubbsii commented on issue #1491: Broken ChaoticBalancerIT

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #1491: Broken ChaoticBalancerIT
URL: https://github.com/apache/accumulo/issues/1491#issuecomment-580678367
 
 
   This doesn't seem to be an issue in 2.x, because it was stabilized already by @milleruntime in ACCUMULO-4602 (520d23d85dfce9297bab887726322861163a1fb2). I'm going to backport those fixes to 1.x

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [accumulo] asfgit closed issue #1491: Broken ChaoticBalancerIT

Posted by GitBox <gi...@apache.org>.
asfgit closed issue #1491: Broken ChaoticBalancerIT
URL: https://github.com/apache/accumulo/issues/1491
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services