Bryan Green <[email protected]> writes: > One notable behavioral change: check hooks using GUC_EXTRA_IS_CONTEXT > now use palloc() instead of guc_malloc(). The old approach with > guc_malloc() allowed check hooks to return false on OOM, letting the > caller handle it at the appropriate error level. With palloc() an OOM > throws an immediate ERROR. This seemed like an acceptable tradeoff
Why? It seems both inconsistent and unsafe.
regards, tom lane
