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

    https://github.com/apache/incubator-hawq/pull/1350#discussion_r178458812
  
    --- Diff: contrib/vexecutor/execVScan.c ---
    @@ -38,8 +40,8 @@ getVScanMethod(int tableType)
                         },
                         //PARQUETSCAN
                         {
    -                            &ParquetScanNext, &BeginScanParquetRelation, 
&EndScanParquetRelation,
    -                            &ReScanParquetRelation, &MarkRestrNotAllowed, 
&MarkRestrNotAllowed
    +                            &ParquetVScanNext, &BeginScanParquetRelation, 
&EndScanParquetRelation,
    +                            NULL,NULL,NULL
    --- End diff --
    
    Why these function pointers are set NULLs ?


---

Reply via email to