[GitHub] carbondata pull request #2414: [CARBONDATA-2658][DataLoad]No difference in m...

[ndwangsen]

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

    https://github.com/apache/carbondata/pull/2414#discussion_r199112802
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/sort/unsafe/merger/UnsafeIntermediateMerger.java
 ---
    @@ -88,13 +88,25 @@ public UnsafeIntermediateMerger(SortParameters 
parameters) {
               
CarbonLoadOptionConstants.CARBON_LOAD_SORT_MEMORY_SPILL_PERCENTAGE,
               
CarbonLoadOptionConstants.CARBON_LOAD_SORT_MEMORY_SPILL_PERCENTAGE_DEFAULT);
           spillPercentage = Integer.valueOf(spillPercentageStr);
    +      if (spillPercentage > 100 || spillPercentage < 0) {
    --- End diff --
    
    yes this pr base on #2397 


---