Murtadha Hubail has posted comments on this change.

Change subject: [NO ISSUE][OBS] Log content of index dir with missing 
checkpoints
......................................................................


Patch Set 1:

(1 comment)

https://asterix-gerrit.ics.uci.edu/#/c/2801/1/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/IndexCheckpointManager.java
File 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/IndexCheckpointManager.java:

PS1, Line 184: if (checkpoints.isEmpty()) {
             :             LOGGER.warn("Couldn't find any checkpoint file for 
index {}. Content of dir are {}.", indexPath,
             :                     
Arrays.toString(indexPath.toFile().listFiles()));
             :         }
this will now be called on every index create. Let's remove it from here and 
put it in getLatest before the illegal state exception


-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2801
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1903a40e129f648d4a0f8be6e820c6d45c722bec
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Reviewer: abdullah alamoudi <[email protected]>
Gerrit-HasComments: Yes

Reply via email to