[GitHub] carbondata pull request #2484: [WIP] added hadoop conf to thread local

[kunal642]

Github user kunal642 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2484#discussion_r202586005
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/impl/FileFactory.java 
---
    @@ -43,11 +45,19 @@
        */
       private static final LogService LOGGER =
           LogServiceFactory.getLogService(FileFactory.class.getName());
    -  private static Configuration configuration = null;
     
    -  static {
    -    configuration = new Configuration();
    +  public static Configuration getConfiguration() {
    +    Configuration configuration;
    +    if (ThreadLocalSessionInfo.getCarbonSessionInfo() == null) {
    +      configuration = new Configuration();
    +    } else {
    +      CarbonConfiguration carbonConfiguration =
    +          (CarbonConfiguration) 
ThreadLocalSessionInfo.getCarbonSessionInfo().getThreadParams()
    +              .getExtraInfo("carbonConf", new CarbonConfiguration());
    +      configuration = carbonConfiguration.getConfiguration();
    +    }
         configuration.addResource(new Path("../core-default.xml"));
    --- End diff --
    
    done


---