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

    https://github.com/apache/carbondata/pull/2706#discussion_r216885637
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/sort/unsafe/UnsafeSortDataRows.java
 ---
    @@ -72,7 +72,7 @@
     
       private SortParameters parameters;
       private TableFieldStat tableFieldStat;
    -  private ThreadLocal<ByteBuffer> rowBuffer;
    +  private static ThreadLocal<ByteBuffer> rowBuffer;
    --- End diff --
    
    I think the 'static' here may cause problem for concurrent loading. Each 
loading should their own rowBuffer.


---

Reply via email to