On Tue, Feb 6, 2018 at 11:32 AM, Amit Khandekar <amitdkhan...@gmail.com> wrote: > Yeah, I think it looks equally good that way, and like you said, the > current code does it that way. So in the attached patch, I have > swapped the two conditions.
I prefer to avoid introducing 2 new variables and instead just prevent the looping directly in the case where we started with a non-partial plan. See attached. Does this look OK? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
fix-hang-issue-rmh.patch
Description: Binary data