Github user Librago commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/1384#discussion_r208435844
--- Diff: src/backend/access/external/fileam.c ---
@@ -1290,6 +1397,97 @@ externalgettup_custom(FileScanDesc scan,
ExternalSelectDesc desc, ScanState *ss)
return NULL;
}
+static HeapTuple
+externalgettup_custom_noextprot(FileScanDesc scan,
+
ExternalSelectDesc desc,
+ ScanState *ss)
+{
+ HeapTuple tuple;
+ CopyState pstate = scan->fs_pstate;
+ FormatterData* formatter = scan->fs_formatter;
+ bool no_more_data = false;
+ MemoryContext oldctxt = CurrentMemoryContext;
+
+ Assert(formatter);
+
+ /* while didn't finish processing the entire file */
+ while (!no_more_data)
+ {
+ bool error_caught = false;
+
+ /*
+ * Invoke the custom formatter function.
+ */
+ PG_TRY();
+ {
+ Datum d;
+ FunctionCallInfoData fcinfo;
--- End diff --
incorrect indent
---