On Fri, Jan 20, 2017 at 12:59 AM, Robert Haas <[email protected]> wrote: > On Thu, Jan 19, 2017 at 4:26 AM, Amit Kapila <[email protected]> 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 ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
