• Jeff Davis's avatar
    Extend ExecBuildAggTrans() to support a NULL pointer check. · c954d490
    Jeff Davis authored
    Optionally push a step to check for a NULL pointer to the pergroup
    state.
    
    This will be important for disk-based hash aggregation in combination
    with grouping sets. When memory limits are reached, a given tuple may
    find its per-group state for some grouping sets but not others. For
    the former, it advances the per-group state as normal; for the latter,
    it skips evaluation and the calling code will have to spill the tuple
    and reprocess it in a later batch.
    
    Add the NULL check as a separate expression step because in some
    common cases it's not needed.
    
    Discussion: https://postgr.es/m/20200221202212.ssb2qpmdgrnx52sj%40alap3.anarazel.de
    c954d490
nodeAgg.c 118 KB