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

    https://github.com/apache/carbondata/pull/2318#discussion_r190279377
  
    --- Diff: 
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonInputFormat.java ---
    @@ -504,7 +505,24 @@ public QueryModel createQueryModel(InputSplit 
inputSplit, TaskAttemptContext tas
         String projectionString = getColumnProjection(configuration);
         String[] projectColumns;
         if (projectionString != null) {
    -      projectColumns = projectionString.split(",");
    +      if (projectionString.equalsIgnoreCase("*")) {
    --- End diff --
    
    instead of passing `*`, I think better to add another function to project 
all columns. You can add `projectAllColumns()`


---

Reply via email to