keith-turner commented on code in PR #5913:
URL: https://github.com/apache/accumulo/pull/5913#discussion_r2376436268


##########
test/src/main/java/org/apache/accumulo/test/ScanServerIT.java:
##########
@@ -103,7 +103,6 @@ public void configureMiniCluster(MiniAccumuloConfigImpl cfg,
 
       cfg.setProperty(Property.MANAGER_TABLET_GROUP_WATCHER_INTERVAL, "5");
       cfg.setProperty(Property.TSERV_ONDEMAND_UNLOADER_INTERVAL, "10");
-      
cfg.setProperty("table.custom.ondemand.unloader.inactivity.threshold.seconds", 
"15");

Review Comment:
   > I'm confused. Why were they hosted if hosting was never requested?
   
   The test initially does 
[this](https://github.com/apache/accumulo/blob/77fd0df3e06d4dc4c9695ecd07b47f1fd18abb17/test/src/main/java/org/apache/accumulo/test/ScanServerIT.java#L369-L371)
 which causes all tablets in the table to be initially hosted.  Then it 
eventually does 
[this](https://github.com/apache/accumulo/blob/77fd0df3e06d4dc4c9695ecd07b47f1fd18abb17/test/src/main/java/org/apache/accumulo/test/ScanServerIT.java#L383-L384)
 and the manager sees tablets that are ondemand, are hosted, and do not have 
the `requestToHost` to column set.  When the manager sees this it computes a 
goal state of unassigned.



-- 
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.

To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to