Re: Added prosupport function for estimating numeric generate_series rows

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: songjinzhou <tsinghualucky912(at)foxmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, japinli <japinli(at)hotmail(dot)com>
Subject: Re: Added prosupport function for estimating numeric generate_series rows
Date: 2024-11-29 12:01:28
Message-ID: CAApHDvqn+M422q8GyJ+e_gcyxPNuNz_CvCJFM0Fxo=eThjO-Xw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, 29 Nov 2024 at 18:50, songjinzhou <tsinghualucky912(at)foxmail(dot)com> wrote:
> Hello, thank you and David Rowley for your comments.
>
> I have used pgindent to adjust the code format and added comments and missing regression test cases. Here is the patch of version v3.

It looks fine to me. The only things I'd adjust are stylistic,
namely; 1) remove two tabs before the goto label, 2) remove redundant
braces around the goto cleanup, 3) rename the variable "q" to
something slightly more meaningful, maybe "res" or "rows".

I'll defer to Dean.

David

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nisha Moond 2024-11-29 12:36:02 Re: Introduce XID age and inactive timeout based replication slot invalidation
Previous Message Michail Nikolaev 2024-11-29 11:55:00 Re: Strange assertion in procarray.c