Обсуждение: Bug in intarray bench script

Поиск
Список
Период
Сортировка

Bug in intarray bench script

От
Daniel Gustafsson
Дата:
In the contrib/intarray benchmarking script bench.pl, the -e option to print
the plan via EXPLAIN is using the DBI do() method which discards output
resulting in nothing being printed.  Judging by the usage help (“show explain”)
I assume the intention is to print the plan to STDOUT when invoked (which is
exactly what I wanted in this case, thus bumping into this) but at least on
recent DBI/DBD::Pg versions the do() method is not returning the rows.

Attached patch use the selectall_arrayref() method instead and output the plan
when -e is invoked.

cheers ./daniel


Вложения

Re: Bug in intarray bench script

От
Andreas 'ads' Scherbaum
Дата:
The following review has been posted through the commitfest application:
make installcheck-world:  not tested
Implements feature:       not tested
Spec compliant:           not tested
Documentation:            not tested

The patch changes the benchmark tool in a way that the explain output is printed to standard out - what one would
expectfrom the "-e" (explain) option. 

The new status of this patch is: Ready for Committer

Re: Bug in intarray bench script

От
Tom Lane
Дата:
"Andreas 'ads' Scherbaum" <ads@pgug.de> writes:
> The patch changes the benchmark tool in a way that the explain output is printed to standard out - what one would
expectfrom the "-e" (explain) option.
 

> The new status of this patch is: Ready for Committer

Pushed, thanks.
        regards, tom lane