On Thu, Oct 13, 2022 at 4:41 AM David Rowley <dgrowle...@gmail.com> wrote:
> We could do something like set some bool flag in PlannerInfo to tell > the planner not to bother adding the final LimitPath as we've already > added another which does the job, but is it really worth adding that > complexity for this patch? You already mentioned that "this patch is > very straightforward". I don't think it would be if we added code to > avoid the LimitPath duplication. Yeah, maybe this is the right way to do it. I agree that this would complicate the code. Not sure if it's worth doing. Thanks Richard