Skip site navigation (1) Skip section navigation (2)

Re: Review of: explain / allow collecting row counts without timing info

From: Tomas Vondra <tv(at)fuzzy(dot)cz>
To: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Review of: explain / allow collecting row counts without timing info
Date: 2012-01-13 23:07:02
Message-ID: 4F10B916.2010602@fuzzy.cz (view raw or flat)
Thread:
Lists: pgsql-hackers
On 13.1.2012 18:07, Josh Berkus wrote:
> Eric's review follows:
> 
> Compiling on Ubuntu 10.04 LTS AMD64 on a GoGrid virtual machine from
> 2012-01-12 git checkout.
> 
> Patch applied fine.
> 
> 'make check' results in failures when this patch is put into place.
> 
> ========================
>  6 of 128 tests failed.
> ========================
> 
> Here are the relevant failures.
> 
> parallel group (2 tests):  create_view create_index
>      create_index             ... FAILED
> parallel group (9 tests):  create_cast create_aggregate drop_if_exists
> typed_table vacuum constraints create_table_like triggers inherit
>      inherit                  ... FAILED
> parallel group (20 tests):  select_distinct_on btree_index update random
> select_distinct select_having namespace delete case hash_index union
> select_implicit select_into transactions portals subselect arrays
> aggregates join prepared_xacts
>      join                     ... FAILED
>      aggregates               ... FAILED
> parallel group (15 tests):  combocid portals_p2 advisory_lock xmlmap
> tsdicts guc functional_deps dependency select_views cluster tsearch
> window foreign_data foreign_key bitmapops
>      foreign_data             ... FAILED
> parallel group (19 tests):  limit prepare copy2 conversion xml plancache
> returning temp sequence without_oid with rowtypes truncate polymorphism
> domain largeobject rangefuncs alter_table plpgsql
>      alter_table              ... FAILED
> 

Fixed. The default value of TIMING option did not work as intended, it
was set to true even for plain EXPLAIN (without ANALYZE). In that case
the EXPLAIN failed.

Tomas

Attachment: explain-analyze-rows-only-v3.diff
Description: text/plain (9.4 KB)

In response to

Responses

pgsql-hackers by date

Next:From: Andrew DunstanDate: 2012-01-13 23:07:43
Subject: Re: [COMMITTERS] pgsql: Fix breakage from earlier plperl fix.
Previous:From: Dimitri FontaineDate: 2012-01-13 22:53:32
Subject: Re: Command Triggers

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group