[GitHub] carbondata pull request #2864: [CARBONDATA-3041] Optimize load minimum size ...

[ndwangsen]

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

    https://github.com/apache/carbondata/pull/2864#discussion_r228708260
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/util/AlterTableUtil.scala ---
    @@ -748,4 +752,18 @@ object AlterTableUtil {
           false
         }
       }
    +
    +  private def validateLoadMinSizeProperties(carbonTable: CarbonTable,
    +      propertiesMap: mutable.Map[String, String]): Unit = {
    +    // validate load min size property
    +    if 
(propertiesMap.get(CarbonCommonConstants.CARBON_LOAD_MIN_SIZE_INMB).isDefined) {
    +      // Cache level is not allowed for child tables and dataMaps
    --- End diff --
    
    Has been modified based on the review


---