I rebased my patch set. New functions in pg_proc.h prevented to apply
previous revision cleanly. Here is no functional changes.
2011/11/3 Kohei KaiGai kai...@kaigai.gr.jp:
2011/11/2 Tom Lane t...@sss.pgh.pa.us:
Kohei KaiGai kai...@kaigai.gr.jp writes:
The reason why I redefined the relid of
Hi,
A new patch:
check for the presence of crtdefs.h in configure
-#if _MSC_VER = 1400 || defined(WIN64)
+#if _MSC_VER = 1400 || HAVE_CRTDEFS_H
#define errcode __msvc_errcode
#include crtdefs.h
#undef errcode
Perhaps there is no guarantee that mingw (not -w64) may not have crtdefs.h in
the
2011/12/3 Robert Haas robertmh...@gmail.com:
On Fri, Dec 2, 2011 at 6:52 AM, Kohei KaiGai kai...@kaigai.gr.jp wrote:
At least, it is working. However, it is not a perfect solution to the
future updates
of code paths in the core.
Hmm. So, do you want this committed? If so, I think the major
Pavel Stehule wrote:
My attempt at a syntax that could also cover Peter's wish for multiple
checker functions:
CHECK FUNCTION { func(args) | ALL [IN SCHEMA schema] [FOR ROLE user] }
[ USING check_function ] OPTIONS (optname optarg [, ...])
check_function should be related to one language,
Magnus Hagander wrote:
I'd vote for whatever matches the general perl pest practices at
this time.
I didn't kow the perl pest practices until now but as the PostgreSQL
community is more into C I think I know what you mean ;-)
--
Sent via pgsql-hackers mailing list
Simon Riggs si...@2ndquadrant.com writes:
Those are especially important because in 9.2 DDL commands will cause
additional locking overheads, so preventing DDL will be essential to
keeping performance stable in high txn rate databases.
The patch now implements any command triggers, and you
On Sat, Dec 3, 2011 at 09:24, NISHIYAMA Tomoaki
tomoa...@staff.kanazawa-u.ac.jp wrote:
Hi,
A new patch:
check for the presence of crtdefs.h in configure
-#if _MSC_VER = 1400 || defined(WIN64)
+#if _MSC_VER = 1400 || HAVE_CRTDEFS_H
#define errcode __msvc_errcode
#include crtdefs.h
Hi,
Have you verified if tihs affects _MSC_VER 1400? Suddently that
branch would care about HAVE_CRTDEFS_H, and I'm not sure if that's
something we need to worry about.
I have no MSVC. In that sense it is not verified in fact, and I hope
those who knows well would kindly comment on it.
Hi,
why the libraries loaded using local_preload_libraries need to be placed
in a different subdirectory than libraries loaded using
shared_preload_libraries?
And why it does not use dynamic_library_path but a hardcoded path
'$libdir/plugins'?
I do understand that leaving the users to load
On Sat, Dec 3, 2011 at 15:49, NISHIYAMA Tomoaki
tomoa...@staff.kanazawa-u.ac.jp wrote:
Hi,
Have you verified if tihs affects _MSC_VER 1400? Suddently that
branch would care about HAVE_CRTDEFS_H, and I'm not sure if that's
something we need to worry about.
I have no MSVC. In that sense it
On 03.12.2011 01:25, Daniel Farina wrote:
Here's a protocol: have pg_start_backup() write a file that just means
backing up. Restarts are OK, because that's all it means, it has no
meaning to a recovery/restoration process.
When one wishes to restore, one must touch a file -- not unlike the
On 19.10.2011 19:41, Greg Jaskiewicz wrote:
On 19 Oct 2011, at 18:28, Florian Pflug wrote:
All the other flags which indicate cancellation reasons are set from signal
handers, I believe. We could of course mark as ClientConnectionLostPending as
volatile just to be consistent. Not sure whether
On lör, 2011-11-26 at 22:36 +0200, Peter Eisentraut wrote:
There is a long-standing oddity in psql that running
psql -f foo.sql
returns error messages with file name and line number, like
psql:foo.sql:1: ERROR: syntax error at or near foo
but running
psql foo.sql does not. I
Tomas Vondra t...@fuzzy.cz writes:
why the libraries loaded using local_preload_libraries need to be placed
in a different subdirectory than libraries loaded using
shared_preload_libraries?
Security: it lets the DBA constrain which libraries are loadable this way.
I do understand that
On Saturday, December 03, 2011 12:04:57 AM Dimitri Fontaine wrote:
Hi,
First thing first: thank you Andres for a great review, I do appreciate
it. Please find attached a newer version of the patch. The github
repository is also updated.
Andres Freund and...@anarazel.de writes:
I think
On 3.12.2011 18:53, Tom Lane wrote:
Tomas Vondra t...@fuzzy.cz writes:
why the libraries loaded using local_preload_libraries need to be placed
in a different subdirectory than libraries loaded using
shared_preload_libraries?
Security: it lets the DBA constrain which libraries are loadable
Robert Haas robertmh...@gmail.com writes:
OK, so I tried to code this up. Adding the new amproc wasn't too
difficult (see attached). It wasn't obvious to me how to tie it into
the tuplesort infrastructure, though, so instead of wasting time
guessing what a sensible approach might be I'm
The patch is in context format, includes docs and additional
regression tests, applies cleanly, passes world regression tests.
The parser changes don't cause any backing up.
Discussion on the list seems to have reached a consensus that this
patch implements a useful feature. A previous version
I noticed the following object types don't have support for an ALTER ...
RENAME command:
DOMAIN (but ALTER TYPE works)
FOREIGN DATA WRAPPER
OPERATOR
RULE
SERVER
Are there any restrictions why these couldn't be added?
(I stumbled upon this while trying to rename a foreign server, but we
might as
Hi,
I wanted to craft an answer here and Peter nailed it before I could. I
use ip4r in a bunch of different projects and environments, it's doing a
perfect job, it's simple to use and damn efficient.
The ipv6 support is on the way, parts of it are already be in the CVS at
Tom Lane t...@sss.pgh.pa.us writes:
IIRC, a lot of the basic behavior of the inet/cidr types was designed by
Paul Vixie (though he's not to blame for their I/O presentation).
So I'm inclined to doubt that they're as broken as Stephen claims.
The ip4r extension's main use case is range lookups.
On Friday, December 02, 2011 03:09:55 AM Tom Lane wrote:
Andres Freund and...@anarazel.de writes:
On Thursday, December 01, 2011 07:21:25 PM Tom Lane wrote:
Making this work cleanly would be a bigger deal than I think you're
thinking.
Obviously that depends on the definition of
Hi,
I've written a simple extension that limits number of connection by
IP/db/user, and I do receive this exception:
psql: FATAL: cannot read pg_class without having selected a database
I've found this happens because the extension defines a client auth hook
that reads pg_stat_activity. The
2011/12/3 Tomas Vondra t...@fuzzy.cz:
psql: FATAL: cannot read pg_class without having selected a database
I've found this happens because the extension defines a client auth hook
that reads pg_stat_activity. The really interesting thing is that this
happens only when I start several
On 3.12.2011 23:37, Robert Haas wrote:
2011/12/3 Tomas Vondra t...@fuzzy.cz:
psql: FATAL: cannot read pg_class without having selected a database
I've found this happens because the extension defines a client auth hook
that reads pg_stat_activity. The really interesting thing is that this
On 12/03/2011 09:59 AM, Magnus Hagander wrote:
On Sat, Dec 3, 2011 at 15:49, NISHIYAMA Tomoaki
tomoa...@staff.kanazawa-u.ac.jp wrote:
Hi,
Have you verified if tihs affects _MSC_VER 1400? Suddently that
branch would care about HAVE_CRTDEFS_H, and I'm not sure if that's
something we need to
While doing some pgbench testing on a new server with 9.1.1, I noticed
quite a lot of $subject in the logs. I did some Googling and found this
previous thread:
http://archives.postgresql.org/pgsql-hackers/2010-01/msg02897.php It
doesn't seem like the issue was resolved?
I did:
# pgbench -i
On 12/03/2011 06:12 PM, Andrew Dunstan wrote:
On 12/03/2011 09:59 AM, Magnus Hagander wrote:
On Sat, Dec 3, 2011 at 15:49, NISHIYAMA Tomoaki
tomoa...@staff.kanazawa-u.ac.jp wrote:
Hi,
Have you verified if tihs affects _MSC_VER 1400? Suddently that
branch would care about
On Friday, December 02, 2011 03:09:55 AM Tom Lane wrote:
Andres Freund and...@anarazel.de writes:
On Thursday, December 01, 2011 07:21:25 PM Tom Lane wrote:
Making this work cleanly would be a bigger deal than I think you're
thinking.
Obviously that depends on the definition of
On Sat, Nov 19, 2011 at 10:56:03PM +0100, Boszormenyi Zoltan wrote:
Hopefully last turn in this topic. For NUMERIC types, the safe minimum
alignment is a pointer because there are 5 int members followed by
two pointer members in this struct. I got a crash from this with a lucky
query and
Excerpts from Alex Hunsaker's message of dom oct 09 03:40:36 -0300 2011:
On Fri, Oct 7, 2011 at 21:30, Nikhil Sontakke nikkh...@gmail.com wrote:
Hi Alex,
I guess we both are in agreement with each other :)
After sleeping over it, I think that check is indeed dead code with this new
Hi,
On 2011/12/04, at 9:45, Andrew Dunstan wrote:
Yes, but there's a deal more work to do here. This whole thing is falling
over in my build environment (64 bit Windows 7, MinGW/MSys, the machine that
runs pitta on the buildfarm.)
This is a long way from a done deal.
In particular,
Tomas Vondra t...@fuzzy.cz writes:
That might explain why it fails at first and then works just fine,
although it's a bit strange. Wouldn't that mean you can't access any
catalogs from the auth hook?
It should be possible to access shared catalogs from an auth hook.
pg_stat_activity is neither
I had a look at this patch today. The pg_dump bits conflict with
another patch I committed a few days ago, so I'm about to merge them.
I have one question which is about this hunk:
Thanks for taking a look Alvaro.
@@ -2312,6 +2317,11 @@ MergeWithExistingConstraint(Relation rel, char
Hello
I have a relative simple query
SELECT q.object_id
FROM queue q
JOIN
outgoing.cps_forms f
ON f.id = q.object_id AND q.object_type = 'cp'
JOIN
flat_file_ex fe
ON fe.id = q.rejected_flat_file_id
WHERE
a plan for modified query is
ohs=# explain analyze SELECT object_id,
inserted,
'ASSIGN_RSLT',
order_id,
2,
seqnum,
rejected_flat_file_id,
true
FROM (
SELECT q.object_id,
fe.inserted,
q.order_id,
q.seqnum,
36 matches
Mail list logo
