>From Hussain Towaileb <[email protected]>: Hussain Towaileb has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18326 )
Change subject: [NO ISSUE]: Skip logging empty list if no files will be deleted ...................................................................... [NO ISSUE]: Skip logging empty list if no files will be deleted Change-Id: I854f39689b30c3c3fa53cb845477c321441e97a2 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18326 Reviewed-by: Murtadha Hubail <[email protected]> Tested-by: Hussain Towaileb <[email protected]> --- M asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/lazy/ParallelCacher.java 1 file changed, 16 insertions(+), 1 deletion(-) Approvals: Murtadha Hubail: Looks good to me, approved Hussain Towaileb: Verified diff --git a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/lazy/ParallelCacher.java b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/lazy/ParallelCacher.java index a3079e6..00db458 100644 --- a/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/lazy/ParallelCacher.java +++ b/asterixdb/asterix-cloud/src/main/java/org/apache/asterix/cloud/lazy/ParallelCacher.java @@ -157,7 +157,10 @@ @Override public boolean remove(Collection<FileReference> deletedFiles) { - LOGGER.info("Deleting {}", deletedFiles); + if (!deletedFiles.isEmpty()) { + LOGGER.info("Deleting {}", deletedFiles); + } + for (FileReference fileReference : deletedFiles) { remove(fileReference); } -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18326 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I854f39689b30c3c3fa53cb845477c321441e97a2 Gerrit-Change-Number: 18326 Gerrit-PatchSet: 2 Gerrit-Owner: Hussain Towaileb <[email protected]> Gerrit-Reviewer: Hussain Towaileb <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-CC: Anon. E. Moose #1000171 Gerrit-CC: Jenkins <[email protected]> Gerrit-MessageType: merged
