Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2148#discussion_r181586769 --- Diff: integration/spark2/src/main/java/org/apache/carbondata/spark/vectorreader/VectorizedCarbonRecordReader.java --- @@ -134,9 +133,7 @@ public void initialize(InputSplit inputSplit, TaskAttemptContext taskAttemptCont queryModel.setTableBlockInfos(tableBlockInfoList); queryModel.setVectorReader(true); try { - if (CarbonProperties.getInstance().getProperty( - CarbonCommonConstants.CARBON_SEARCH_MODE_ENABLE, - CarbonCommonConstants.CARBON_SEARCH_MODE_ENABLE_DEFAULT).equals("true")) { + if (CarbonProperties.isSearchModeEnabled()) { --- End diff -- I guess no need to do if check here, already `QueryExecutorFactory` available so move the code there.
---