Re: misleading comments in pgbench - Mailing list pgsql-hackers

From Tatsuo Ishii
Subject Re: misleading comments in pgbench
Date
Msg-id 20091026.092605.96918067.t-ishii@sraoss.co.jp
Whole thread Raw
In response to misleading comments in pgbench  (Jeff Janes <jeff.janes@gmail.com>)
List pgsql-hackers
>      * Note: TPC-B requires at least 100 bytes per row, and the "filler"
>      * fields in these table declarations were intended to comply with that.
>      * But because they default to NULLs, they don't actually take any
>      * space.  We could fix that by giving them non-null default values.
>      * However, that would completely break comparability of pgbench
>      * results with prior versions.  Since pgbench has never pretended
>      * to be fully TPC-B compliant anyway, we stick with the historical
>      * behavior.
> 
> The statement about NULLs applies to the other 3 tables, but
> pgbench_accounts.filler is loaded (via COPY) as empty strings, not as
> NULLs.  When stored into char(84), the empty string takes the full
> specified storage.  So on my system, rows in pgbench_accounts take up
> about 130 bytes, rather than than the about 40 they would with a NULL.

Ok, what about this?
    * Note: TPC-B requires at least 100 bytes per row, and the    * "filler" fields in these table declarations were
intendedto    * comply with that.  But because they default to NULLs, they    * don't actually take any space(except
pgbench_accounts   * table. It's filled with spaces by pgbench -i command).  We    * could fix that by giving them
non-nulldefault values.    * However, that would completely break comparability of pgbench    * results with prior
versions. Since pgbench has never pretended    * to be fully TPC-B compliant anyway, we stick with the    * historical
behavior.
--
Tatsuo Ishii
SRA OSS, Inc. Japan


pgsql-hackers by date:

Previous
From: Roger Leigh
Date:
Subject: Re: Unicode UTF-8 table formatting for psql text output
Next
From: Robert Haas
Date:
Subject: Re: Parsing config files in a directory