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

    https://github.com/apache/incubator-hawq/pull/1384#discussion_r208438165
  
    --- Diff: src/include/access/formatter.h ---
    @@ -36,10 +36,19 @@
     typedef enum FmtNotification
     {
        FMT_NONE,
    +   FMT_DONE,
        FMT_NEED_MORE_DATA
     
     } FmtNotification;
     
    +typedef enum FmtActionMask
    --- End diff --
    
    The pluggable storage framework do not use this structure, remove it


---

Reply via email to