Persuade GIN to react to control-C in a reasonable amount of time

while building a GIN index.

diff --git a/src/backend/access/gin/gininsert.c b/src/backend/access/gin/gininsert.c
index 6fa07caf5aee72e63cb04b50053d59b83071cfa2..359d18615e7d8f5a7e6b24d4a102f0b30ff63fb6 100644 (file)
--- a/src/backend/access/gin/gininsert.c
+++ b/src/backend/access/gin/gininsert.c
@@ -246,7 +246,11 @@ ginBuildCallback(Relation index, HeapTuple htup, Datum *values,
                uint32          nlist;
 
                while ((list = ginGetEntry(&buildstate->accum, &entry, &nlist)) != NULL)
+               {
+                       /* there could be many entries, so be willing to abort here */
+                       CHECK_FOR_INTERRUPTS();
                        ginEntryInsert(index, &buildstate->ginstate, entry, list, nlist, TRUE);
+               }
 
                MemoryContextReset(buildstate->tmpCtx);
                ginInitBA(&buildstate->accum);
@@ -331,9 +335,14 @@ ginbuild(PG_FUNCTION_ARGS)
        reltuples = IndexBuildHeapScan(heap, index, indexInfo,
                                                                   ginBuildCallback, (void *) &buildstate);
 
+       /* dump remaining entries to the index */
        oldCtx = MemoryContextSwitchTo(buildstate.tmpCtx);
        while ((list = ginGetEntry(&buildstate.accum, &entry, &nlist)) != NULL)
+       {
+               /* there could be many entries, so be willing to abort here */
+               CHECK_FOR_INTERRUPTS();
                ginEntryInsert(index, &buildstate.ginstate, entry, list, nlist, TRUE);
+       }
        MemoryContextSwitchTo(oldCtx);
 
        MemoryContextDelete(buildstate.tmpCtx);