postgres_fdw: don't push ORDER BY with no vars (bug #15352)
authorAndrew Gierth <rhodiumtoad@postgresql.org>
Tue, 28 Aug 2018 13:43:51 +0000 (14:43 +0100)
committerAndrew Gierth <rhodiumtoad@postgresql.org>
Tue, 28 Aug 2018 13:43:51 +0000 (14:43 +0100)
commitbf2d0462cd735f76bcf6eb8b399723674b2221ef
tree9eb6b83c2acf81b9a2c6d8db7cabbd165476bfa2
parentbfea331a5e1b993d22071fc1696e6e8811d2d0d4
postgres_fdw: don't push ORDER BY with no vars (bug #15352)

Commit aa09cd242 changed a condition in find_em_expr_for_rel from
being a bms_equal comparison of relids to bms_is_subset, in order to
support order by clauses on foreign joins. But this also allows
through the degenerate case of expressions with no Vars at all (and
hence empty relids), including integer constants which will be parsed
unexpectedly on the remote (viz. "ERROR: ORDER BY position 0 is not in
select list" as in the bug report).

Repair by adding an additional !bms_is_empty test.

Backpatch through to 9.6 where the aforementioned change was made.

Per bug #15352 from Maksym Boguk; analysis and patch by me.

Discussion: https://postgr.es/m/153518420278.1478.14875560810251994661@wrigleys.postgresql.org
contrib/postgres_fdw/postgres_fdw.c