On 21 March 2016 at 15:48, Alvaro Herrera <alvhe...@2ndquadrant.com> wrote: > David Rowley wrote: > >> I've rewritten the comment to become: >> >> /* >> * Providing that the estimated size of the hashtable does not exceed >> * work_mem, we'll generate a HashAgg Path, although if we were unable >> * to sort above, then we'd better generate a Path, so that we at least >> * have one. >> */ >> >> How about that? > > I think "Providing" should be "Provided".
Both make sense, although I do only see instances of "Provided that" in the source. I'm not opposed to changing it, it just does not seem worth emailing a complete patch to do that, but let me know if you feel differently. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers