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