You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by gvramana <gi...@git.apache.org> on 2018/04/16 06:31:32 UTC

[GitHub] carbondata pull request #2141: [CARBONDATA-2313] Fixed SDK writer issues and...

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

    https://github.com/apache/carbondata/pull/2141#discussion_r181632323
  
    --- Diff: core/src/main/java/org/apache/carbondata/core/scan/executor/util/RestructureUtil.java ---
    @@ -337,11 +351,12 @@ public static Object getMeasureDefaultValueByType(ColumnSchema columnSchema,
        * @param blockExecutionInfo
        * @param queryMeasures        measures present in query
        * @param currentBlockMeasures current block measures
    +   * @param queryModel carbonQueryModel
        * @return measures present in the block
        */
       public static List<ProjectionMeasure> createMeasureInfoAndGetCurrentBlockQueryMeasures(
           BlockExecutionInfo blockExecutionInfo, List<ProjectionMeasure> queryMeasures,
    -      List<CarbonMeasure> currentBlockMeasures) {
    +      List<CarbonMeasure> currentBlockMeasures, QueryModel queryModel) {
    --- End diff --
    
    Pass flag, dont pass all the querymodel, Restructuring doesnot have any relation with Querymodel


---