On Mon, 19 Oct 2020 at 13:06, Tom Lane <t...@sss.pgh.pa.us> wrote: > > David Rowley <dgrowle...@gmail.com> writes: > > I guess we could resolve that concern by just changing the failing > > assert to become: Assert(outer_skip_rows <= outer_rows || > > isinf(outer_rows)); > > I can't really object to just weakening the Assert a tad. > My thoughts would have run towards checking for the NaN though.
I ended up back-patching a change that does that. Thanks for your input on this and for the report, Onder. David