This is an automated email from the ASF dual-hosted git repository.

wyk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git


The following commit(s) were added to refs/heads/master by this push:
     new 273fb7762a [NO ISSUE][OTH] Check folder exists when loading a directory
273fb7762a is described below

commit 273fb7762a712b2763391a37b523b0ad45d418e9
Author: Wail Alkowaileet <[email protected]>
AuthorDate: Tue Aug 8 18:17:57 2023 -0700

    [NO ISSUE][OTH] Check folder exists when loading a directory
    
    - user model changes: no
    - storage format changes: no
    - interface changes: no
    
    Details:
    When loading a directory to S3Mock, check if the directory
    already exists.
    
    Change-Id: I8709370239d39512c87d94b3232e773275887ccf
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17710
    Integration-Tests: Jenkins <[email protected]>
    Reviewed-by: Hussain Towaileb <[email protected]>
    Tested-by: Jenkins <[email protected]>
---
 .../asterix/test/external_dataset/ExternalDatasetTestUtils.java    | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/external_dataset/ExternalDatasetTestUtils.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/external_dataset/ExternalDatasetTestUtils.java
index b50c353464..7da233454a 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/external_dataset/ExternalDatasetTestUtils.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/external_dataset/ExternalDatasetTestUtils.java
@@ -283,7 +283,12 @@ public class ExternalDatasetTestUtils {
     }
 
     private static void loadDirectory(String dataBasePath, String rootPath, 
FilenameFilter filter) {
-        Collection<File> files = IoUtil.getMatchingFiles(Path.of(dataBasePath, 
rootPath), filter);
+        File dir = new File(dataBasePath, rootPath);
+        if (!dir.exists() || !dir.isDirectory()) {
+            return;
+        }
+
+        Collection<File> files = IoUtil.getMatchingFiles(dir.toPath(), filter);
         int size = 0;
         for (File file : files) {
             String path = file.getPath();

Reply via email to