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

    https://github.com/apache/incubator-hawq/pull/1350#discussion_r178502483
  
    --- Diff: contrib/vexecutor/vcheck.h ---
    @@ -39,6 +39,7 @@ typedef struct VectorizedState
     {
        bool vectorized;
        PlanState *parent;
    +    bool* proj;
    --- End diff --
    
    Because this variable is only related to ao table, or has different content 
based on different table format. It's better to rename it to a more readable 
name, aoprojs or just use (void * opaque).


---

Reply via email to