Re: [HACKERS] Visual Studio 2010/Windows SDK 7.1 support

2011-07-03 Thread Brar Piening
schrieb Craig Ringer: I'd appreciate it if you'd post the config.pl you're building with, mainly so I can see what libraries and optional features you are or are not using with your test builds. I've already posted it here: http://archives.postgresql.org/message-id/4dcda3cd.4030...@gmx.de

Re: [HACKERS] Visual Studio 2010/Windows SDK 7.1 support

2011-07-03 Thread Dave Page
On Sun, Jul 3, 2011 at 8:28 AM, Brar Piening b...@gmx.de wrote: Perhaps someone who routinely does 64-bit builds could help out with a config.pl and some info from where to get the 64-bit libraries to build with. Here's what we use for the installers. # Configuration arguments for vcbuild.

Re: [HACKERS] [v9.2] Fix leaky-view problem, part 2

2011-07-03 Thread Kohei KaiGai
The simplified version of fix-leaky-view patch. The part of reloptions for views got splitted out into the part-0 patch, so it needs to be applied prior to this patch. Rest of logic to prevent unexpected pushing down across security barrier is not changed. Thanks, 2011/6/6 Kohei Kaigai

Re: [HACKERS] per-column generic option

2011-07-03 Thread Kohei KaiGai
Hanada-san, I checked the per-column generic option patch. Right now, I have nothing to comment on anymore. So, it should be reviewed by committers. Thanks, 2011年6月27日16:47 Shigeru Hanada shigeru.han...@gmail.com: (2011/06/26 18:34), Kohei KaiGai wrote: I checked your patch. Thanks for the

Re: [HACKERS] Deriving release notes from git commit messages

2011-07-03 Thread Martijn van Oosterhout
On Sat, Jul 02, 2011 at 03:45:03PM -0400, Robert Haas wrote: There are git notes which you can attach to a commit after the fact... I like the fact that they would keep the information in the repository (where they seem to belong). Yeah, but I think it's still basically append-only,

[HACKERS] %ENV warnings during builds

2011-07-03 Thread Craig Ringer
Hi folks It looks like pgflex.bat and pgbison.bat in the win32 build files would benefit from a tweak to eliminate a complaint from Perl. Perl emits the error: Hash %ENV missing the % in argument 1 of each() at -e line 1 on the lines that process buildenv.pl. This is because % in batch

Re: [HACKERS] %ENV warnings during builds

2011-07-03 Thread Andrew Dunstan
On Sun, July 3, 2011 8:14 am, Craig Ringer wrote: Hi folks It looks like pgflex.bat and pgbison.bat in the win32 build files would benefit from a tweak to eliminate a complaint from Perl. Perl emits the error: Hash %ENV missing the % in argument 1 of each() at -e line 1 on the lines

Re: [HACKERS] %ENV warnings during builds

2011-07-03 Thread Peter Eisentraut
On sön, 2011-07-03 at 20:14 +0800, Craig Ringer wrote: It looks like pgflex.bat and pgbison.bat in the win32 build files would benefit from a tweak to eliminate a complaint from Perl. Perl emits the error: Hash %ENV missing the % in argument 1 of each() at -e line 1 on the lines

Re: [HACKERS] `make world` exit code

2011-07-03 Thread Peter Eisentraut
On lör, 2011-07-02 at 22:36 -0700, David E. Wheeler wrote: Hackers, I'm building a new server using 9.1beta2. My build script includes these two line: make world -j3 || exit $? make install-world || exit $? Much to my annoyance, `make world` seems to succeed, but the script

Re: [HACKERS] plpgsql extension install nitpick

2011-07-03 Thread Peter Eisentraut
On lör, 2011-07-02 at 23:22 -0400, Tom Lane wrote: We just went through an exercise to suppress comments on functions that are meant to be accessed through operators, and this seems like much the same kind of situation. I think it will not be long before COMMENT ON PROCEDURAL LANGUAGE is a

Re: [HACKERS] %ENV warnings during builds

2011-07-03 Thread Magnus Hagander
On Sun, Jul 3, 2011 at 13:06, Andrew Dunstan and...@dunslane.net wrote: On Sun, July 3, 2011 8:14 am, Craig Ringer wrote: Hi folks It looks like pgflex.bat and pgbison.bat in the win32 build files would benefit from a tweak to eliminate a complaint from Perl. Perl emits the error:    Hash

Re: [HACKERS] %ENV warnings during builds

2011-07-03 Thread Andrew Dunstan
On Sun, July 3, 2011 8:54 am, Peter Eisentraut wrote: On sön, 2011-07-03 at 20:14 +0800, Craig Ringer wrote: It looks like pgflex.bat and pgbison.bat in the win32 build files would benefit from a tweak to eliminate a complaint from Perl. Perl emits the error: Hash %ENV missing the % in

Re: [HACKERS] plpgsql extension install nitpick

2011-07-03 Thread Tom Lane
Peter Eisentraut pete...@gmx.net writes: On lör, 2011-07-02 at 23:22 -0400, Tom Lane wrote: [ don't want to put duplicate comments on PLs and their extensions ] Well, you still write functions in the language and refer to it in the SQL commands, so it is reasonable to know about the language

Re: [HACKERS] Deriving release notes from git commit messages

2011-07-03 Thread Tom Lane
Martijn van Oosterhout klep...@svana.org writes: On Sat, Jul 02, 2011 at 03:45:03PM -0400, Robert Haas wrote: There are git notes which you can attach to a commit after the fact... I like the fact that they would keep the information in the repository (where they seem to belong). Yeah, but

Re: [HACKERS] plpgsql extension install nitpick

2011-07-03 Thread Tom Lane
Josh Kupershmidt schmi...@gmail.com writes: On Sat, Jul 2, 2011 at 11:22 PM, Tom Lane t...@sss.pgh.pa.us wrote: Josh Kupershmidt schmi...@gmail.com writes: [ plpgsql's comment is now attached to the extension, not the PL itself ] ... And if you're wondering why I care about any of this, it's

Re: [HACKERS] Deriving release notes from git commit messages

2011-07-03 Thread Andres Freund
On Sunday, July 03, 2011 06:46:15 PM Tom Lane wrote: A look at the git-notes man page says that you can only have one note per commit, but you can edit that note, and git does track the revision history of each note. I think that we should adopt git notes as a better solution than making

[HACKERS] Full GUID support

2011-07-03 Thread Michael Gould
I would like to request that full support for the UUID data type can added.  I think that even though there is a contrib module, since this is a standard datatype that Postgres ought to be the one actually assigning the value. Best Regards Michael Gould -- Michael Gould, Managing Partner

Re: [HACKERS] [COMMITTERS] pgsql: Enable CHECK constraints to be declared NOT VALID

2011-07-03 Thread Peter Eisentraut
On tor, 2011-06-30 at 15:09 -0400, Alvaro Herrera wrote: Robert Hass (whose name I misspelled in the commit message above) just mentioned to me (in an answer to my apologizing about it) that he didn't think that mentioning sponsors for patch development was a good idea. I don't think we

Re: [HACKERS] Full GUID support

2011-07-03 Thread Peter Eisentraut
On sön, 2011-07-03 at 13:42 -0500, Michael Gould wrote: I would like to request that full support for the UUID data type can added. I think that even though there is a contrib module, since this is a standard datatype that Postgres ought to be the one actually assigning the value. What

Re: [HACKERS] [COMMITTERS] pgsql: Enable CHECK constraints to be declared NOT VALID

2011-07-03 Thread Magnus Hagander
On Sun, Jul 3, 2011 at 20:51, Peter Eisentraut pete...@gmx.net wrote: On tor, 2011-06-30 at 15:09 -0400, Alvaro Herrera wrote: Robert Hass (whose name I misspelled in the commit message above) just mentioned to me (in an answer to my apologizing about it) that he didn't think that mentioning

Re: [HACKERS] plpgsql extension install nitpick

2011-07-03 Thread Robert Haas
On Sun, Jul 3, 2011 at 12:33 PM, Tom Lane t...@sss.pgh.pa.us wrote: Peter Eisentraut pete...@gmx.net writes: On lör, 2011-07-02 at 23:22 -0400, Tom Lane wrote: [ don't want to put duplicate comments on PLs and their extensions ] Well, you still write functions in the language and refer to it

Re: [HACKERS] Deriving release notes from git commit messages

2011-07-03 Thread Magnus Hagander
On Sun, Jul 3, 2011 at 20:04, Andres Freund and...@anarazel.de wrote: On Sunday, July 03, 2011 06:46:15 PM Tom Lane wrote: A look at the git-notes man page says that you can only have one note per commit, but you can edit that note, and git does track the revision history of each note. I

Re: [HACKERS] Deriving release notes from git commit messages

2011-07-03 Thread Andres Freund
On Sunday, July 03, 2011 09:18:52 PM Magnus Hagander wrote: On Sun, Jul 3, 2011 at 20:04, Andres Freund and...@anarazel.de wrote: On Sunday, July 03, 2011 06:46:15 PM Tom Lane wrote: A look at the git-notes man page says that you can only have one note per commit, but you can edit that

Re: [HACKERS] [COMMITTERS] pgsql: Enable CHECK constraints to be declared NOT VALID

2011-07-03 Thread Simon Riggs
On Sun, Jul 3, 2011 at 7:51 PM, Peter Eisentraut pete...@gmx.net wrote: On tor, 2011-06-30 at 15:09 -0400, Alvaro Herrera wrote: Robert Hass (whose name I misspelled in the commit message above) just mentioned to me (in an answer to my apologizing about it) that he didn't think that mentioning

Re: [HACKERS] Full GUID support

2011-07-03 Thread Michael Gould
Peter, I don't believe that the library that the contrib module runs with can run on Window 64 bit servers or even Windows 7 64 bit. That is problem as most shops are using 64 bit OS and if Window the contrib module is going to fail. Taking the responsibility to handle this internally means

Re: [HACKERS] Full GUID support

2011-07-03 Thread Dave Page
On Sunday, July 3, 2011, Michael Gould mgo...@intermodalsoftwaresolutions.net wrote: gt; Peter, gt; gt; I don't believe that the library that the contrib module runs with can run gt; on Window 64 bit servers or even Windows 7 64 bit.  That is problem as most gt; shops are using 64 bit OS and if

Re: [HACKERS] Full GUID support

2011-07-03 Thread Tom Lane
Dave Page dp...@pgadmin.org writes: Even If that weren't the case, integrating the type wouldn't fix the problem anyway, unless you're suggesting we implement our own UUID generator (which isn't nearly as straightforward as it might seem, as I understand it).. Yeah. If there were One True

Re: [HACKERS] %ENV warnings during builds

2011-07-03 Thread Brar Piening
schrieb Magnus Hagander: I think you've stumbled on just about all the bits of the MSVC build system we haven't perlized. Maybe we should complete that task, and turn clean.bat, pgbison.bat and pgflex.bat into pure one-line wrappers. (It was done for builddoc just a few weeks ago). Yeah, give

Re: [HACKERS] [COMMITTERS] pgsql: Make the visibility map crash-safe.

2011-07-03 Thread Robert Haas
On Thu, Jun 30, 2011 at 6:19 PM, Jeff Davis pg...@j-davis.com wrote: On Thu, 2011-06-30 at 07:50 -0400, Robert Haas wrote: I compare the performance of commit 431ab0e82819b31fcd1e33ecb52c2cd3b4b41da7 (post-patch) with commit 431ab0e82819b31fcd1e33ecb52c2cd3b4b41da7 (pre-patch). I believe

Re: [HACKERS] Full GUID support

2011-07-03 Thread Michael Gould
Does this look to be something that will surface around for 9.1 Sent from Samsung mobile Dave Page dp...@pgadmin.org wrote: On Sunday, July 3, 2011, Michael Gould mgo...@intermodalsoftwaresolutions.net wrote: gt; Peter, gt; gt; I don't believe that the library that the contrib module runs with

[HACKERS] non-superuser reserved connections? connection pools?

2011-07-03 Thread Michael Glaesemann
It would be nice to be able to set aside a few connections for non-superusers, such as stats-monitoring connections. There's often no reason to grant these users superuser privileges (they're just observers, and security-definer functions can make anything visible that they may need)), but at

Re: [HACKERS] plpgsql extension install nitpick

2011-07-03 Thread Tom Lane
Robert Haas robertmh...@gmail.com writes: On Sun, Jul 3, 2011 at 12:33 PM, Tom Lane t...@sss.pgh.pa.us wrote: Well, if I'm in the minority I'll yield gracefully.  Anybody else have an opinion? I don't see any harm in commenting it in both places. OK, done. I did not add comment commands to

[HACKERS] flex on win64 - workaround for flex: fatal internal error, exec failed

2011-07-03 Thread Craig Ringer
Hi all On my win64 build machine I've been suffering from hard to track down errors from flex like: flex: fatal internal error, exec failed ... even though I have m4 properly installed and whether or not the M4 environment variable is set to point to my M4 executable. It all works fine

Re: [HACKERS] relpersistence and temp table

2011-07-03 Thread Robert Haas
On Fri, Jul 1, 2011 at 1:59 PM, Robert Haas robertmh...@gmail.com wrote: On Fri, Jul 1, 2011 at 10:32 AM, Robert Haas robertmh...@gmail.com wrote: On Fri, Jul 1, 2011 at 8:06 AM, Amit Khandekar amit.khande...@enterprisedb.com wrote: In 9.1, if a table is created using an explicit pg_temp

Re: [HACKERS] per-column generic option

2011-07-03 Thread Shigeru Hanada
(2011/07/03 18:50), Kohei KaiGai wrote: I checked the per-column generic option patch. Right now, I have nothing to comment on anymore. So, it should be reviewed by committers. Thanks for the review!. Regards, -- Shigeru Hanada -- Sent via pgsql-hackers mailing list

Re: [HACKERS] Online base backup from the hot-standby

2011-07-03 Thread Fujii Masao
2011/7/1 Jun Ishiduka ishizuka@po.ntts.co.jp: On this commitfest, the goal of the patch is to be able to be recovered using Minimum recovery ending location in the control file. Done. When the standby restarts after it crashes during recovery, it always checks whether recovery has

Re: [HACKERS] Crash dumps

2011-07-03 Thread Craig Ringer
On 15/06/2011 2:37 AM, Radosław Smogura wrote: Hello, Because, I work a little bit on streaming protocol and from time to time I have crashes. I want ask if you wont crash reporting (this is one of minors products from mmap playing) those what I have there is mmaped areas, and call stacks, and