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

    https://github.com/apache/carbondata/pull/2882#discussion_r229300837
  
    --- Diff: 
tools/cli/src/main/java/org/apache/carbondata/tool/DataSummary.java ---
    @@ -314,23 +312,26 @@ private void printColumnStats(String columnName) 
throws IOException, MemoryExcep
               minPercent = String.format("%.1f", 
blocklet.getColumnChunk().getMinPercentage() * 100);
               maxPercent = String.format("%.1f", 
blocklet.getColumnChunk().getMaxPercentage() * 100);
               DataFile.ColumnChunk columnChunk = blocklet.columnChunk;
    -          if (columnChunk.column.isDimensionColumn() && DataTypeUtil
    +          if (columnChunk.column.hasEncoding(Encoding.DICTIONARY) || 
blocklet
    +              .getColumnChunk().column.getColumnName().contains(".val") || 
blocklet
    --- End diff --
    
    this will be for no dictionary colmplex column, for complex column min max 
can be shown as NA, that will be ok right


---

Reply via email to