[GitHub] carbondata pull request #2606: [CARBONDATA-2817]Thread Leak in Update and in...

[kumarvishal09]

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

    https://github.com/apache/carbondata/pull/2606#discussion_r207321150
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/steps/CarbonRowDataWriterProcessorStepImpl.java
 ---
    @@ -80,11 +82,16 @@
     
       private Map<String, LocalDictionaryGenerator> 
localDictionaryGeneratorMap;
     
    +  private List<CarbonFactHandler> carbonFactHandlers;
    +
    +  ExecutorService executorService = null;
    --- End diff --
    
    make it private


---