[GitHub] carbondata pull request #2484: [HOTFIX] 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_r203263551
  
    --- 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();
    -    configuration.addResource(new Path("../core-default.xml"));
    +  public static Configuration getConfiguration() {
    +    Configuration configuration;
    +    if (ThreadLocalSessionInfo.getCarbonSessionInfo() == null) {
    +      configuration = new Configuration();
    +      configuration.addResource(new Path("../core-default.xml"));
    +    } else {
    +      CarbonConfiguration carbonConfiguration =
    +          (CarbonConfiguration) 
ThreadLocalSessionInfo.getCarbonSessionInfo().getThreadParams()
    --- End diff --
    
    already handled If carbonConf is not present taking new CarbonConfiguration 
 as default value


---