shauryachats commented on code in PR #17205:
URL: https://github.com/apache/pinot/pull/17205#discussion_r2525376490
##########
pinot-integration-test-base/src/test/java/org/apache/pinot/integration/tests/BaseClusterIntegrationTest.java:
##########
@@ -809,6 +815,11 @@ protected void waitForDocsLoaded(long timeoutMs, boolean
raiseError, String tabl
"Failed to load " + countStarResult + " documents", raiseError,
Duration.ofMillis(timeoutMs / 10));
}
+ protected void waitForNonZeroDocsLoaded(long timeoutMs, boolean raiseError,
String tableName) {
+ TestUtils.waitForCondition(() -> getCurrentCountStarResult(tableName) > 0,
100L, timeoutMs,
+ "Failed to load non zero documents", raiseError,
Duration.ofMillis(timeoutMs / 10));
Review Comment:
I copied it from the above method.
This field is `@Nullable Duration logPeriod`, which determines the
periodicity at which the error log should be printed to avoid spamming the
error log. (timeouts are generally 60s and this method runs every 100ms).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
