You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by to...@apache.org on 2018/04/11 12:46:53 UTC

svn commit: r1828898 - /jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java

Author: tomekr
Date: Wed Apr 11 12:46:53 2018
New Revision: 1828898

URL: http://svn.apache.org/viewvc?rev=1828898&view=rev
Log:
OAK-7398: SegmentArchiveManager#listArchives should only return tar files

- the AzureArchiveManager#exists() should return correct values for the
  non-tar archives as well (eg. .tar.bak)

Modified:
    jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java

Modified: jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java?rev=1828898&r1=1828897&r2=1828898&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java (original)
+++ jackrabbit/oak/trunk/oak-segment-azure/src/main/java/org/apache/jackrabbit/oak/segment/azure/AzureArchiveManager.java Wed Apr 11 12:46:53 2018
@@ -155,7 +155,7 @@ public class AzureArchiveManager impleme
     @Override
     public boolean exists(String archiveName) {
         try {
-            return listArchives().contains(archiveName);
+            return getBlobs(archiveName).findAny().isPresent();
         } catch (IOException e) {
             log.error("Can't check the existence of {}", archiveName, e);
             return false;