From 76ceb5da249fe6023fbccdbd9f46debbad6b23d9 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 12 Mar 2008 02:18:33 +0000 Subject: [PATCH] Add a comment explaining one of the ways that pgbench fails to fully comply with TPC-B. Per Itagaki Takahiro and discussion of bug#3681. --- contrib/pgbench/pgbench.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/contrib/pgbench/pgbench.c b/contrib/pgbench/pgbench.c index 70daa3ec07..e5b04e95a2 100644 --- a/contrib/pgbench/pgbench.c +++ b/contrib/pgbench/pgbench.c @@ -811,6 +811,16 @@ init(void) { PGconn *con; PGresult *res; + /* + * 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. + */ static char *DDLs[] = { "drop table if exists branches", "create table branches(bid int not null,bbalance int,filler char(88)) with (fillfactor=%d)", -- 2.39.5