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

    https://github.com/apache/carbondata/pull/2254#discussion_r185429022
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datamap/DataMapChooser.java ---
    @@ -291,8 +295,8 @@ private boolean contains(DataMapMeta mapMeta, 
List<ColumnExpression> columnExpre
         }
         boolean contains = true;
         for (ColumnExpression expression : columnExpressions) {
    -      if 
(!mapMeta.getIndexedColumns().contains(expression.getColumnName()) || !mapMeta
    -          .getOptimizedOperation().containsAll(expressionTypes)) {
    +      if 
(!mapMeta.getIndexedColumnNames().contains(expression.getColumnName()) ||
    +          !mapMeta.getOptimizedOperation().containsAll(expressionTypes)) {
    --- End diff --
    
    expressionTypes is from the FilterResolverIntf. So it is from the 
expression of the user query. See selectDataMap function


---

Reply via email to