Re: [HACKERS] pgbench enhancements

2006-07-26 Thread Tatsuo Ishii
> Tatsuo Ishii <[EMAIL PROTECTED]> writes:
> > BTW, running long benchmark using pgbench on BIG tables easily causes
> > an integer overflow error in following SQLs:
> 
> Right.
> 
> > I'm inclined to change abalance, tbalance and bbalance column to
> > BIGINT to avoid the error. Opinion?
> 
> No.  The problem is that the deltas are invariably positive, which is
> not realistic (at least *my* bank balance isn't uniformly increasing :-().
> I think the correct fix is just to tweak the range of the randomly
> distributed deltas to be plus and minus not always plus.

I have never thought about it. Seems nice idea. Thanks.

> If you change to bigint then post-change results won't be strictly
> comparable to pre-change results because of the difference in execution
> costs.

Yes, that was my concerning too.
--
Tatsuo Ishii
SRA OSS, Inc. Japan

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


Re: [HACKERS] pgbench enhancements

2006-07-26 Thread andrew
>> > Hi,
>> >
>> > I have committed contrib/pgbench enhanments contributed by Tomoaki
>> > Sato from SRA OSS, Inc. Japan.
>> >
>> > - predefined variable "tps"
>> >   The value of variable tps is taken from the scaling factor
>> >   specified by -s option.
>> >
>> > - -D option
>> >   Variable values can be defined by -D option.
>> >
>> > - \set command now allows arithmetic calculations.
>> >
>>
>>
>> Was there a previous patch posted for this? I don't recall seeing it.
>
> Are you saying that I should have posted patches and called for
> discussion? I omit the step because a) pgbench is a contrib program,
> b) the changes do not break the backward compatibility.
>
> I always call for discussions for questionable part. (for example,
> column type change proposal for pgbench tables).

It's not a big deal, but I was under the impression that even in these
circumstances a patch should be posted first, especially if it changes
user visible behaviour.

cheers

andrew


---(end of broadcast)---
TIP 5: don't forget to increase your free space map settings


Re: [HACKERS] pgbench enhancements

2006-07-26 Thread Tatsuo Ishii
> > Hi,
> >
> > I have committed contrib/pgbench enhanments contributed by Tomoaki
> > Sato from SRA OSS, Inc. Japan.
> >
> > - predefined variable "tps"
> >   The value of variable tps is taken from the scaling factor
> >   specified by -s option.
> >
> > - -D option
> >   Variable values can be defined by -D option.
> >
> > - \set command now allows arithmetic calculations.
> >
> 
> 
> Was there a previous patch posted for this? I don't recall seeing it.

Are you saying that I should have posted patches and called for
discussion? I omit the step because a) pgbench is a contrib program, 
b) the changes do not break the backward compatibility.

I always call for discussions for questionable part. (for example,
column type change proposal for pgbench tables).
--
Tatsuo Ishii
SRA OSS, Inc. Japan

---(end of broadcast)---
TIP 1: if posting/reading through Usenet, please send an appropriate
   subscribe-nomail command to [EMAIL PROTECTED] so that your
   message can get through to the mailing list cleanly


Re: [HACKERS] pgbench enhancements

2006-07-26 Thread andrew
> Hi,
>
> I have committed contrib/pgbench enhanments contributed by Tomoaki
> Sato from SRA OSS, Inc. Japan.
>
> - predefined variable "tps"
>   The value of variable tps is taken from the scaling factor
>   specified by -s option.
>
> - -D option
>   Variable values can be defined by -D option.
>
> - \set command now allows arithmetic calculations.
>


Was there a previous patch posted for this? I don't recall seeing it.

cheers

andrew


---(end of broadcast)---
TIP 2: Don't 'kill -9' the postmaster


Re: [HACKERS] pgbench enhancements

2006-07-26 Thread Tom Lane
Tatsuo Ishii <[EMAIL PROTECTED]> writes:
> BTW, running long benchmark using pgbench on BIG tables easily causes
> an integer overflow error in following SQLs:

Right.

> I'm inclined to change abalance, tbalance and bbalance column to
> BIGINT to avoid the error. Opinion?

No.  The problem is that the deltas are invariably positive, which is
not realistic (at least *my* bank balance isn't uniformly increasing :-().
I think the correct fix is just to tweak the range of the randomly
distributed deltas to be plus and minus not always plus.

If you change to bigint then post-change results won't be strictly
comparable to pre-change results because of the difference in execution
costs.

regards, tom lane

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


Re: [HACKERS] pgbench enhancements

2006-07-26 Thread Tatsuo Ishii
> I have committed contrib/pgbench enhanments contributed by Tomoaki
> Sato from SRA OSS, Inc. Japan.
> 
> - predefined variable "tps"
>   The value of variable tps is taken from the scaling factor
>   specified by -s option.
> 
> - -D option
>   Variable values can be defined by -D option.
> 
> - \set command now allows arithmetic calculations.
> 
> enjoy,

BTW, running long benchmark using pgbench on BIG tables easily causes
an integer overflow error in following SQLs:

update accounts set abalance = abalance + :delta where aid = :aid;

update tellers set tbalance = tbalance + :delta where tid = :tid;

update branches set bbalance = bbalance + :delta where bid = :bid;

I'm inclined to change abalance, tbalance and bbalance column to
BIGINT to avoid the error. Opinion?
--
Tatsuo Ishii
SRA OSS, Inc. Japan

---(end of broadcast)---
TIP 9: In versions below 8.0, the planner will ignore your desire to
   choose an index scan if your joining column's datatypes do not
   match


[HACKERS] pgbench enhancements

2006-07-26 Thread Tatsuo Ishii
Hi,

I have committed contrib/pgbench enhanments contributed by Tomoaki
Sato from SRA OSS, Inc. Japan.

- predefined variable "tps"
  The value of variable tps is taken from the scaling factor
  specified by -s option.

- -D option
  Variable values can be defined by -D option.

- \set command now allows arithmetic calculations.

enjoy,
--
Tatsuo Ishii
SRA OSS, Inc. Japan

---(end of broadcast)---
TIP 3: Have you checked our extensive FAQ?

   http://www.postgresql.org/docs/faq