Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1350#discussion_r178484642
--- 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 --
So we need to understand what's the different between MarkRestrNotAllowed
and NULL, is it possible to cause crash if setting NULL ?
---
