On Sun, Jan 19, 2014 at 5:27 PM, David Rowley dgrowle...@gmail.com wrote:
It's probably far more worth it for the bool and/or aggregates. We
could just
keep track of the values aggregated and the count of values as true
and return
true if those are the same in the case of AND, then check
On 18 January 2014 03:07, Marko Tiikkaja ma...@joh.to wrote:
On 1/12/14, 5:53 AM, I wrote:
On 1/9/14, 2:57 PM, Dean Rasheed wrote:
How it should behave for multi-dimensional arrays is less clear, but
I'd argue that it should return the total number of elements, i.e.
Stephen Frost sfr...@snowman.net writes:
* Robert Haas (robertmh...@gmail.com) wrote:
I kind of don't see the point of having IF NOT EXISTS for things that
have OR REPLACE, and am generally in favor of implementing OR REPLACE
rather than IF NOT EXISTS where possible. The point is usually to
2014-01-19 Andreas Karlsson andr...@proxel.se:
Hi,
I will review your two patches (gist support + selectivity). This is part 1
of my review. I will look more into the actual GiST implementation in a
couple of days, but thought I could provide you with my initial input right
away.
Thank you
On 1/19/14, 9:12 AM, Dean Rasheed wrote:
On 18 January 2014 03:07, Marko Tiikkaja ma...@joh.to wrote:
Here's the patch as promised. Thoughts?
A couple of points:
The answer for empty (zero dimensional) arrays is wrong --- you need
special case handling for this case to return 0.
How
On Sat, Jan 18, 2014 at 11:34 AM, David Rowley dgrowle...@gmail.com wrote:
The latest version of the patch is attached.
I've attached an updated version of the patch.
I'm now using github to track the changes on the patch, so I've included
the commit sha in the file name of the latest commit
On Fri, Jan 17, 2014 at 4:53 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Magnus Hagander mag...@hagander.net writes:
Applied, thanks!
Minor bikeshedding: the messages would read better, to my eye, as
user=%s database=%s SSL enabled (protocol=%s, cipher=%s)
Putting enabled where it is requires
On 19 January 2014 11:43, Marko Tiikkaja ma...@joh.to wrote:
On 1/19/14, 9:12 AM, Dean Rasheed wrote:
On 18 January 2014 03:07, Marko Tiikkaja ma...@joh.to wrote:
Here's the patch as promised. Thoughts?
A couple of points:
The answer for empty (zero dimensional) arrays is wrong ---
On 1/19/14, 2:12 PM, Dean Rasheed wrote:
That might seem overly pedantic, but it's quite annoying when API
documentation doesn't fully specify the behaviour, and you're forced
to use trial-and-error to find out how the functions behave.
For what it's worth, I was thinking the same thing when I
On 01/18/2014 02:31 PM, Robert Haas wrote:
On Thu, Jan 16, 2014 at 10:15 PM, Craig Ringer cr...@2ndquadrant.com wrote:
Anybody who actually uses SHIFT_JIS as an operational encoding, rather
than as an input/output encoding, is into pain and suffering. Personally
I'd be quite happy to see it
On Thu, Dec 5, 2013 at 7:54 PM, MauMau maumau...@gmail.com wrote:
Hello,
I've removed a limitation regarding event log on Windows with the attached
patch. I hesitate to admit this is a bug fix and want to regard this an
improvement, but maybe it's a bug fix from users' perspective.
On Sat, Jan 18, 2014 at 3:59 PM, Andrew Dunstan and...@dunslane.net wrote:
On 01/16/2014 08:01 AM, Magnus Hagander wrote:
On Wed, Jan 15, 2014 at 6:57 PM, Tom Lane t...@sss.pgh.pa.us mailto:
t...@sss.pgh.pa.us wrote:
Magnus Hagander mag...@hagander.net mailto:mag...@hagander.net
Hi all,
We have encountered an issue when executing an insert command,
when one of the values' length was bigger than the column size limitation.
the error message which been displayed was value too long for type char...
but there was no indication which value has exceeded the limited size.
(See
On Jan19, 2014, at 05:27 , David Rowley dgrowle...@gmail.com wrote:
I just finished implementing the inverse transition functions for bool_and
and bool_or, these aggregates had a sort operator which I assume would have
allowed an index scan to be performed, but since I had to change the first
Maor Lipchuk mlipc...@redhat.com writes:
We have encountered an issue when executing an insert command,
when one of the values' length was bigger than the column size limitation.
the error message which been displayed was value too long for type char...
but there was no indication which value
Today is your last chance to submit a proposal for PGCon 2014.
PGCon 2014 will be on 20-24 May 2014 at University of Ottawa.
* 20-21 (Tue-Wed) tutorials
* 22-23 (Thu-Fri) talks - the main part of the conference
* 24 (Sat) The Unconference (very popular in 2013, the first year)
See
On Mon, Jan 20, 2014 at 5:53 AM, Florian Pflug f...@phlo.org wrote:
On Jan19, 2014, at 05:27 , David Rowley dgrowle...@gmail.com wrote:
I just finished implementing the inverse transition functions for
bool_and
and bool_or, these aggregates had a sort operator which I assume would
have
On Sun, Jan 19, 2014 at 8:10 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Complaining about a too-long varchar string in this style
seems practically guaranteed to violate that.
Agreed. But I think it would be useful to add the length of the string
being inserted; we already have it in the len
Marti Raudsepp ma...@juffo.org writes:
Agreed. But I think it would be useful to add the length of the string
being inserted; we already have it in the len variable.
Hm, maybe, but I'm having a hard time visualizing cases in which it
helps much.
regards, tom lane
--
2014/1/17 Jov am...@amutu.com
but in the psql --help,-F say:
set field separator (default: |)
if user don't read the offical doc carefully,he can use:
psql -F , -c 'select ...'
But can't get what he want.
It is a bad user Experience.
+1 from me, patch applies and is helpful.
After
On Tue, Nov 19, 2013 at 6:38 PM, Andres Freund and...@2ndquadrant.com wrote:
The attached patches compile and make check successfully on linux x86,
amd64 and msvc x86 and amd64. This time and updated configure is
included. The majority of operations still rely on CAS emulation.
Note that the
Marti Raudsepp ma...@juffo.org writes:
On Sun, Jan 19, 2014 at 8:10 PM, Tom Lane t...@sss.pgh.pa.us wrote:
One thing that occurs to me just now is that perhaps we could report
the failure as if it were a syntax error
That would be cool, if it can be made to work.
Just as a five-minute
On 01/19/2014 11:10 AM, Emre Hasegeli wrote:
I am not convinced an adjacent operator is useful for the inet type, but if
it is included it should be indexed just like -|- of ranges. We should try
to keep these lists of indexed operators the same.
I added it just not to leave negotor field
Since contrib/test_shm_mq went into the tree, crake has been crashing
intermittently (maybe one time in three) at the contribcheck step.
While there's not conclusive proof that test_shm_mq is at fault, the
circumstantial evidence is pretty strong. For instance, in the
postmaster log for the last
I wrote:
I don't think any other buildfarm critters are showing this either,
Scratch that --- closer inspection of the buildfarm logs shows that
kouprey and lapwing have suffered identical failures, though much less
often than crake. That lets out the theory that it's x86_64 specific, or
even
From: Amit Kapila amit.kapil...@gmail.com
Today, I was trying to reproduce this issue and found that if user tries
to register event source second time with same name, we just replace
the previous event source's path in registry.
Shouldn't we try to stop user at this step only, means if he tries
Here comes part 2 of 2 of the review.
inet-gist
-
In general the code looks good and I think your approach makes sense.
Not an expert on GiST though so I would like a second opinion on this.
Maybe there is some clever trick which would make the index more
efficient, but the design I
On 01/17/2014 07:41 PM, Magnus Hagander wrote:
Regardless of where the other thread goes, this seems like something we
should fix. Thus - applied, with minor changes to the comment, thanks.
Thanks.
My understanding is that this change alone doesn't actually help us very
much, so I haven't
Hi Tom and Marti
Thank you so much for your respond.
The solution Tom proposed is much more better, and it will be a great
solution for clarifying many issues regarding this error.
Regards,
Maor
On 01/19/2014 10:00 PM, Tom Lane wrote:
Marti Raudsepp ma...@juffo.org writes:
On Sun, Jan 19,
On 01/19/2014 04:14 PM, Tom Lane wrote:
I wrote:
I don't think any other buildfarm critters are showing this either,
Scratch that --- closer inspection of the buildfarm logs shows that
kouprey and lapwing have suffered identical failures, though much less
often than crake. That lets out the
On Sun, Jan 19, 2014 at 7:53 PM, Andrew Dunstan and...@dunslane.net wrote:
Also crake does produce backtraces on core dumps, and they are at the
bottom of the buildfarm log. The latest failure backtrace is reproduced
below.
== stack trace:
On Fri, Jan 17, 2014 at 5:45 AM, Marko Tiikkaja ma...@joh.to wrote:
On 1/17/14 11:26 AM, Pavel Stehule wrote:
After some thinking I don't think so this design is not good. It changing
a working with exception (error) levels - and it is not consistent with
other PostgreSQL parts.
A benefit
On 01/18/2014 03:27 AM, Gregory Smith wrote:
With my advocacy hat on, I'd like to revisit this idea now that there's
a viable updatable security barrier view submission. I thought the most
serious showstopper feedback from the last CF's RLS submission was that
this needed to be sorted out
On 01/18/2014 12:26 AM, David Rowley wrote:
-#if defined(_WIN32)
+#if defined(_WIN32) !defined(WIN32)
That makes sense, since we force WIN32 in the build system. I should've
seen that. Thanks for catching it.
--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL
On 01/17/2014 05:28 AM, Stephen Frost wrote:
Greetings,
Harking back to 10 years ago when tablespaces were added, it looks
like we originally figured that users didn't need permissions to
create tables in the database default, per 2467394e. That strikes
me as perfectly fair.
On Jan19, 2014, at 20:00 , David Rowley dgrowle...@gmail.com wrote:
I've applied that patch again and put in the sort operators.
I've push a new version to https://github.com/fgp/postgres/tree/invtrans
This branch includes the following changes
* A bunch of missing declaration for *_inv
On Mon, Jan 20, 2014 at 4:05 AM, MauMau maumau...@gmail.com wrote:
From: Amit Kapila amit.kapil...@gmail.com
Today, I was trying to reproduce this issue and found that if user tries
to register event source second time with same name, we just replace
the previous event source's path in
On Sat, Jan 18, 2014 at 7:59 PM, Michael Paquier
michael.paqu...@gmail.com wrote:
Hi all,
After going through commit 65d6e4c (introducing ALTER SYSTEM SET), I
noticed a couple of typo mistakes as well as (I think) a weird way of
using the temporary auto-configuration name
Thank you.
tgl Hello, I noticed that pg_stat_statements.query can have funny values.
tgl
tgl I don't think that's an acceptable reason for lobotomizing the parser's
tgl ability to print error cursors, which is what your first patch does
tgl (and without even any documentation that would keep
I went to review this, and found that there's not actually a patch
attached ...
regards, tom lane
Attached. Sorry for that.
--
Jeevan B Chalke
Principal Software Engineer, Product Development
EnterpriseDB Corporation
The Enterprise PostgreSQL Company
diff --git
Hi,
I have begun the review as part of the commitfest. Below are my comments
*_guts() functions are defined as returning Datum, while they are actually
returning Oid. They should be defined as returning Oid.
Also the PG_RETURN_OID() has been still used in some of the *_guts()
Hello,
Please consider the following test:
create database foo;
\c foo
create table foo_test ( a int );
\c postgres
create extension if not exists postgres_fdw;
create server foo_server foreign data wrapper postgres_fdw options ( dbname
'foo' );
create user mapping for current_user server
Hello
you need installed devel packages
Regards
Pavel Stehule
2014/1/20 Sameer Kumar sameer.ku...@ashnik.com
Hi,
I have downloaded the tar source code of PostgreSQL and also the SPEC
file. I am trying to use rpmbuild command but I always get below error:
error: Failed build
Please consider attached patch here as earlier attached wrong patch.
Sorry for the inconvenience.
On Mon, Jan 20, 2014 at 1:21 PM, Rushabh Lathia rushabh.lat...@gmail.comwrote:
Hello,
Please consider the following test:
create database foo;
\c foo
create table foo_test ( a int );
\c
44 matches
Mail list logo