On Fri, 24 Mar 2023 22:17:33 +0000
Cary Huang <cary.hu...@highgo.ca> wrote:
> The following review has been posted through the commitfest application:
> make installcheck-world: not tested
> Implements feature: tested, passed
> Spec compliant: not tested
> Documentation: not tested
The patch would need documentations describing the new options.
>
> Hi
>
> thank you for the patch. It can be applied to current master branch and
> compiled fine.
I also confirmed that it can be applied and complied, although it raised
warnings
about whitespace errors.
/tmp/pgbench-plpgsql-001.patch:68: trailing whitespace.
executeStatement(con,
/tmp/pgbench-plpgsql-001.patch:102: trailing whitespace.
executeStatement(con,
warning: 2 lines add whitespace errors.
> The feature works as described, I am able to run plpgsql-tpcb-like and
> plpgsql-simple-update scripts as you have added them.
>
> But I am not sure the purpose of --no-function to prevent the creation of
> pl/pgsql functions when the new plpgsql test scripts need them.
>
> I initialized pgbench database with --no-function, and plpgsql-tpcb-like and
> plpgsql-simple-update scripts will fail to run
I am not sure either whether --no-function option is necessary.
Although there is --no-vacuum, I guess this would be intended to
reduce the initialization time. I don't think omitting creating
functions has such effect. So, I wonder -no-function is unnecessary,
as similar to that there are no option to omitting to create tables.
Regards,
Yugo Nagata
--
Yugo NAGATA <nag...@sraoss.co.jp>
