Re: [HACKERS] Streaming replication status

2010-01-11 Thread Craig Ringer
On 9/01/2010 6:20 AM, Josh Berkus wrote: On 1/8/10 1:16 PM, Heikki Linnakangas wrote: * A standby that connects to master, initiates streaming, and then sits idle without stalls recycling of old WAL files in the master. That will eventually lead to a full disk in master. Do we need some kind of

Re: [HACKERS] Testing with concurrent sessions

2010-01-11 Thread Martijn van Oosterhout
On Mon, Jan 11, 2010 at 04:17:42AM -, Greg Sabino Mullane wrote: Because you'd have to build DBD::Pg against the new libpq, as you do psql. That means you need DBD::Pg sources and the build environment for Perl (headers etc) not just a working Perl runtime. Big difference. Yes,

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Magnus Hagander
2010/1/11 Alvaro Herrera alvhe...@commandprompt.com: Dimitri Fontaine wrote: Andrew Dunstan and...@dunslane.net writes: That is assuming that the MUA gives you the option of specifying the attachment MIME type. Many (including mine) do not. It would mean an extra step - I'd have to gzip

Re: [HACKERS] Listen / Notify - what to do when the queue is full

2010-01-11 Thread Arnaud Betremieux
A use case : use NOTIFY in a rule to send the primary key of a row that has been updated (for instance to manage a cache). This requires a patch on top of this one, and it really is a separate concern, but I thought I'd give the use case anyway, since I believe it is relevant to the issues

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dimitri Fontaine
Alvaro Herrera alvhe...@commandprompt.com writes: Absolutely. The month boundary problem boils down to the fact that Mhonarc does not scale very well, so we can't have mboxes that are too large. This is why most people split their archives per month, and then each month is published as an

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Matteo Beccati
Hi, Il 11/01/2010 11:18, Dimitri Fontaine ha scritto: AOX is already a database backed email solution, offering an archive page with searching. I believe the searching is baked by tsearch indexing. That's why I think it'd be suitable. They already archive and offer search over one of our

Re: [HACKERS] quoting psql varible as identifier

2010-01-11 Thread Pavel Stehule
Hello No longer applies, please rebase. fixed, sorry Pavel Thanks, ...Robert variable_escaping-fix.diff Description: Binary data -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription:

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dave Page
On Mon, Jan 11, 2010 at 5:23 PM, Matteo Beccati p...@beccati.com wrote: Hi, Il 11/01/2010 11:18, Dimitri Fontaine ha scritto: AOX is already a database backed email solution, offering an archive page with searching. I believe the searching is baked by tsearch indexing. That's why I think

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Matteo Beccati
Il 11/01/2010 12:58, Dave Page ha scritto: On Mon, Jan 11, 2010 at 5:23 PM, Matteo Beccatip...@beccati.com wrote: I recall having tried AOX a long time ago but I can't remember the reason why I was not satisfied. I guess I can give another try by setting up a test ML archive. I tried it too,

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dimitri Fontaine
Magnus Hagander mag...@hagander.net writes: As for AOX, my understanding is that it is no longer maintained, so I'd be worried about choosing such a solution for a complex problem. But it's open for discussion. Ouch. -- dim -- Sent via pgsql-hackers mailing list

Re: [HACKERS] Listen / Notify - what to do when the queue is full

2010-01-11 Thread Tom Lane
Arnaud Betremieux arnaud.betremi...@keyconsulting.fr writes: 3) My use case : NOTIFY channel 'pay'||'load' (actually NOTIFY channel 'table_name#'||OLD.id) 4) Taken one step further : NOTIFY channel (SELECT payload FROM payloads WHERE ...) I'm working on a proof of concept patch to

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dimitri Fontaine
Dave Page dp...@pgadmin.org writes: I recall having tried AOX a long time ago but I can't remember the reason why I was not satisfied. I guess I can give another try by setting up a test ML archive. I tried it too, before I started writing the new prototype archiver from scratch. I too

Re: [HACKERS] Listen / Notify - what to do when the queue is full

2010-01-11 Thread Andrew Chernow
Arnaud Betremieux wrote: A use case : use NOTIFY in a rule to send the primary key of a row that has been updated (for instance to manage a cache). This requires a patch on top of this one, and it really is a separate concern, but I thought I'd give the use case anyway, since I believe it is

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Magnus Hagander
2010/1/11 Dimitri Fontaine dfonta...@hi-media.com: Dave Page dp...@pgadmin.org writes: I recall having tried AOX a long time ago but I can't remember the reason why I was not satisfied. I guess I can give another try by setting up a test ML archive. I tried it too, before I started writing

Re: [HACKERS] Listen / Notify - what to do when the queue is full

2010-01-11 Thread Merlin Moncure
On Mon, Jan 11, 2010 at 8:25 AM, Tom Lane t...@sss.pgh.pa.us wrote: I'm working on a proof of concept patch to use Joachim's new notify function to introduce case 3. I think this means going through the planner and executor, so I might as well do case 4 as well. It would be a lot less work

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dimitri Fontaine
Dimitri Fontaine dfonta...@hi-media.com writes: Magnus Hagander mag...@hagander.net writes: As for AOX, my understanding is that it is no longer maintained, so I'd be worried about choosing such a solution for a complex problem. But it's open for discussion. Ouch. It seems that the company

Re: [HACKERS] Typed tables

2010-01-11 Thread Merlin Moncure
On Mon, Nov 9, 2009 at 5:15 AM, Peter Eisentraut pete...@gmx.net wrote: On Sun, 2009-11-08 at 21:17 +, Simon Riggs wrote: Not sure I see why this is good. Why is issuing CREATE TYPE so much easier than using CREATE TABLE? Is it worth the extra syntax and code to support it? Can we do

Re: [HACKERS] Listen / Notify - what to do when the queue is full

2010-01-11 Thread Arnaud Betremieux
On 11/01/2010 14:25, Tom Lane wrote: Arnaud Betremieuxarnaud.betremi...@keyconsulting.fr writes: 3) My use case : NOTIFY channel 'pay'||'load' (actually NOTIFY channel 'table_name#'||OLD.id) 4) Taken one step further : NOTIFY channel (SELECT payload FROM payloads WHERE ...)

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Andrew Dunstan
Magnus Hagander wrote: No, the current archiver is a set of MBOX files that are processed incrementally by mhonarc. (yes, this is why it doesn't scale) *search* is in a postgresql database, but it doesn't contain the entire messages - doesn't have attachments for examples - only the parts

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Abhijit Menon-Sen
(Many thanks to Dimitri for bringing this thread to my attention.) At 2010-01-11 10:46:10 +0100, mag...@hagander.net wrote: As for AOX, my understanding is that it is no longer maintained, so I'd be worried about choosing such a solution for a complex problem. I'll keep this short: Oryx, the

[HACKERS] pg_dump sort order for functions

2010-01-11 Thread Peter Eisentraut
pg_dump sorts its output first by object type, then by object name, and then processes all that for dependencies. This works well, but for overloaded functions this still gives a random sort order that can produce annoying diffs in the dump. Would it be acceptable to introduce a secondary sort

[HACKERS] CommitFest 2010-01: Call for Reviewers

2010-01-11 Thread Robert Haas
Well, folks, it's that time again! CommitFest 2010-01 will begin in just a few days (specifically, Fri Jan 15 00:00:00 UTC) and we are in need of reviewers. If you're willing to help out, please drop me an email and I'll assign you a patch; if you have a preference, please let me know that, too.

Re: [HACKERS] [RRR] CommitFest 2010-01: Call for Reviewers

2010-01-11 Thread Pavel Stehule
Hello I could to start with https://commitfest.postgresql.org/action/patch_view?id=251 Regards Pavel 2010/1/11 Robert Haas robertmh...@gmail.com: Well, folks, it's that time again!  CommitFest 2010-01 will begin in just a few days (specifically, Fri Jan 15 00:00:00 UTC) and we are in need of

Re: [HACKERS] [RRR] CommitFest 2010-01: Call for Reviewers

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 10:14 AM, Pavel Stehule pavel.steh...@gmail.com wrote: I could to start with https://commitfest.postgresql.org/action/patch_view?id=251 OK. Please edit that patch and list your name as the reviewer. One thing I forgot to mention is - if you are volunteering to review,

Re: [HACKERS] pg_dump sort order for functions

2010-01-11 Thread Tom Lane
Peter Eisentraut pete...@gmx.net writes: pg_dump sorts its output first by object type, then by object name, and then processes all that for dependencies. This works well, but for overloaded functions this still gives a random sort order that can produce annoying diffs in the dump. Would it

Re: [HACKERS] pg_dump sort order for functions

2010-01-11 Thread Peter Eisentraut
On mån, 2010-01-11 at 10:44 -0500, Tom Lane wrote: Peter Eisentraut pete...@gmx.net writes: pg_dump sorts its output first by object type, then by object name, and then processes all that for dependencies. This works well, but for overloaded functions this still gives a random sort order

Re: [HACKERS] Testing with concurrent sessions

2010-01-11 Thread Greg Sabino Mullane
-BEGIN PGP SIGNED MESSAGE- Hash: RIPEMD160 Is there a reason why you're suggesting using DBI? There is also the Pg perl module which works as well and is one tenth of the size. It also doesn't have external dependancies. It's just a plain wrapper around libpq, which for the purposes

Re: [HACKERS] pg_dump sort order for functions

2010-01-11 Thread Tom Lane
Peter Eisentraut pete...@gmx.net writes: On mån, 2010-01-11 at 10:44 -0500, Tom Lane wrote: I think you could probably use the existing tag field; no need for a new one. Sorry, which tag field are you referring to? The one called tag in the source code. It prints out as Name: -- -- Name:

Re: [HACKERS] point_ops for GiST

2010-01-11 Thread Teodor Sigaev
I have reviewed this patch and it looks good to me. The only substantive question I have is why gist_point_consistent() uses a different coding pattern for the box case than it does for the polygon and circle cases? It's not obvious to me on the face of it why these aren't consistent.

Re: [HACKERS] Red-black tree for GIN

2010-01-11 Thread Teodor Sigaev
...and I would say the same logic applies to this patch, maybe even moreso. Tom has already applied a partial workaround for this problem, and I'm feeling like it won't be trivial to figure out what That partial workaround is not work for some corner cases:

Re: [HACKERS] point_ops for GiST

2010-01-11 Thread Robert Haas
2010/1/11 Teodor Sigaev teo...@sigaev.ru: I have reviewed this patch and it looks good to me.  The only substantive question I have is why gist_point_consistent() uses a different coding pattern for the box case than it does for the polygon and circle cases?  It's not obvious to me on the face

Re: [HACKERS] Red-black tree for GIN

2010-01-11 Thread Robert Haas
2010/1/11 Teodor Sigaev teo...@sigaev.ru: knngist uses that implementation of rb-tree. One more candidate is a ts_stat which now uses unbalanced binary tree. Ah, OK. That's great if we can reuse that code in 2 or 3 places. ...Robert -- Sent via pgsql-hackers mailing list

Re: [HACKERS] Setting oom_adj on linux?

2010-01-11 Thread Tom Lane
Alex Hunsaker bada...@gmail.com writes: On Fri, Jan 8, 2010 at 07:27, Tom Lane t...@sss.pgh.pa.us wrote: Then, somebody who wants the feature would build with, say,        -DLINUX_OOM_ADJ=0 or another value if they want that. Here is a stab at that. Applied with some editorialization.

Re: [HACKERS] Typed tables

2010-01-11 Thread Josh Berkus
Peter, Typed tables is the official SQL standard name for the feature, and it's also used in DB2 documentation. So I kind of would prefer to keep it. Sorry, I missed the SQL standard part in the thread. Ignore the noise. Oh, and BTW, +1 on accepting this, pending patch quality and all

Re: [HACKERS] Add .gitignore files to CVS?

2010-01-11 Thread Aidan Van Dyk
* Magnus Hagander mag...@hagander.net [100109 13:05]: On Sat, Jan 9, 2010 at 18:33, Aidan Van Dyk ai...@highrise.ca wrote: * Magnus Hagander mag...@hagander.net [100109 12:03]: If that's the only remaining obstacle, I'm willing to work up some test scripts around that. But I'm not going

Re: [HACKERS] Typed tables

2010-01-11 Thread Andrew Chernow
Peter Eisentraut wrote: On tor, 2009-11-05 at 19:24 +0200, Peter Eisentraut wrote: I'm planning to work on typed tables support. The idea is that you create a table out of a composite type (as opposed to the other way around, which is currently done automatically). CREATE TYPE persons_type AS

[HACKERS] Feature patch 1 for plperl - open issues

2010-01-11 Thread Tim Bunce
Thanks for all the feedback! I'm going to try to summarize and address the issues raised. (Meanwhile I've started spliting the patch into at least three parts, per Andrew's suggestion: utility functions, the GUCs, the rest.) * Long GUC values I think the underlying assumption that they'll be

[HACKERS] Compression Library and Usages

2010-01-11 Thread Dimitri Fontaine
Hi, The topic came on IRC and it might be that the later attempts at using another library missed one of the offering, namely FastLZ. It's made for being quick rather than minimize size, it's MIT licenced, 551 lines of portable ansi-C code, already tested on a host of systems and compilers.

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Kevin Grittner
Dimitri Fontaine dfonta...@hi-media.com wrote: The topic came on IRC and it might be that the later attempts at using another library missed one of the offering, namely FastLZ. It's made for being quick rather than minimize size, it's MIT licenced, 551 lines of portable ansi-C code, already

[HACKERS] dbt2-shell script problem

2010-01-11 Thread Jonas J
Hi guys, I get the dbt2 project from the git tree. And i'm trying to make it work with postgres. The problem is in one of the dbt2 shell scripts. Since I dont understand of Shell Script Programing, here is my problem: (I will post the script on the end) 1 - I executed: dbt2-pgsql-load-db -d

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Dimitri Fontaine
Kevin Grittner kevin.gritt...@wicourts.gov writes: Are the concerns from previous discussions off-base? http://archives.postgresql.org/pgsql-performance/2009-08/msg00053.php I knew I was forgetting about something, thanks for the reminder. Damn it, patents. -- dim -- Sent via

Re: [HACKERS] Testing with concurrent sessions

2010-01-11 Thread Martijn van Oosterhout
On Mon, Jan 11, 2010 at 05:42:08PM -, Greg Sabino Mullane wrote: Is there a reason why you're suggesting using DBI? There is also the Pg perl module which works as well and is one tenth of the size. It also doesn't have external dependancies. It's just a plain wrapper around libpq,

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Matteo Beccati
Il 11/01/2010 15:00, Abhijit Menon-Sen ha scritto: I'll keep this short: Oryx, the company behind Archiveopteryx (aox), is no longer around, but the software is still maintained. The developers (myself included) are still interested in keeping it alive. It's been a while since the last release,

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Kevin Grittner
Dimitri Fontaine dfonta...@hi-media.com wrote: http://archives.postgresql.org/pgsql-performance/2009-08/msg00053.php I knew I was forgetting about something, thanks for the reminder. Damn it, patents. I'm not sure that there was anything there which absolutely ruled out using LZ0; all of

Re: [HACKERS] Typed tables

2010-01-11 Thread Peter Eisentraut
On mån, 2010-01-11 at 15:02 -0500, Andrew Chernow wrote: ISTM that the ultimate would be a 'create table (_) without storage' (or some'm) and make 'create type' an alternate syntax for SQL conformance. I don't really understand the purpose of that. For various reasons, we've

Re: [HACKERS] dbt2-shell script problem

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 4:18 PM, Jonas J autoram...@gmail.com wrote: I get the dbt2 project from the git tree. And i'm trying to make it work with postgres. The problem is in one of the dbt2 shell scripts. Since I dont understand of Shell Script Programing, here is my problem: (I will post the

Re: [HACKERS] xpath improvement suggestion

2010-01-11 Thread Arie Bikker
Peter Eisentraut wrote: On ons, 2010-01-06 at 23:46 +0100, Arie Bikker wrote: Hope this is the right attachement type (I'm new at this) BTW. here a some nice examples: - Get the number of attributes of the first childnode: select ( xpath('count(@*)',(xpath('*[1]','a b=cd e=f

Re: [HACKERS] Typed tables

2010-01-11 Thread Andrew Chernow
Peter Eisentraut wrote: On mån, 2010-01-11 at 15:02 -0500, Andrew Chernow wrote: ISTM that the ultimate would be a 'create table (_) without storage' (or some'm) and make 'create type' an alternate syntax for SQL conformance. I don't really understand the purpose of that. What is the

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Merlin Moncure
On Mon, Jan 11, 2010 at 5:25 PM, Kevin Grittner kevin.gritt...@wicourts.gov wrote: Dimitri Fontaine dfonta...@hi-media.com wrote: http://archives.postgresql.org/pgsql-performance/2009-08/msg00053.php I knew I was forgetting about something, thanks for the reminder. Damn it, patents. I'm

Re: [HACKERS] damage control mode

2010-01-11 Thread Bruce Momjian
Josh Berkus wrote: I'll also say: if we can't make time-based releases work, we're probably dead as a project. MySQL and Ingres both tried feature-based releases, and look where they are now. That is a simplification. We have always done time-based releases with adjustments for

Re: [HACKERS] damage control mode

2010-01-11 Thread Joshua D. Drake
On Mon, 2010-01-11 at 19:50 -0500, Bruce Momjian wrote: Josh Berkus wrote: I'll also say: if we can't make time-based releases work, we're probably dead as a project. MySQL and Ingres both tried feature-based releases, and look where they are now. That is a simplification. We have

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Tom Lane
Merlin Moncure mmonc...@gmail.com writes: plus, it looks like that most of the patents have either expired, or are about to expire. lzo is used all over the place, including the linux kernel...i think the burden of proof rests with anyone claiming there are patent problems, not the other way

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Joshua D. Drake
On Mon, 2010-01-11 at 19:54 -0500, Tom Lane wrote: Merlin Moncure mmonc...@gmail.com writes: plus, it looks like that most of the patents have either expired, or are about to expire. lzo is used all over the place, including the linux kernel...i think the burden of proof rests with anyone

Re: [HACKERS] damage control mode

2010-01-11 Thread Bruce Momjian
Robert Treat wrote: Now the other approach we could take is that we'll ship *something* on 7 Mar, even if it's less stable than what we've traditionally considered to be beta quality. I don't think that really helps much though; it just means we need more time in beta. There are

Re: [HACKERS] damage control mode

2010-01-11 Thread Tom Lane
Bruce Momjian br...@momjian.us writes: Robert Treat wrote: There are three reasons I'd probably be comfortable with that; 1) the CF process means we've likely had more eyes on the code going in than in past releases. The reality check is that was had commit-fests for 8.4 development and

Re: [HACKERS] damage control mode

2010-01-11 Thread David Fetter
On Mon, Jan 11, 2010 at 07:50:23PM -0500, Bruce Momjian wrote: Josh Berkus wrote: I'll also say: if we can't make time-based releases work, we're probably dead as a project. MySQL and Ingres both tried feature-based releases, and look where they are now. That is a simplification. We

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Bruce Momjian
Simon Riggs wrote: * I don't think we should require superuser rights for replication. Although you see all WAL and potentially all data in the system through that, a standby doesn't need any write access to the master, so it would be good practice to create a dedicated account with

Re: [HACKERS] [COMMITTERS] pgsql: Also update ChangerLog file.

2010-01-11 Thread Bruce Momjian
Tom Lane wrote: Michael Meskes mes...@postgresql.org writes: On Fri, Jan 08, 2010 at 03:41:10PM +0100, Stefan Kaltenbrunner wrote: Hmm not sure I find that commit message really helpful - but is it actually of any use to maintain a Changelog file specifically for ECPG? Makes it easier

Re: [HACKERS] [PATCH] remove redundant ownership checks

2010-01-11 Thread Bruce Momjian
Robert Haas wrote: On Sun, Jan 10, 2010 at 4:54 PM, Tom Lane t...@sss.pgh.pa.us wrote: Robert Haas robertmh...@gmail.com writes: I have looked this over a little bit and I guess I don't see why the lack of a grand plan for how to organize all of our permissions checks ought to keep us

[HACKERS] 8.5alpha3 bug in information_schema.table_privileges

2010-01-11 Thread Robert Treat
pagila=# select * from information_schema.table_privileges; TRAP: FailedAssertion(!(idx[0] (((int *) (((char *) (acl)) + sizeof(ArrayType)))[0])), File: acl.c, Line: 1740) server closed the connection unexpectedly This probably means the server terminated abnormally before or

Re: [HACKERS] damage control mode

2010-01-11 Thread Josh Berkus
They weren't easily identified, or we'd have found them before 8.4.0 release. I think the notion that 8.4.0 was much worse than previous .0 releases is largely bogus, anyway; we've just forgotten all the bugs in older releases ... It was worse than some, and better than others. Bruce's

Re: [HACKERS] 8.5alpha3 bug in information_schema.table_privileges

2010-01-11 Thread Tom Lane
Robert Treat xzi...@users.sourceforge.net writes: pagila=# select * from information_schema.table_privileges; TRAP: FailedAssertion(!(idx[0] (((int *) (((char *) (acl)) + sizeof(ArrayType)))[0])), File: acl.c, Line: 1740) server closed the connection unexpectedly This probably means

Re: [HACKERS] damage control mode

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 8:14 PM, David Fetter da...@fetter.org wrote: On Mon, Jan 11, 2010 at 07:50:23PM -0500, Bruce Momjian wrote: Josh Berkus wrote: I'll also say: if we can't make time-based releases work, we're probably dead as a project.  MySQL and Ingres both tried feature-based

Re: [HACKERS] damage control mode

2010-01-11 Thread Bruce Momjian
Robert Haas wrote: The consensus view on this thread seems to be that we should have a time-based code freeze, but not a time-based release. No one has argued (and I sincerely hope no one will argue) that we should let the last CommitFest drag on and on, as we did for 8.4. However, many

Re: [HACKERS] damage control mode

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 9:30 PM, Bruce Momjian br...@momjian.us wrote: Robert Haas wrote: The consensus view on this thread seems to be that we should have a time-based code freeze, but not a time-based release.  No one has argued (and I sincerely hope no one will argue) that we should let the

Re: [HACKERS] damage control mode

2010-01-11 Thread Bruce Momjian
Robert Haas wrote: On Mon, Jan 11, 2010 at 9:30 PM, Bruce Momjian br...@momjian.us wrote: Robert Haas wrote: The consensus view on this thread seems to be that we should have a time-based code freeze, but not a time-based release. ?No one has argued (and I sincerely hope no one will

Re: [HACKERS] pg_migrator issues

2010-01-11 Thread Bruce Momjian
Bruce Momjian wrote: Bruce Momjian wrote: 2) Right now pg_migrator renames old tablespaces to .old, which fails if the tablespaces are on mount points. I have already received a report of such a failure. $PGDATA also has that issue, but that renaming has to be done by the user before

Re: [HACKERS] pg_migrator issues

2010-01-11 Thread Bruce Momjian
FYI, I consider all the issues below to be addressed (we did all but #4), and pg_migrator will take advantage of these new facilities for 8.5. --- Bruce Momjian wrote: pg_migrator has become more popular recently, so it

Re: [HACKERS] damage control mode

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 9:45 PM, Bruce Momjian br...@momjian.us wrote: We could if we could all stop long enough to address them.  I think there is the feeling that a great idea will pop up eventually, and only when we are looking at beta do we realize we are out of time, and the hard,

Re: [HACKERS] damage control mode

2010-01-11 Thread Bruce Momjian
Robert Haas wrote: On Mon, Jan 11, 2010 at 9:45 PM, Bruce Momjian br...@momjian.us wrote: We could if we could all stop long enough to address them. ?I think there is the feeling that a great idea will pop up eventually, and only when we are looking at beta do we realize we are out of time,

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Fujii Masao
On Sat, Jan 9, 2010 at 4:25 PM, Heikki Linnakangas heikki.linnakan...@enterprisedb.com wrote: I don't think we need all that, a simple select() should be enough. Though I must admit I'm not very familiar with select/poll(). I'm not sure whether poll(2) should be called for this purpose. But

Re: [HACKERS] Compression Library and Usages

2010-01-11 Thread Merlin Moncure
On Mon, Jan 11, 2010 at 7:54 PM, Tom Lane t...@sss.pgh.pa.us wrote: Merlin Moncure mmonc...@gmail.com writes: plus, it looks like that most of the patents have either expired, or are about to expire.  lzo is used all over the place, including the linux kernel...i think the burden of proof

[HACKERS] Fix for memory leak in dblink

2010-01-11 Thread Takahiro Itagaki
There is a memory leak in dblink when we cancel a query during returning tuples. It could leak a PGresult because memory used by it is not palloc'ed one. I wrote a patch[1] before, but I've badly used global variables to track the resource. The attached is a cleaned up patch rewritten to use a

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Fujii Masao
On Sun, Jan 10, 2010 at 8:17 PM, Simon Riggs si...@2ndquadrant.com wrote: What could happen is that the standby could slowly lag behind master. We don't have any way of monitoring that, as yet. Setting ps display is not enough here. I agree that the statistical information about replication

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Fujii Masao
On Mon, Jan 11, 2010 at 5:36 PM, Craig Ringer cr...@postnewspapers.com.au wrote: Personally, I'd be uncomfortable enabling something like that without _both_ an admin alert _and_ the ability to refresh the slave's base backup without admin intervention. What feature do you specifically need as

Re: [HACKERS] damage control mode

2010-01-11 Thread Greg Smith
Bruce Momjian wrote: I think the big issue with 8.4 was, do we close the commit-fest when we have open issues, and we aren't clear on how to fix them? A lot of unresolve issues get kept for that pre-beta period because all of a sudden we have to resolve all those complex problems. I don't see

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Greg Smith
Fujii Masao wrote: On Sun, Jan 10, 2010 at 8:17 PM, Simon Riggs si...@2ndquadrant.com wrote: What could happen is that the standby could slowly lag behind master. We don't have any way of monitoring that, as yet. Setting ps display is not enough here. I agree that the statistical

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Greg Smith
Fujii Masao wrote: On Mon, Jan 11, 2010 at 5:36 PM, Craig Ringer cr...@postnewspapers.com.au wrote: Personally, I'd be uncomfortable enabling something like that without _both_ an admin alert _and_ the ability to refresh the slave's base backup without admin intervention. What feature

Re: [HACKERS] damage control mode

2010-01-11 Thread Robert Haas
On Mon, Jan 11, 2010 at 11:15 PM, Greg Smith g...@2ndquadrant.com wrote: Personally, I'd like the topic of a thread on damage control to be all about testing the one big patch that's already in there (HS), its related bits like the VACUUM FULL changes, and potentially SR too.  Those are things

Re: [HACKERS] mailing list archiver chewing patches

2010-01-11 Thread Dave Page
2010/1/12 Matteo Beccati p...@beccati.com: So, I've decided to spend a bit more time on this and here is a proof of concept web app that displays mailing list archives reading from the AOX database: http://archives.beccati.org/ Seems to work. Please take it as an exercise I've made

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Fujii Masao
On Tue, Jan 12, 2010 at 1:21 PM, Greg Smith g...@2ndquadrant.com wrote: I don't think anybody can deploy this feature without at least some very basic monitoring here.  I like the basic proposal you made back in September for adding a pg_standbys_xlog_location to replace what you have to get

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Fujii Masao
On Tue, Jan 12, 2010 at 1:24 PM, Greg Smith g...@2ndquadrant.com wrote: It's impossible for the database to have any idea whatsoever how people are going to want to be alerted.  Provide functions to monitor things like replication lag, like the number of segments queued up to feed to

Re: [HACKERS] Hot Standy introduced problem with query cancel behavior

2010-01-11 Thread Andres Freund
On 01/07/10 22:37, Andres Freund wrote: On Thursday 07 January 2010 22:28:46 Tom Lane wrote: Andres Freundand...@anarazel.de writes: I did not want to suggest using Simons code there. Sorry for the brevity. should have read as revert to old code and add two step killing (thats likely around

Re: [HACKERS] Hot Standy introduced problem with query cancel behavior

2010-01-11 Thread Andres Freund
On 01/07/10 22:37, Andres Freund wrote: On Thursday 07 January 2010 22:28:46 Tom Lane wrote: Andres Freundand...@anarazel.de writes: I did not want to suggest using Simons code there. Sorry for the brevity. should have read as revert to old code and add two step killing (thats likely around

Re: [HACKERS] damage control mode

2010-01-11 Thread Pavel Stehule
The consensus view on this thread seems to be that we should have a time-based code freeze, but not a time-based release.  No one has argued (and I sincerely hope no one will argue) that we should let the last CommitFest drag on and on, as we did for 8.4.  However, many people are still eager

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Heikki Linnakangas
Greg Smith wrote: I don't think anybody can deploy this feature without at least some very basic monitoring here. I like the basic proposal you made back in September for adding a pg_standbys_xlog_location to replace what you have to get from ps right now:

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Stefan Kaltenbrunner
Fujii Masao wrote: On Tue, Jan 12, 2010 at 1:21 PM, Greg Smith g...@2ndquadrant.com wrote: I don't think anybody can deploy this feature without at least some very basic monitoring here. I like the basic proposal you made back in September for adding a pg_standbys_xlog_location to replace what

Re: [HACKERS] Streaming replication status

2010-01-11 Thread Simon Riggs
On Tue, 2010-01-12 at 08:24 +0100, Stefan Kaltenbrunner wrote: Fujii Masao wrote: On Tue, Jan 12, 2010 at 1:21 PM, Greg Smith g...@2ndquadrant.com wrote: I don't think anybody can deploy this feature without at least some very basic monitoring here. I like the basic proposal you made back

Re: [HACKERS] [PATCH] remove redundant ownership checks

2010-01-11 Thread KaiGai Kohei
(2010/01/12 10:27), Bruce Momjian wrote: Robert Haas wrote: On Sun, Jan 10, 2010 at 4:54 PM, Tom Lanet...@sss.pgh.pa.us wrote: Robert Haasrobertmh...@gmail.com writes: I have looked this over a little bit and I guess I don't see why the lack of a grand plan for how to organize all of our

[HACKERS] planner or statistical bug on 8.5

2010-01-11 Thread Pavel Stehule
Hello I checked query and I was surprised with very strange plan: postgres=# create table a(a int primary key); NOTICE: CREATE TABLE / PRIMARY KEY will create implicit index a_pkey for table a CREATE TABLE postgres=# create table b(b int primary key); NOTICE: CREATE TABLE / PRIMARY KEY will