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

kishoreg pushed a commit to branch pinot-fs-bug-fix
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git


The following commit(s) were added to refs/heads/pinot-fs-bug-fix by this push:
     new 4c3eed8  Fixing code to fetch the fsConfig from the right parent
4c3eed8 is described below

commit 4c3eed8655a3b5d1cd6f702af2f7541b5c688ce1
Author: kishoreg <g.kish...@gmail.com>
AuthorDate: Wed Jul 8 17:34:23 2020 -0700

    Fixing code to fetch the fsConfig from the right parent
---
 .../java/org/apache/pinot/spi/filesystem/PinotFSFactory.java | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/pinot-spi/src/main/java/org/apache/pinot/spi/filesystem/PinotFSFactory.java 
b/pinot-spi/src/main/java/org/apache/pinot/spi/filesystem/PinotFSFactory.java
index 7003a77..eb5367c 100644
--- 
a/pinot-spi/src/main/java/org/apache/pinot/spi/filesystem/PinotFSFactory.java
+++ 
b/pinot-spi/src/main/java/org/apache/pinot/spi/filesystem/PinotFSFactory.java
@@ -60,19 +60,19 @@ public class PinotFSFactory {
     }
   }
 
-  public static void init(PinotConfiguration fsConfig) {
+  public static void init(PinotConfiguration config) {
     // Get schemes and their respective classes
-    PinotConfiguration schemesConfiguration = fsConfig.subset(CLASS);
+    PinotConfiguration schemesConfiguration = config.subset(CLASS);
     List<String> schemes = schemesConfiguration.getKeys();
     if (!schemes.isEmpty()) {
       LOGGER.info("Did not find any fs classes in the configuration");
     }
     
     for(String scheme : schemes){
-      String fsClassName = (String) schemesConfiguration.getProperty(scheme);
-      
-      LOGGER.info("Got scheme {}, classname {}, starting to initialize", 
scheme, fsClassName);
-      register(scheme, fsClassName, schemesConfiguration.subset(scheme));
+      String fsClassName = schemesConfiguration.getProperty(scheme);
+      PinotConfiguration fsConfiguration = config.subset(scheme);
+      LOGGER.info("Got scheme {}, initializing class {} with config : {} ", 
scheme, fsClassName, fsConfiguration.toMap());
+      register(scheme, fsClassName, fsConfiguration);
     }
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to