On Thu, Feb 1, 2024 at 5:03 PM David Rowley <dgrowle...@gmail.com> wrote:
> On Thu, 1 Feb 2024 at 19:36, Richard Guo <guofengli...@gmail.com> wrote: > > maybe we can add an Assert or a comment clarifying that the pathlist of > > partially_grouped_rel cannot be empty here. > > There'd be no need to Assert that as set_cheapest() will raise an > error when given a rel with an empty pathlist. > > I don't think putting an Assert that would fail in the same situation > that we'd later ERROR out on would be a good idea. It'd make it harder > to debug the issue. Fair point. Thanks Richard