Allow _h_indexbuild() to be interrupted.

When we are building a hash index that is large enough to need
pre-sorting (larger than either maintenance_work_mem or NBuffers),
the initial sorting phase is interruptible, but the insertion
phase wasn't.  Add the missing CHECK_FOR_INTERRUPTS().

Per bug #18616 from Alexander Lakhin.  Back-patch to all
supported branches.

Pavel Borisov

Discussion: https://postgr.es/m/18616-acbb9e5caf41e964@postgresql.org

diff --git a/src/backend/access/hash/hashsort.c b/src/backend/access/hash/hashsort.c
index 293f80fe24cf27e957c46b46931cb19abb7f2226..75d3f4c3bc45ac3d760fff16d6d056d90e4f668c 100644 (file)
--- a/src/backend/access/hash/hashsort.c
+++ b/src/backend/access/hash/hashsort.c
@@ -145,6 +145,9 @@ _h_indexbuild(HSpool *hspool, Relation heapRel)
 
        _hash_doinsert(hspool->index, itup, heapRel);
 
+       /* allow insertion phase to be interrupted, and track progress */
+       CHECK_FOR_INTERRUPTS();
+
        pgstat_progress_update_param(PROGRESS_CREATEIDX_TUPLES_DONE,
                                     ++tups_done);
    }