On Fri, Jan 20, 2017 at 12:59 AM, Robert Haas <robertmh...@gmail.com> wrote: > On Thu, Jan 19, 2017 at 4:26 AM, Amit Kapila <amit.kapil...@gmail.com> wrote: >> Fixed. > > > If all of that were no issue, the considerations in > TargetListSupportsBackwardScan could be a problem, too. But I think > there shouldn't be any issue having Gather just continue to return > false. >
You are right. I have added that code under the assumption that if the underlying node (in this case index scan) can support backward scan, gather can also support. I forgot/missed that ExecSupportsBackwardScan is to support cursors operations. Will fix in next version of patch. -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
