luoluoyuyu commented on code in PR #17730:
URL: https://github.com/apache/iotdb/pull/17730#discussion_r3309367795
##########
integration-test/src/test/java/org/apache/iotdb/db/it/IoTDBLoadTsFileWithModIT.java:
##########
@@ -222,22 +223,29 @@ public void testWithNewModFileAndLoadAttributes()
tmpDir.getAbsolutePath(), databaseName));
boolean databaseFound = false;
- out:
for (int i = 0; i < 10; i++) {
try (final ResultSet resultSet = statement.executeQuery("show
databases")) {
while (resultSet.next()) {
final String currentDatabase = resultSet.getString(1);
if (databaseName.equalsIgnoreCase(currentDatabase)) {
databaseFound = true;
- break out;
+ break;
}
}
+ } catch (InconsistentDataException ignored) {
Review Comment:
Catching InconsistentDataException and retrying is appropriate for this
async load scenario. Optional: if all 10 retries fail, consider mentioning
retry count in the final assertion message to simplify debugging. Not blocking.
--
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]
