• Tom Lane's avatar
    Add a regression test case to improve code coverage for tuplesort. · cec55013
    Tom Lane authored
    Test the external-sort code path in CLUSTER for two different scenarios:
    multiple-pass external sorting, and the best case for replacement
    selection, where only one run is produced, so that no merge is required.
    This test would have caught the bug fixed in commit 1b0fc850, at
    least when run with valgrind enabled.
    
    In passing, add a short-circuit test in plan_cluster_use_sort() to make
    dead certain that it selects sorting when enable_indexscan is off.  As
    things stand, that would happen anyway, but it seems like good future
    proofing for this test.
    
    Peter Geoghegan
    
    Discussion: <CAM3SWZSgxehDkDMq1FdiW2A0Dxc79wH0hz1x-TnGy=1BXEL+nw@mail.gmail.com>
    cec55013
planner.c 165 KB