[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

[qiuchenjian]

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

    https://github.com/apache/carbondata/pull/3063#discussion_r246973311
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java
 ---
    @@ -947,6 +947,16 @@ public int getNumberOfNoDictSortColumns() {
         return numberOfNoDictSortColumns;
       }
     
    +  public CarbonColumn getRangeColumn() {
    +    String rangeColumn =
    +        
tableInfo.getFactTable().getTableProperties().get(CarbonCommonConstants.RANGE_COLUMN);
    +    if (rangeColumn == null) {
    --- End diff --
    
    Does this branch need to judge that rangeColumn is ""


---