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

    https://github.com/apache/incubator-hawq/pull/1384#discussion_r208435735
  
    --- Diff: src/backend/access/external/fileam.c ---
    @@ -782,6 +864,28 @@ external_insert(ExternalInsertDesc extInsertDesc, 
TupleTableSlot *tupTableSlot)
     void
     external_insert_finish(ExternalInsertDesc extInsertDesc)
     {
    +   /* Tell formatter to close */
    +   if (extInsertDesc->ext_formatter_data != NULL &&
    +           (extInsertDesc->ext_formatter_data->fmt_mask & FMT_NEEDEXTBUFF) 
== 0)
    +   {
    +           Datum d;
    +           FunctionCallInfoData fcinfo;
    +
    +           extInsertDesc->ext_formatter_data->fmt_mask |= FMT_WRITE_END;
    +
    +           /* per call formatter prep */
    +           FunctionCallPrepareFormatter(&fcinfo,
    --- End diff --
    
    incorrect indent


---

Reply via email to