While reviewing [1] it occurred to me that it might not be clean that postgresGetForeignUpperPaths() adds ORDER BY to the remote query when it receives stage=UPPERREL_GROUP_AGG. Shouldn't that only happen for UPPERREL_ORDERED? Thus the processing of UPPERREL_ORDERED would have to be able to handle grouping too, so some refactoring would be needed. Do I misunderstand anything?
[1] https://commitfest.postgresql.org/22/1950/ -- Antonin Houska Cybertec Schönig & Schönig GmbH Gröhrmühlgasse 26, A-2700 Wiener Neustadt Web: https://www.cybertec-postgresql.com
