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

    https://github.com/apache/carbondata/pull/2469#discussion_r201261685
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
    @@ -3185,11 +3185,15 @@ private static int 
unsetLocalDictForComplexColumns(List<ColumnSchema> allColumns
           ColumnSchema column = allColumns.get(dimensionOrdinal);
           if (column.getNumberOfChild() > 0) {
             dimensionOrdinal++;
    -        unsetLocalDictForComplexColumns(allColumns, dimensionOrdinal, 
column.getNumberOfChild());
    +        // Value of dimensionOrdinal returned from recursive function was 
not considered
    --- End diff --
    
    do not give the reason in comment, why this issue was coming, add proper 
comment


---

Reply via email to