[GitHub] [helix] lei-xia commented on a change in pull request #1497: Fix TestRoutingTableProviderPeriodicRefresh.

[GitBox <gi...@apache.org>]

lei-xia commented on a change in pull request #1497:
URL: https://github.com/apache/helix/pull/1497#discussion_r514336457



##########
File path: helix-core/src/test/java/org/apache/helix/integration/spectator/TestRoutingTableProviderPeriodicRefresh.java
##########
@@ -64,6 +62,8 @@
   private static final int PARTITION_NUMBER = 20;
   private static final int REPLICA_NUMBER = 3;
 
+  private static final long REFRESH_PERIOD_MS = 1000L;

Review comment:
       why 1000ms, could 500ms, or even 100ms work? That will reduce the test time.




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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org