Re: [PATCHES] Load distributed checkpoint V3

2007-04-06 Thread Takayuki Tsunakawa
From: "Greg Smith" <[EMAIL PROTECTED]> > If you compare how Oracle handles their writes and checkpoints to the > Postgres code, it's obvious they have a different architecture that > enables them to support sync writing usefully. I'd recommend the Database > Writer Process section of > http://www.

Re: [PATCHES] Packed Varlena Update (v21)

2007-04-06 Thread Gregory Stark
"Tom Lane" <[EMAIL PROTECTED]> writes: > I wrote: >> stark <[EMAIL PROTECTED]> writes: >>> [ packed varlena patch ] > >> Applied with revisions. > > Forgot to mention: one of the revisions was to not add the "sizes.sql" > test, because the output was platform-dependent and is likely to get > more

Re: [PATCHES] Load distributed checkpoint V3

2007-04-06 Thread Simon Riggs
On Fri, 2007-04-06 at 02:53 -0400, Greg Smith wrote: > If you compare how Oracle handles their writes and checkpoints to the > Postgres code, it's obvious they have a different architecture that > enables them to support sync writing usefully. I'd recommend the > Database > Writer Process secti

[PATCHES] Fix for large file support

2007-04-06 Thread Zdenek Kotala
Current version of postgres support only 1GB chunks. This limit is defined in the pg_config_manual.h header file. However this setting allows to have maximal 2GB chunks. Main problem is that md storage manager and buffile use "long" data type (32bits) for offset instead "off_t" defined in . o

Re: [PATCHES] Optimized pgbench for 8.3

2007-04-06 Thread Tatsuo Ishii
Patch committed. Thanks. -- Tatsuo Ishii SRA OSS, Inc. Japan > The attached is a patch to optimize contrib/pgbench using new 8.3 features. > > - Use DROP IF EXISTS to suppress errors for initial loadings. > - Use a combination of TRUNCATE and COPY to reduce WAL on creating > the accounts table.

Re: [PATCHES] Fix for large file support

2007-04-06 Thread Andrew Dunstan
Zdenek Kotala wrote: Current version of postgres support only 1GB chunks. This limit is defined in the pg_config_manual.h header file. However this setting allows to have maximal 2GB chunks. Main problem is that md storage manager and buffile use "long" data type (32bits) for offset instead

Re: [PATCHES] Fix for large file support

2007-04-06 Thread Zdenek Kotala
Andrew Dunstan wrote: Does it mean the maximum field size will grow beyond 1Gb? No. Because it is limited by varlena size. See http://www.postgresql.org/docs/8.2/interactive/storage-toast.html Or give better performance? Yes. List of chunks is stored as linked list and for some operati

Re: [PATCHES] Packed Varlena Update (v21)

2007-04-06 Thread Tom Lane
Gregory Stark <[EMAIL PROTECTED]> writes: > "Tom Lane" <[EMAIL PROTECTED]> writes: >> Forgot to mention: one of the revisions was to not add the "sizes.sql" >> test, because the output was platform-dependent and is likely to get >> more so if any ability to change the toast thresholds gets put in.

Re: [PATCHES] Fix for large file support

2007-04-06 Thread Tom Lane
Zdenek Kotala <[EMAIL PROTECTED]> writes: > Andrew Dunstan wrote: >> Or give better performance? > Yes. List of chunks is stored as linked list and for some operation > (e.g. expand) are all chunks opened and their size is checked. On big > tables it takes some time. For example if you have 1TB

Re: [PATCHES] Fix for large file support

2007-04-06 Thread Zdenek Kotala
Tom Lane wrote: Zdenek Kotala <[EMAIL PROTECTED]> writes: Andrew Dunstan wrote: Indeed, but that would be far more effectively addressed by fixing the *other* code path that doesn't segment at all (the LET_OS_MANAGE_FILESIZE option, which is most likely broken these days for lack of testing).

Re: [PATCHES] Load distributed checkpoint V3

2007-04-06 Thread Greg Smith
On Fri, 6 Apr 2007, Takayuki Tsunakawa wrote: Hmm... what makes you think that sync writes is useful for Oracle and not for PostgreSQL? They do more to push checkpoint-time work in advance, batch writes up more efficiently, and never let clients do the writing. All of which make for a diffe

Re: [PATCHES] index support is NULL

2007-04-06 Thread Tom Lane
Teodor Sigaev <[EMAIL PROTECTED]> writes: > http://www.sigaev.ru/misc/indexnulls-0.8.gz Applied with revisions (except I didn't touch the gist code, figuring you probably understand that better than me). regards, tom lane ---(end of broadcast)-

[PATCHES] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread David Fetter
Folks, Per a question Alexey Parshin asked in the IRC channel, I'm attaching a patch to the GRANT and REVOKE syntax summaries which replaces the misleading word "column" with "parameter." "Column" is misleading because it could be read to imply a column-level GRANT/REVOKE, which we don't have yet

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Tom Lane
David Fetter <[EMAIL PROTECTED]> writes: > Per a question Alexey Parshin asked in the IRC channel, I'm attaching > a patch to the GRANT and REVOKE syntax summaries which replaces the > misleading word "column" with "parameter." "Column" is misleading > because it could be read to imply a column-le

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Russell Smith
Tom Lane wrote: David Fetter <[EMAIL PROTECTED]> writes: Per a question Alexey Parshin asked in the IRC channel, I'm attaching a patch to the GRANT and REVOKE syntax summaries which replaces the misleading word "column" with "parameter." "Column" is misleading because it could be read to imp

Re: [PATCHES] [HACKERS] [Fwd: Index Advisor]

2007-04-06 Thread Tom Lane
"Gurjeet Singh" <[EMAIL PROTECTED]> writes: > Please find attached the latest version of the patch. It applies cleanly on > REL8_2_STABLE. The interface to the planner in this seems rather brute-force. To run a plan involving a hypothetical index, you have to make a bunch of catalog entries, run

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Tom Lane
Russell Smith <[EMAIL PROTECTED]> writes: > Tom Lane wrote: >> The entire *point* of that paragraph is that we don't have the >> feature. This proposed change is surely not an improvement... >> > Maybe removing the entire example would be more helpful. I don't find > it clear to have a command

Re: [PATCHES] Optimized pgbench for 8.3

2007-04-06 Thread Bruce Momjian
FYI, patch applied by Tatsuo. Thanks. --- ITAGAKI Takahiro wrote: > The attached is a patch to optimize contrib/pgbench using new 8.3 features. > > - Use DROP IF EXISTS to suppress errors for initial loadings. > - Use a co

Re: [PATCHES] pgbench transaction timestamps

2007-04-06 Thread Bruce Momjian
FYI, patch applied by Tatsuo. Thanks. --- Greg Smith wrote: > This patch changes the way pgbench outputs its latency log files so that > every transaction gets a timestamp and notes which transaction type was > executed.

[PATCHES] Correct the spelling of SYMMETRIC

2007-04-06 Thread Michael Fuhr
Correct the spelling of SYMMETRIC. -- Michael Fuhr Index: doc/src/sgml/func.sgml === RCS file: /projects/cvsroot/pgsql/doc/src/sgml/func.sgml,v retrieving revision 1.375 diff -c -r1.375 func.sgml *** doc/src/sgml/func.sgml 5 Apr 2007

Re: [PATCHES] Correct the spelling of SYMMETRIC

2007-04-06 Thread Bruce Momjian
Patch applied. Thanks. Your documentation changes can be viewed in five minutes using links on the developer's page, http://www.postgresql.org/developer/testing. --- Michael Fuhr wrote: > Correct the spelling of SYMMETRI

Re: [PATCHES] HOT Patch - Ready for review

2007-04-06 Thread Bruce Momjian
Your patch has been added to the PostgreSQL unapplied patches list at: http://momjian.postgresql.org/cgi-bin/pgpatches It will be applied as soon as one of the PostgreSQL committers reviews and approves it. --- Pa

Re: [PATCHES] [DOCS] Fix misleading references to columns in GRANT/REVOKE summaries

2007-04-06 Thread Bruce Momjian
Tom Lane wrote: > Russell Smith <[EMAIL PROTECTED]> writes: > > Tom Lane wrote: > >> The entire *point* of that paragraph is that we don't have the > >> feature. This proposed change is surely not an improvement... > >> > > Maybe removing the entire example would be more helpful. I don't find >

Re: [PATCHES] UTF8MatchText

2007-04-06 Thread Bruce Momjian
I do not understand this patch. You have defined two functions, UTF8MatchText() and UTF8MatchTextIC(), and the difference between them is that one calls CHAREQ and the other calls ICHAREQ, but just above those two functions you define the macros identically: #define CHAREQ(p1, p2)wch

Re: [PATCHES] Concurrent psql v4 [WIP]

2007-04-06 Thread Bruce Momjian
Please remove \cwait and supply an updated version with documentation. --- stark wrote: > > This is just an update against CVS. > > The interface is still as described at this URL: > > http://community.enterprisedb.com/co

Re: [PATCHES] UTF8MatchText

2007-04-06 Thread Bruce Momjian
Bruce Momjian wrote: > > I do not understand this patch. You have defined two functions, > UTF8MatchText() and UTF8MatchTextIC(), and the difference between them > is that one calls CHAREQ and the other calls ICHAREQ, but just above > those two functions you define the macros identically: > >