You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by vi...@apache.org on 2020/07/25 01:47:23 UTC

[hudi] branch master updated: [MINOR] Add Databricks File System to StorageSchemes (#1877)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 467d097  [MINOR] Add Databricks File System to StorageSchemes (#1877)
467d097 is described below

commit 467d097dae5d38bfba8e1249ab52fe0f294e6172
Author: Gary Li <ya...@gmail.com>
AuthorDate: Fri Jul 24 18:47:09 2020 -0700

    [MINOR] Add Databricks File System to StorageSchemes (#1877)
---
 .../src/main/java/org/apache/hudi/common/fs/StorageSchemes.java       | 4 +++-
 .../src/test/java/org/apache/hudi/common/fs/TestStorageSchemes.java   | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/hudi-common/src/main/java/org/apache/hudi/common/fs/StorageSchemes.java b/hudi-common/src/main/java/org/apache/hudi/common/fs/StorageSchemes.java
index 06b92fd..3e721d1 100644
--- a/hudi-common/src/main/java/org/apache/hudi/common/fs/StorageSchemes.java
+++ b/hudi-common/src/main/java/org/apache/hudi/common/fs/StorageSchemes.java
@@ -49,7 +49,9 @@ public enum StorageSchemes {
   //ALLUXIO
   ALLUXIO("alluxio", false),
   // Tencent Cloud Object Storage
-  COSN("cosn", false);
+  COSN("cosn", false),
+  // Databricks file system
+  DBFS("dbfs", false);
 
   private String scheme;
   private boolean supportsAppend;
diff --git a/hudi-common/src/test/java/org/apache/hudi/common/fs/TestStorageSchemes.java b/hudi-common/src/test/java/org/apache/hudi/common/fs/TestStorageSchemes.java
index 48e4b75..dcb1206 100644
--- a/hudi-common/src/test/java/org/apache/hudi/common/fs/TestStorageSchemes.java
+++ b/hudi-common/src/test/java/org/apache/hudi/common/fs/TestStorageSchemes.java
@@ -42,6 +42,7 @@ public class TestStorageSchemes {
     assertTrue(StorageSchemes.isAppendSupported("viewfs"));
     assertFalse(StorageSchemes.isAppendSupported("alluxio"));
     assertFalse(StorageSchemes.isAppendSupported("cosn"));
+    assertFalse(StorageSchemes.isAppendSupported("dbfs"));
     assertThrows(IllegalArgumentException.class, () -> {
       StorageSchemes.isAppendSupported("s2");
     }, "Should throw exception for unsupported schemes");