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

    https://github.com/apache/carbondata/pull/2869#discussion_r230274806
  
    --- Diff: 
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReaderBuilder.java 
---
    @@ -51,6 +54,7 @@
       private Expression filterExpression;
       private String tableName;
       private Configuration hadoopConf;
    +  private boolean useVectorReader;
    --- End diff --
    
    I suggest we can make vector as default as we see benefit in the 
performance.  
    withVectorReader() can be a deprecated to support just for disable  in case 
of issues till the time vector reader is stabilized. @ravipesala , is the fine?


---

Reply via email to