Re: [HACKERS] inherit support for foreign tables

2014-11-07 Thread Etsuro Fujita
Hi Furuya-san, (2014/11/07 16:54), furu...@pm.nttdata.co.jp wrote: (2014/08/28 18:00), Etsuro Fujita wrote: (2014/08/22 11:51), Noah Misch wrote: Today's ANALYZE VERBOSE messaging for former inheritance parents (tables with relhassubclass = true but no pg_inherits.inhparent links) is

Re: [HACKERS] Tweaking Foreign Keys for larger tables

2014-11-07 Thread Andreas Karlsson
On 11/07/2014 08:15 AM, Simon Riggs wrote: How about we set lock level on each Foreign Key like this [USING LOCK [lock level]] level is one of KEY - [FOR KEY SHARE] - default ROW - [FOR SHARE] TABLE SHARE - [ ] TABLE EXCLUSIVE - [FOR TABLE EXCLUSIVE] I like the idea and thinks it solves the

Re: [HACKERS] inherit support for foreign tables

2014-11-07 Thread Etsuro Fujita
(2014/11/07 14:57), Kyotaro HORIGUCHI wrote: Here are separated patches. fdw-chk.patch - CHECK constraints on foreign tables fdw-inh.patch - table inheritance with foreign tables The latter has been created on top of [1]. [1]

Re: [HACKERS] two dimensional statistics in Postgres

2014-11-07 Thread Katharina Büchse
Ahoj ;-) On 06.11.2014 11:56, Tomas Vondra wrote: Hi, Dne 6 Listopad 2014, 11:15, Katharina Büchse napsal(a): Hi, I'm a phd-student at the university of Jena, Thüringen, Germany, in the field of data bases, more accurate query optimization. I want to implement a system in PostgreSQL that

Re: [HACKERS] New Event Trigger: table_rewrite

2014-11-07 Thread Dimitri Fontaine
Simon Riggs si...@2ndquadrant.com writes: It would be more useful to work on the applications of this 1. INSERT into a table * Action start time * Schema * Tablename * Number of blocks in table which would then allow you to do these things run an assessment report showing which tables

Re: [HACKERS] pg_receivexlog --status-interval add fsync feedback

2014-11-07 Thread Fujii Masao
On Fri, Oct 31, 2014 at 5:46 PM, furu...@pm.nttdata.co.jp wrote: We seem to be going in circles. You suggested having two options, --feedback, and --fsync, which is almost exactly what Furuya posted originally. I objected to that, because I think that user interface is too complicated.

[HACKERS] Defining dedicated macro to grab a relation's persistence

2014-11-07 Thread Michael Paquier
Hi all, After looking at a patch of this commit fest using rd_rel-relpersistence, I got a look at how many times this expression was being used directly in the backend code and wondered if it would not be useful to add a dedicated macro in rel.h to get the persistence of a relation like in the

Re: [HACKERS] Defining dedicated macro to grab a relation's persistence

2014-11-07 Thread Andres Freund
Hi, On 2014-11-07 22:08:33 +0900, Michael Paquier wrote: After looking at a patch of this commit fest using rd_rel-relpersistence, I got a look at how many times this expression was being used directly in the backend code and wondered if it would not be useful to add a dedicated macro in

Re: [HACKERS] [BUGS] ltree::text not immutable?

2014-11-07 Thread Robert Haas
On Thu, Nov 6, 2014 at 5:13 PM, Tom Lane t...@sss.pgh.pa.us wrote: Jim Nasby jim.na...@bluetreble.com writes: On 11/5/14, 7:38 PM, Robert Haas wrote: I don't understand why you went to all the trouble of building a versioning system for extensions if you're not going to use it. I was about

Re: [HACKERS] Proposal: Log inability to lock pages during vacuum

2014-11-07 Thread Alvaro Herrera
Greg Stark wrote: I agree bloat isn't really a threat, but what about the relfrozenxid? If we skip even one page we don't get to advance it and retrying could eliminate those skipped pages and allow us to avoid a vacuum freeze which can be really painful. Of course that only works if you can

Re: [HACKERS] Representing a SRF return column in catalogs

2014-11-07 Thread Robert Haas
On Wed, Nov 5, 2014 at 8:24 AM, Atri Sharma atri.j...@gmail.com wrote: I am working on something that requires representing a SRF return column in pg_proc and being able to retrieve it, retrieve the name of the column and make a ColumnRef node from it. The catch here are aliases: SELECT

Re: [HACKERS] Add CREATE support to event triggers

2014-11-07 Thread Alvaro Herrera
Michael Paquier wrote: Here are more thoughts among those lines looking at the current state of the patch 4 that introduces the infrastructure of the whole feature. This would make possible in-memory manipulation of jsonb containers without relying on a 3rd-part set of APIs like what this

Re: [HACKERS] Sequence Access Method WIP

2014-11-07 Thread Alvaro Herrera
Craig Ringer wrote: On 11/05/2014 05:01 AM, Petr Jelinek wrote: I guess I could port BDR sequences to this if it would help (once we have bit more solid agreement that the proposed API at least theoretically seems ok so that I don't have to rewrite it 10 times if at all possible).

[HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Ross Reedstrom
This is a serious bug in 9.3.5 and 9.4 beta3: row_to_json() yields empty strings for json keys if the data is fulfilled by an index only scan. Example: testjson=# select count(*) from document_acl; count --- 426 (1 row) testjson=# SELECT row_to_json(combined_rows) FROM ( SELECT uuid,

Re: [HACKERS] WIP: dynahash replacement for buffer table

2014-11-07 Thread Robert Haas
On Wed, Nov 5, 2014 at 6:19 PM, Andres Freund and...@2ndquadrant.com wrote: * In benchmarks it becomes apparent that the dynamic element width makes some macros like CHashTableGetNode() and CHashTableGetGarbageByBucket() quite expensive. At least gcc on x86 can't figure how to build an

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Robert Haas
On Wed, Nov 5, 2014 at 9:15 PM, Josh Berkus j...@agliodbs.com wrote: What I'm pointing out is that you can't actually do that. You think you can, but you can't. I do think that. You haven't explained why I'm wrong; just asserted than I am. Which doesn't really get us anywhere. However, if

Re: [HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Andrew Dunstan
On 11/07/2014 10:51 AM, Ross Reedstrom wrote: This is a serious bug in 9.3.5 and 9.4 beta3: row_to_json() yields empty strings for json keys if the data is fulfilled by an index only scan. Example: testjson=# select count(*) from document_acl; count --- 426 (1 row) testjson=#

Re: [HACKERS] Representing a SRF return column in catalogs

2014-11-07 Thread Atri Sharma
On Fri, Nov 7, 2014 at 7:15 PM, Robert Haas robertmh...@gmail.com wrote: On Wed, Nov 5, 2014 at 8:24 AM, Atri Sharma atri.j...@gmail.com wrote: I am working on something that requires representing a SRF return column in pg_proc and being able to retrieve it, retrieve the name of the column

Re: [HACKERS] Sequence Access Method WIP

2014-11-07 Thread Petr Jelinek
On 06/11/14 11:22, Craig Ringer wrote: On 11/05/2014 05:01 AM, Petr Jelinek wrote: I guess I could port BDR sequences to this if it would help (once we have bit more solid agreement that the proposed API at least theoretically seems ok so that I don't have to rewrite it 10 times if at all

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Petr Jelinek
On 06/11/14 08:50, Andres Freund wrote: On 2014-11-05 19:31:52 -0500, Steve Singer wrote: It also doesn't allow you to load two extensions at once on a system. You wouldn't be able to have both the 'steve_commit_order' extension and BDR installed at the same time. I don't think this patch does

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Petr Jelinek
On 06/11/14 01:31, Steve Singer wrote: On 11/05/2014 05:43 PM, Andres Freund wrote: Is this patch supposed to: A) Add commit timestamp tracking but nothing more B) Add infrastructure to store commit timestamps and provide a facility for storing additional bits of data extensions might want

Re: [HACKERS] jsonb generator functions

2014-11-07 Thread Andrew Dunstan
On 10/28/2014 09:49 AM, Andrew Dunstan wrote: On 10/27/2014 05:57 PM, Alvaro Herrera wrote: Anyway this whole business of searching through the CASTSOURCETARGET syscache seems like it could be refactored. If I'm counting correctly, that block now appears four times (three in this patch,

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Josh Berkus
On 11/07/2014 08:12 AM, Robert Haas wrote: On Wed, Nov 5, 2014 at 9:15 PM, Josh Berkus j...@agliodbs.com wrote: What I'm pointing out is that you can't actually do that. You think you can, but you can't. I do think that. You haven't explained why I'm wrong; just asserted than I am. Which

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Josh Berkus
All, The point of this thread was to determine: Is the current interaction of recovery_target_time and standby_mode (that is, that recovery_target_time causes standby_mode to be ignorned) the correct behavior? If Yes, then we have a tech bug and a doc bug: Tech Bug: if the user sets both

[HACKERS] remove pg_standby?

2014-11-07 Thread Peter Eisentraut
While we're talking about removing old things, is there any use left for pg_standby? -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] TODO request: log_long_transaction

2014-11-07 Thread Michael Banck
Hi, Am Montag, den 27.10.2014, 19:29 + schrieb Thom Brown: On 27 October 2014 19:21, Josh Berkus j...@agliodbs.com wrote: I just realized that there is one thing we can't log currently: transactions which last more than #ms. This is valuable diagnostic information when looking for

Re: [HACKERS] two dimensional statistics in Postgres

2014-11-07 Thread Tomas Vondra
On 7.11.2014 13:19, Katharina Büchse wrote: On 06.11.2014 11:56, Tomas Vondra wrote: Dne 6 Listopad 2014, 11:15, Katharina Büchse napsal(a): because correlations might occur only in parts of the data. In this case a histogram based on a sample of the whole table might not get the point and

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Andres Freund
On 2014-11-07 17:54:32 +0100, Petr Jelinek wrote: On 06/11/14 08:50, Andres Freund wrote: On 2014-11-05 19:31:52 -0500, Steve Singer wrote: It also doesn't allow you to load two extensions at once on a system. You wouldn't be able to have both the 'steve_commit_order' extension and BDR

Re: [HACKERS] BRIN indexes - TRAP: BadArgument

2014-11-07 Thread Alvaro Herrera
I just pushed this, after some more minor tweaks. Thanks, and please do continue testing! -- Álvaro Herrerahttp://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 1:35 PM, Josh Berkus j...@agliodbs.com wrote: On 11/07/2014 08:12 AM, Robert Haas wrote: On Wed, Nov 5, 2014 at 9:15 PM, Josh Berkus j...@agliodbs.com wrote: What I'm pointing out is that you can't actually do that. You think you can, but you can't. I do think that.

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 1:40 PM, Josh Berkus j...@agliodbs.com wrote: Is the current interaction of recovery_target_time and standby_mode (that is, that recovery_target_time causes standby_mode to be ignorned) the correct behavior? I think this summary of the behavior is probably not correct in

Re: [HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Andrew Dunstan
On 11/07/2014 11:17 AM, Andrew Dunstan wrote: On 11/07/2014 10:51 AM, Ross Reedstrom wrote: This is a serious bug in 9.3.5 and 9.4 beta3: row_to_json() yields empty strings for json keys if the data is fulfilled by an index only scan. Example: testjson=# select count(*) from document_acl;

Re: [HACKERS] Representing a SRF return column in catalogs

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 11:31 AM, Atri Sharma atri.j...@gmail.com wrote: Let me give an example: Consider an user defined SRF (or an inbuilt one (generate_series)). I am working on a path keys tracking project (more details on it in a separate email). I am interested in one of the columns of

Re: [HACKERS] On partitioning

2014-11-07 Thread Robert Haas
On Thu, Nov 6, 2014 at 9:17 PM, Amit Langote langote_amit...@lab.ntt.co.jp wrote: I mentioned upthread about the possibility of resurrecting Itagaki-san's patch [1] to try to make things work in this direction. I would be willing to spend time on this. I see useful reviews of the patch by

Re: [HACKERS] Repeatable read and serializable transactions see data committed after tx start

2014-11-07 Thread Robert Haas
On Wed, Nov 5, 2014 at 7:17 PM, Álvaro Hernández Tortosa a...@8kdata.com wrote: Given a transaction started with BEGIN (REPEATABLE READ | SERIALIZABLE), if a concurrent session commits some data before *any* query within the first transaction, that committed data is seen by the

Re: [HACKERS] B-Tree index builds, CLUSTER, and sortsupport

2014-11-07 Thread Robert Haas
On Wed, Nov 5, 2014 at 8:30 PM, Peter Geoghegan p...@heroku.com wrote: Thanks for the review. Committed with some copy-editing based on Andreas's comments. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list

Re: [HACKERS] split builtins.h to quote.h

2014-11-07 Thread Alvaro Herrera
Michael Paquier wrote: On Fri, Nov 7, 2014 at 2:31 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: I thought the consensus was that the SQL-callable function declarations should remain in builtins.h -- mainly so that quote.h does not need to include fmgr.h. Moving everything to

Re: [HACKERS] Doing better at HINTing an appropriate column within errorMissingColumn()

2014-11-07 Thread Robert Haas
On Mon, Oct 6, 2014 at 3:09 AM, Heikki Linnakangas hlinnakan...@vmware.com wrote: On 07/18/2014 10:47 AM, Michael Paquier wrote: On Fri, Jul 18, 2014 at 3:54 AM, Peter Geoghegan p...@heroku.com wrote: I am not opposed to moving the contrib code into core in the manner that you oppose. I

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Josh Berkus
On 11/07/2014 12:02 PM, Robert Haas wrote: Several people then suggested that you could accomplish your originally stated goal - namely restore a master *and replica* to a point in time before Bad Stuff happened, and then have a working master-replica pair - by just connecting the new standby

Re: [HACKERS] Repeatable read and serializable transactions see data committed after tx start

2014-11-07 Thread Greg Sabino Mullane
-BEGIN PGP SIGNED MESSAGE- Hash: RIPEMD160 Kevin Grittner wrote: I think most people have always assumed that BEGIN starts the transaction and that is the point at which the snapshot is obtained. But there is so much evidence to the contrary. Not only does the *name* of the

Re: [HACKERS] B-Tree index builds, CLUSTER, and sortsupport

2014-11-07 Thread Peter Geoghegan
On Fri, Nov 7, 2014 at 12:52 PM, Robert Haas robertmh...@gmail.com wrote: Committed with some copy-editing based on Andreas's comments. Thank you both. -- Peter Geoghegan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription:

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 4:00 PM, Josh Berkus j...@agliodbs.com wrote: On 11/07/2014 12:02 PM, Robert Haas wrote: Several people then suggested that you could accomplish your originally stated goal - namely restore a master *and replica* to a point in time before Bad Stuff happened, and then

Re: [HACKERS] split builtins.h to quote.h

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 3:55 PM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: Michael Paquier wrote: On Fri, Nov 7, 2014 at 2:31 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: I thought the consensus was that the SQL-callable function declarations should remain in builtins.h --

Re: [HACKERS] printing table in asciidoc with psql

2014-11-07 Thread Alvaro Herrera
I did \o /tmp/tst, then \dS create table eh | oh (); \dS and then filtered the output file to HTML. The CREATE TABLE tag ended up in the same line as the title of the following table. I think there's a newline is missing somewhere. The good news is that the | in the table name was processed

Re: [HACKERS] Representing a SRF return column in catalogs

2014-11-07 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: On Fri, Nov 7, 2014 at 11:31 AM, Atri Sharma atri.j...@gmail.com wrote: Can I store relattnos or something? I need to get the stored att in planner and build pathkeys from it. I still can't really follow. The columns returned by an SRF can't change

Re: [HACKERS] split builtins.h to quote.h

2014-11-07 Thread Alvaro Herrera
Robert Haas wrote: I personally think that's getting our priorities backwards, but there's clearly a spectrum in terms of how much people care about the cost of partial compiles, and I'm clearly all the way on one end of it. I don't like having to think hard about where a function prototype

Re: [HACKERS] split builtins.h to quote.h

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 4:42 PM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: Robert Haas wrote: I personally think that's getting our priorities backwards, but there's clearly a spectrum in terms of how much people care about the cost of partial compiles, and I'm clearly all the way on one

Re: [HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Tom Lane
Andrew Dunstan and...@dunslane.net writes: On 11/07/2014 10:51 AM, Ross Reedstrom wrote: row_to_json() yields empty strings for json keys if the data is fulfilled by an index only scan. Could this be a bug in lookup_rowtype_tupdesc()? I think this is probably a variant of bug #11210, in

Re: [HACKERS] recovery_target_time and standby_mode

2014-11-07 Thread Josh Berkus
On 11/07/2014 01:30 PM, Robert Haas wrote: On Fri, Nov 7, 2014 at 4:00 PM, Josh Berkus j...@agliodbs.com wrote: In order for this to work, the archive would need to stop before recovery_target_time. Yeah, good point. I didn't think of the case where you've rewound the master but not the

Re: [HACKERS] Representing a SRF return column in catalogs

2014-11-07 Thread Atri Sharma
On Saturday, November 8, 2014, Tom Lane t...@sss.pgh.pa.us wrote: I'm confused too. The original example seemed to imagine that details of a query (not the function, but the calling query) would be stored in the catalogs, which is completely nuts. pg_proc already has provisions to remember

Re: [HACKERS] [v9.5] Custom Plan API

2014-11-07 Thread Robert Haas
On Mon, Oct 27, 2014 at 2:35 AM, Kouhei Kaigai kai...@ak.jp.nec.com wrote: FYI, patch v12 part 2 no longer applies cleanly. Thanks. I rebased the patch set according to the latest master branch. The attached v13 can be applied to the master. I've committed parts 1 and 2 of this, without the

Re: [HACKERS] [v9.5] Custom Plan API

2014-11-07 Thread Kouhei Kaigai
On Mon, Oct 27, 2014 at 2:35 AM, Kouhei Kaigai kai...@ak.jp.nec.com wrote: FYI, patch v12 part 2 no longer applies cleanly. Thanks. I rebased the patch set according to the latest master branch. The attached v13 can be applied to the master. I've committed parts 1 and 2 of this,

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Robert Haas
On Nov 5, 2014, at 7:31 PM, Steve Singer st...@ssinger.info wrote: On 11/05/2014 05:43 PM, Andres Freund wrote: On 2014-11-05 17:17:05 -0500, Steve Singer wrote: Imo that's essentially a different feature. What you essentially would need here is a 'commit sequence number' - but no timestamps.

Re: [HACKERS] BRIN indexes - TRAP: BadArgument

2014-11-07 Thread David Rowley
On Sat, Nov 8, 2014 at 8:56 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: I just pushed this, after some more minor tweaks. Thanks, and please do continue testing! I'm having problems getting this to compile on MSVC. Attached is a patch which fixes the problem. There also seems to be

Re: [HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Andrew Dunstan
On 11/07/2014 04:59 PM, Tom Lane wrote: Andrew Dunstan and...@dunslane.net writes: On 11/07/2014 10:51 AM, Ross Reedstrom wrote: row_to_json() yields empty strings for json keys if the data is fulfilled by an index only scan. Could this be a bug in lookup_rowtype_tupdesc()? I think this is

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Petr Jelinek
On 08/11/14 00:35, Robert Haas wrote: On Nov 5, 2014, at 7:31 PM, Steve Singer st...@ssinger.info wrote: On 11/05/2014 05:43 PM, Andres Freund wrote: On 2014-11-05 17:17:05 -0500, Steve Singer wrote: Imo that's essentially a different feature. What you essentially would need here is a 'commit

Re: [HACKERS] pg_multixact not getting truncated

2014-11-07 Thread Josh Berkus
On 11/05/2014 11:15 AM, Josh Berkus wrote: On 11/05/2014 10:40 AM, Jim Nasby wrote: Can you post the contents of pg_multixact/members/? Well, not as of last week, obviously. https://gist.github.com/jberkus/d05db3629e8c898664c4 I haven't pasted all the filenames, because, well, look at

Re: [HACKERS] Sequence Access Method WIP

2014-11-07 Thread Petr Jelinek
On 08/11/14 00:57, Petr Jelinek wrote: On 08/11/14 00:45, Robert Haas wrote: On Nov 5, 2014, at 5:43 PM, Petr Jelinek p...@2ndquadrant.com wrote: I don't see how to make that work with ALTER SEQUENCE USING to be honest and I do care quite a lot about that use-case (I think the ability to

Re: [HACKERS] pg_multixact not getting truncated

2014-11-07 Thread Alvaro Herrera
Josh Berkus wrote: So, just did a quick survey of our managed services customers. 4 out of 6 have pg_multixact bloat. Basically it looks like this: User1 (the one above): 1.1GB User2: 64K User3: 929MB User4: 1.3GB User5: 301MB User6: 48K Nobody (out of 6 more) running a version

Re: [HACKERS] pg_multixact not getting truncated

2014-11-07 Thread Josh Berkus
On 11/07/2014 04:43 PM, Alvaro Herrera wrote: This says that the live multixact range goes from 123 million to 162 million; roughly 40 million values. (The default value for vacuum_multixact_freeze_table_age is 150 million, which is what determines how many values are kept.) You

Re: [HACKERS] pg_multixact not getting truncated

2014-11-07 Thread Alvaro Herrera
Josh Berkus wrote: On 11/07/2014 04:43 PM, Alvaro Herrera wrote: This says that the live multixact range goes from 123 million to 162 million; roughly 40 million values. (The default value for vacuum_multixact_freeze_table_age is 150 million, which is what determines how many values are

Re: [HACKERS] pg_basebackup fails with long tablespace paths

2014-11-07 Thread Peter Eisentraut
On 11/4/14 3:52 PM, Peter Eisentraut wrote: Here are patches to address that. First, it reports errors when attempting to create a tar header that would truncate file or symlink names. Second, it works around the problem in the tests by creating a symlink from the short-name tempdir that we

Re: [HACKERS] tracking commit timestamps

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 7:07 PM, Petr Jelinek p...@2ndquadrant.com wrote: The list of what is useful might be long, That's FUD. It might also be short. but we can't have everything there as there are space constraints, and LSN is another 8 bytes and I still want to have some bytes for

Re: [HACKERS] Sequence Access Method WIP

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 7:26 PM, Petr Jelinek p...@2ndquadrant.com wrote: My main problem is actually not with having tuple per-seqAM, but more with the fact that Heikki does not want to have last_value as compulsory column/parameter. How is the new AM then supposed to know where to pick up and

Re: [HACKERS] TODO request: log_long_transaction

2014-11-07 Thread Robert Haas
You should add this patch here, so it doesn't get forgotten: https://commitfest.postgresql.org/action/commitfest_view/open On Fri, Nov 7, 2014 at 2:19 PM, Michael Banck michael.ba...@credativ.de wrote: 1. Should this log when the duration is exceeded (like log_lock_waits), or on commit? I

Re: [HACKERS] Proposal: Log inability to lock pages during vacuum

2014-11-07 Thread Robert Haas
On Thu, Nov 6, 2014 at 8:03 PM, Jim Nasby jim.na...@bluetreble.com wrote: The problem right now is there's no way to actually obtain evidence that this is (or isn't) something to worry about, because we just silently skip pages. If we had any kind of tracking on this we could stop guessing. :(

Re: [HACKERS] BRIN indexes - TRAP: BadArgument

2014-11-07 Thread Tom Lane
David Rowley dgrowle...@gmail.com writes: I'm having problems getting this to compile on MSVC. Attached is a patch which fixes the problem. The committed code is completely broken on compilers that don't accept varargs macros, and this patch will not make them happier. Probably what needs to

Re: [HACKERS] Add CREATE support to event triggers

2014-11-07 Thread Robert Haas
On Fri, Nov 7, 2014 at 10:45 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: Michael Paquier wrote: Here are more thoughts among those lines looking at the current state of the patch 4 that introduces the infrastructure of the whole feature. This would make possible in-memory manipulation

[HACKERS] Re: [BUGS] BUG #11867: Strange behaviour with composite types after resetting database tablespace

2014-11-07 Thread Robert Haas
On Tue, Nov 4, 2014 at 11:45 AM, Tom Lane t...@sss.pgh.pa.us wrote: I wrote: However: at no point in this sequence did we flush the original catalog blocks out of shared buffers. Therefore, a read of the database's pg_class or pg_type at this point is likely to find the previous states of

Re: [HACKERS] [PATCH] HINT: pg_hba.conf changed since last config reload

2014-11-07 Thread Robert Haas
On Thu, Nov 6, 2014 at 5:46 PM, Peter Eisentraut pete...@gmx.net wrote: I think it's fine to log a message in the server log if the pg_hba.conf file needs reloading. But the client shouldn't know about this at all. I agree. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The

Re: [HACKERS] BRIN indexes - TRAP: BadArgument

2014-11-07 Thread Alvaro Herrera
Tom Lane wrote: David Rowley dgrowle...@gmail.com writes: I'm having problems getting this to compile on MSVC. Attached is a patch which fixes the problem. The committed code is completely broken on compilers that don't accept varargs macros, and this patch will not make them happier. I

Re: [HACKERS] row_to_json bug with index only scans: empty keys!

2014-11-07 Thread Tom Lane
Andrew Dunstan and...@dunslane.net writes: On 11/07/2014 04:59 PM, Tom Lane wrote: I think this is probably a variant of bug #11210, in which the problem is that tupledescs bubbled up from inheritance children never get column names assigned to them. I've been speculating about ways to fix

Re: [HACKERS] remove pg_standby?

2014-11-07 Thread Michael Paquier
On Wed, Nov 5, 2014 at 6:36 AM, Peter Eisentraut pete...@gmx.net wrote: While we're talking about removing old things, is there any use left for pg_standby? +1 for this autumn cleanup. -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your

[HACKERS] Race in tablespace test on Windows

2014-11-07 Thread Noah Misch
In my Windows development environment, the tablespace regression test fails approximately half the time. Buildfarm member frogmouth failed in the same manner at least once: http://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=frogmouthdt=2014-05-21%2014%3A30%3A01 Here is a briefer command

Re: [HACKERS] split builtins.h to quote.h

2014-11-07 Thread Michael Paquier
On Sat, Nov 8, 2014 at 5:55 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: Michael Paquier wrote: On Fri, Nov 7, 2014 at 2:31 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: I thought the consensus was that the SQL-callable function declarations should remain in builtins.h --

Re: [HACKERS] Convert query plan to sql query

2014-11-07 Thread mariem
Hi Tom, If you don't need that, why are you insistent on extracting the information from a plan tree? I need to resolve expressions and apply rewrite rules before I reverse the query plan to a query. It seems far simpler to me to make use of ruleutils.c to reverse-list the original parsetree.

Re: [HACKERS] BRIN indexes - TRAP: BadArgument

2014-11-07 Thread David Rowley
On Sat, Nov 8, 2014 at 8:56 AM, Alvaro Herrera alvhe...@2ndquadrant.com wrote: I just pushed this, after some more minor tweaks. Thanks, and please do continue testing! Here's another small fix for some unused variable warnings. Unfortunately this Microsoft compiler that I'm using does not