On Thu, 7 Apr 2022 at 15:41, Zhihong Yu <z...@yugabyte.com> wrote: > + * We must keep the original qual in place if there is a > + * PARTITION BY clause as the top-level WindowAgg remains in > + * pass-through mode and does nothing to filter out unwanted > + * tuples. > + */ > + *keep_original = false; > > The comment talks about keeping original qual but the assignment uses the > value false. > Maybe the comment can be rephrased so that it matches the assignment.
Thanks. I've just removed that comment locally now. You're right, it was out of date. David