Re: [PATCHES] Patch for testing query modes on pgbench

2008-03-18 Thread Tatsuo Ishii
Patches applied. Thanks.
--
Tatsuo Ishii
SRA OSS, Inc. Japan

> Tatsuo Ishii <[EMAIL PROTECTED]> wrote:
> 
> > The cause of this is obvious and easy to fix. However I wonder this is
> > because of accidental old or wrong patches. Can you resubmit new patches
> > against CVS Head?
> 
> Thanks for your review. Sorry, it's a wrong patch.
> I found another bugs in handling external scripts and ':'
> at cast operators '::'. I'll send new one.
> 
> Regards,
> ---
> ITAGAKI Takahiro
> NTT Open Source Software Center


-- 
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches


Re: [PATCHES] Patch for testing query modes on pgbench

2008-03-18 Thread ITAGAKI Takahiro

Tatsuo Ishii <[EMAIL PROTECTED]> wrote:

> The cause of this is obvious and easy to fix. However I wonder this is
> because of accidental old or wrong patches. Can you resubmit new patches
> against CVS Head?

Thanks for your review. Sorry, it's a wrong patch.
I found another bugs in handling external scripts and ':'
at cast operators '::'. I'll send new one.

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center




pgbench_querymode-3.patch
Description: Binary data

-- 
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches


Re: [PATCHES] Patch for testing query modes on pgbench

2008-03-18 Thread Tatsuo Ishii
> Here is a updated patch to add query modes into pgbench in order to
> measure performance of simple protocol, extended protocol and
> prepared statements with it.
> 
> [PATCHES] Patch for testing query modes on pgbench
> http://momjian.us/mhonarc/patches/msg00335.html
> 
> A new runtime option (-M) is added:
>   -M querymode
>Choose the query mode from the follows. default is simple.
>- simple: using simple query protocol.
>- extended: using extended protocol.
>- prepared: using extended protocol with prepared statements.
> 
> 
> Extended protocol is slower than simple protocol. I think it is a problem
> because we have to use the protocol (and cannot use prepared statements)
> in some situations, such as connectiong with JDBC to partitioned tables.
> 
> $ pgbench -S -n -c5 -t1 -M simple
> tps = 3567.351598 (excluding connections establishing)
> $ pgbench -S -n -c5 -t1 -M extended
> tps = 2877.697842 (excluding connections establishing)
> $ pgbench -S -n -c5 -t1 -M prepared
> tps = 4532.680627 (excluding connections establishing)
> 
> Regards,
> ---
> ITAGAKI Takahiro
> NTT Open Source Software Center

I got following:

gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wendif-labels 
-fno-strict-aliasing -I../../src/interfaces/libpq -I. -I../../src/include 
-D_GNU_SOURCE   -c -o pgbench.o pgbench.c
pgbench.c: In function `parseQuery':
pgbench.c:1105: error: `var' undeclared (first use in this function)
pgbench.c:1105: error: (Each undeclared identifier is reported only once
pgbench.c:1105: error: for each function it appears in.)
pgbench.c:1092: warning: unused variable `hvar'

(note that the line number might be different what it should be in
your work file since I have applied Yoshiyuki's patches in prior).

The cause of this is obvious and easy to fix. However I wonder this is
because of accidental old or wrong patches. Can you resubmit new patches
against CVS Head?
--
Tatsuo Ishii
SRA OSS, Inc. Japan

-- 
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches


[PATCHES] Patch for testing query modes on pgbench

2008-03-18 Thread ITAGAKI Takahiro
Here is a updated patch to add query modes into pgbench in order to
measure performance of simple protocol, extended protocol and
prepared statements with it.

[PATCHES] Patch for testing query modes on pgbench
http://momjian.us/mhonarc/patches/msg00335.html

A new runtime option (-M) is added:
  -M querymode
   Choose the query mode from the follows. default is simple.
   - simple: using simple query protocol.
   - extended: using extended protocol.
   - prepared: using extended protocol with prepared statements.


Extended protocol is slower than simple protocol. I think it is a problem
because we have to use the protocol (and cannot use prepared statements)
in some situations, such as connectiong with JDBC to partitioned tables.

$ pgbench -S -n -c5 -t1 -M simple
tps = 3567.351598 (excluding connections establishing)
$ pgbench -S -n -c5 -t1 -M extended
tps = 2877.697842 (excluding connections establishing)
$ pgbench -S -n -c5 -t1 -M prepared
tps = 4532.680627 (excluding connections establishing)

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center



pgbench_querymode.patch
Description: Binary data

-- 
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches


Re: [PATCHES] Patch for testing query modes on pgbench

2007-11-05 Thread Bruce Momjian

This has been saved for the 8.4 release:

http://momjian.postgresql.org/cgi-bin/pgpatches_hold

---


ITAGAKI Takahiro wrote:
> Here is a patch for testing query modes on pgbench. It is discussed on
> http://archives.postgresql.org/pgsql-hackers/2007-10/msg00755.php
> 
> I think this is useful to measure performance of protocols and
> prepared statements. I hope the extended protocol will be improved
> as fast as the simple protocol. It is 10-20% slower now.
> (for 8.4 item)
> 
> The querymode option is added:
>   -M querymode
>   Choose the query mode from the follows:
>   simple: using simple query protocol.
>   extended: using extended protocol.
>   prepared: using extended protocol with prepared statements.
>   default is simple.
> 
> Usage:
> $ pgbench -M [simple|extended|prepared]
> 
> Regards,
> ---
> ITAGAKI Takahiro
> NTT Open Source Software Center
> 

[ Attachment, skipping... ]

> 
> ---(end of broadcast)---
> TIP 4: Have you searched our list archives?
> 
>http://archives.postgresql.org

-- 
  Bruce Momjian  <[EMAIL PROTECTED]>http://momjian.us
  EnterpriseDB http://postgres.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

---(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


[PATCHES] Patch for testing query modes on pgbench

2007-10-16 Thread ITAGAKI Takahiro
Here is a patch for testing query modes on pgbench. It is discussed on
http://archives.postgresql.org/pgsql-hackers/2007-10/msg00755.php

I think this is useful to measure performance of protocols and
prepared statements. I hope the extended protocol will be improved
as fast as the simple protocol. It is 10-20% slower now.
(for 8.4 item)

The querymode option is added:
-M querymode
Choose the query mode from the follows:
simple: using simple query protocol.
extended: using extended protocol.
prepared: using extended protocol with prepared statements.
default is simple.

Usage:
$ pgbench -M [simple|extended|prepared]

Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center



pgbench_querymode.patch
Description: Binary data

---(end of broadcast)---
TIP 4: Have you searched our list archives?

   http://archives.postgresql.org