Github user wengyanqing commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1350#discussion_r178459027
--- Diff: contrib/vexecutor/execVScan.c ---
@@ -148,7 +181,10 @@ ExecVScan(ScanState *node, ExecScanAccessMtd accessMtd)
* Form a projection tuple, store it in the result tuple
slot
* and return it.
*/
- return ExecProject(projInfo, NULL);
+ ((TupleBatch)projInfo->pi_slot->PRIVATE_tb)->nrows =
((TupleBatch)slot->PRIVATE_tb)->nrows;
+ memcpy(((TupleBatch)projInfo->pi_slot->PRIVATE_tb)->skip,
--- End diff --
If possible, avoid memory copy during tuple batch processing.
---
