• Tom Lane's avatar
    Rethink the GetForeignUpperPaths API (again). · 9e703987
    Tom Lane authored
    In the previous design, the GetForeignUpperPaths FDW callback hook was
    called before we got around to labeling upper relations with the proper
    consider_parallel flag; this meant that any upper paths created by an FDW
    would be marked not-parallel-safe.  While that's probably just as well
    right now, we aren't going to want it to be true forever.  Hence, abandon
    the idea that FDWs should be allowed to inject upper paths before the core
    code has gotten around to creating the relevant upper relation.  (Well,
    actually they still can, but it's on their own heads how well it works.)
    Instead, adopt the same API already designed for create_upper_paths_hook:
    we call GetForeignUpperPaths after each upperrel has been created and
    populated with the paths the core planner knows how to make.
    9e703987
fdwapi.h 8.43 KB