Zoltan Boszormenyi [EMAIL PROTECTED] writes:
Gregory Stark írta:
1) Please don't include configure in your patch. I don't know why it's
checked
into CVS but it is so that means manually removing it from any patch. It's
usually a huge portion of the diff so it's worth removing.
Noted.
Just
I haven't had time to look through the patch, but reading Gregs comments
I noted:
2) The genbki.sh change could be a bit tricky for multi-platform builds (ie
OSX). I don't really see an alternative so it's just something to note for
the folks setting that up (Hi Dave).
Changes to
Tom Lane wrote:
Zoltan Boszormenyi [EMAIL PROTECTED] writes:
Gregory Stark írta:
1) Please don't include configure in your patch. I don't know why it's checked
into CVS but it is so that means manually removing it from any patch. It's
usually a huge portion of the diff so it's worth
Tom Lane wrote:
Zoltan Boszormenyi [EMAIL PROTECTED] writes:
Gregory Stark írta:
1) Please don't include configure in your patch. I don't know why it's checked
into CVS but it is so that means manually removing it from any patch. It's
usually a huge portion of the diff so it's worth
you have extra space onPavel Stehule napsal(a):
Hello
http://archives.postgresql.org/pgsql-patches/2007-10/msg00161.php
I actualized this patch for current CVS
Hi Pavel,
I tested your patch and it works regarding to proposal. However
I have some small comments:
1) you have extra space in
I don't think
my $int64passbyval = (?($real64 = 1)t|f);
works. Perhaps
my $int64passbyval = $real64 ? 't' : 'f';
--
Alvaro Herrerahttp://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
pg84-passedbyval-v4.patch.gz
Alvaro Herrera írta:
I don't think
my $int64passbyval = (?($real64 = 1)t|f);
works. Perhaps
my $int64passbyval = $real64 ? 't' : 'f';
Thanks. Modified patch attached.
Stupid question follows. Now that float4 is passed by value
unconditionally, is it worth modifying the
Tom Lane wrote:
Bruce Momjian [EMAIL PROTECTED] writes:
I have developed the attached patch with improves wording for the
pg_dump -i (ignore version) option.
I think this is going in exactly the wrong direction --- it makes
both the documentation and the warning message less scary not
On Thu, 2008-02-28 at 16:37 -0800, Neil Conway wrote:
Attached is a patch that allows an array of Datums + nulls to be
inserted into a tuplestore without first creating a HeapTuple, per
recent suggestion on -hackers. This avoids making an unnecessary copy.
Applied to HEAD.
-Neil
--
Sent
Attached is a refreshed patch that makes integer datetimes the default.
Platforms that don't have a working 64-bit integer type will fail to
configure by default; they can specify --disable-integer-datetimes to
switch back to using floating-point based datetimes.
Barring any objections, I'll
Attached is the doc patch for the recent DTrace changes
http://archives.postgresql.org/pgsql-committers/2008-03/msg00316.php
Regards,
-Robert
Index: doc/src/sgml/installation.sgml
===
RCS file:
On Sat, 2008-03-22 at 21:35 -0400, Tom Lane wrote:
After a quick read, looks sane except for one stylistic gripe:
in exec_stmt_return_next, you added an initialization of tuple = NULL
in order to remove a couple of lines like
tuple = NULL; /* keep compiler quiet */
On Tue, 2008-03-18 at 16:19 -0300, Alvaro Herrera wrote:
The other approach, of course, is to just keep all the code in tqual.c
and not create a separate module at all. Opinions? I prefer to keep
them separate, but I'm not wedded to it if there's any strong reason not
to do it. Also, the
Brendan Jurd [EMAIL PROTECTED] writes:
As discussed on -hackers, I'm trying to get rid of some redundant code
by creating a widely useful set of functions to convert between text
and C string in the backend.
Applied with revisions --- the functions were modified as per recent
discussion, and I
On Tue, 2008-03-25 at 15:45 -0500, Robert Lor wrote:
Attached is the doc patch for the recent DTrace changes
Applied, thanks.
-Neil
--
Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
Merlin Moncure escribió:
The latest version of libpq type system is available here:
http://www.esilo.com/projects/postgresql/libpq/typesys-0.9a.tar.gz
This patch is not in diff -c format ... please provide a diff -c patch,
and add the URL to the wiki patch queue:
Neil Conway [EMAIL PROTECTED] writes:
On Tue, 2008-03-18 at 16:19 -0300, Alvaro Herrera wrote:
The other approach, of course, is to just keep all the code in tqual.c
and not create a separate module at all. Opinions? I prefer to keep
them separate, but I'm not wedded to it if there's any
Brendan Jurd wrote:
On 9/29/07, Bruce Momjian [EMAIL PROTECTED] wrote:
I think we need more than one person's request to add this function.
Well, I don't expect it would get requested. Most DBAs would likely
look for the function in the docs, see it's not there and then just
implement it
18 matches
Mail list logo