GCC 4.9 build on Solaris 10 shows these warnings about isinf:
float.c: In function 'is_infinite':
float.c:178:2: warning: implicit declaration of function 'isinf'
[-Wimplicit-function-declaration]
See
http://pgbuildfarm.org/cgi-bin/show_stage_log.pl?nm=dingo&dt=2014-09-23%2002%3A52%3A00&stg=m
Hi all,
thanks for all your answers, I see your point. And I also understand the
argument according to which there always be some other use case to satisfy.
However your suggestion to use COPY TO sql TO PROGRAM doesn’t seem to me to fit
well the use case I have in mind.
Imagine you access PG f
On 22 September 2014 18:51, Stephen Frost Wrote:
* Rajeev rastogi (rajeev.rast...@huawei.com) wrote:
> Thanks, I shall start to prepare a patch for this optimization and share in 1
> or 2 days.
> This sounded interesting to me also- please be sure to put it on the open
> commitfest once you hav
Jan Wieck writes:
> On 09/15/2014 09:46 PM, Craig Ringer wrote:
>> Anyway - this is looking like the change will go in, and with it a
>> catversion bump. Introduction of a jsonb version/flags byte might be
>> worthwhile at the same time. It seems likely that there'll be more room
>> for improvemen
On Tue, Sep 23, 2014 at 10:02 PM, Jan Wieck wrote:
>> Is it worth paying a byte per value to save on possible upgrade pain?
>>
>
> This comment seems to have drowned in the discussion.
>
> If there indeed has to be a catversion bump in the process of this, then I
> agree with Craig.
-1. We alread
On 09/15/2014 09:46 PM, Craig Ringer wrote:
On 09/16/2014 07:44 AM, Peter Geoghegan wrote:
FWIW, I am slightly concerned about weighing use cases around very
large JSON documents too heavily. Having enormous jsonb documents just
isn't going to work out that well, but neither will equivalent desi
On 9/23/14, 7:13 PM, Robert Haas wrote:
I think we expose far too little information in our system views. Just
to take one example, we expose no useful information about lwlock
acquire or release, but a lot of real-world performance problems are
caused by lwlock contention.
I sent over a propos
On 9/23/14, 10:52 PM, David Johnston wrote:
Given the following why not just pick "ms" for log_rotation_age now?
Right now there are people out there who have configurations that look
like this:
log_rotation_age=60
In order to get hourly rotation. These users will suffer some trauma
shou
Robert Haas wrote:
> On Tue, Sep 23, 2014 at 9:23 PM, Alvaro Herrera
> wrote:
> > If you or somebody else want to give it a look, I have
> > no problem waiting a bit longer. I don't want to delay indefinitely,
> > though, because I think it's better shipped early in the release cycle
> > than la
On Tue, Sep 23, 2014 at 10:11 PM, Gregory Smith
wrote:
> On 9/23/14, 1:21 AM, David Johnston wrote:
>
>> This patch should fix the round-to-zero issue. If someone wants to get
>> rid of zero as a special case let them supply a separate patch for that
>> "improvement".
>>
>
> I am going to wander
(2014/09/17 1:58), Robert Haas wrote:
On Tue, Sep 16, 2014 at 11:31 AM, Kevin Grittner wrote:
Etsuro Fujita wrote:
(2014/08/15 6:18), Rukh Meski wrote:
Based on the feedback on my previous patch, I've separated only the
LIMIT part into its own feature. This version plays nicely with
inherit
On Wed, Aug 27, 2014 at 7:43 PM, Peter Geoghegan wrote:
> Omission
> ===
>
> The patch currently lacks a way of referencing datums rejected for
> insertion when updating.
Attached revision of the patch set (which I'll call v1.2) adds this
capability in a separate commit. It now becomes possib
On 9/23/14, 1:21 AM, David Johnston wrote:
This patch should fix the round-to-zero issue. If someone wants to
get rid of zero as a special case let them supply a separate patch for
that "improvement".
I am going to wander into this fresh after just reading everything once
(but with more than
(2014/09/13 2:42), Fujii Masao wrote:
On Wed, Sep 10, 2014 at 10:37 PM, Alvaro Herrera
wrote:
Fujii Masao wrote:
On Wed, Sep 10, 2014 at 12:15 PM, Etsuro Fujita
wrote:
PENDING_LIST_CLEANUP_SIZE and work_mem, for this setting.
Wouldn't it be easy-to-use to have only one parameter,
PENDING_L
On Tue, Sep 23, 2014 at 7:42 PM, Andres Freund wrote:
>> It will actually be far worse than that, because we'll acquire and
>> release the spinlock for every buffer over which we advance the clock
>> sweep, instead of just once for the whole thing.
>
> I said double, because we already acquire the
On Tue, Sep 23, 2014 at 9:23 PM, Alvaro Herrera
wrote:
> Robert Haas wrote:
>> With all respect, I think this is a bad idea. I know you've put a lot
>> of energy into this patch and I'm confident it's made a lot of
>> progress. But as with Stephen's patch, the final form deserves a
>> thorough r
On Tue, Sep 23, 2014 at 7:35 PM, Michael Paquier
wrote:
> Would this person be it an extra committer or an simple reviewer? It
> would give more insurance if such huge patches (couple of thousands of
> lines) get an extra +1 from another committer, proving that the code
> has been reviewed by peop
On 24/09/14 11:29, Mingzhe Li wrote:
Hi experts,
I want to know what's the "core" function used in Postgres server? I am
looking for something corresponding to main() in a simple C program. I
want to know the file path and the function name. I am using Postgres
9.3.5, however I assume the "core"
Robert Haas wrote:
> With all respect, I think this is a bad idea. I know you've put a lot
> of energy into this patch and I'm confident it's made a lot of
> progress. But as with Stephen's patch, the final form deserves a
> thorough round of looking over by someone else before it goes in.
As y
On 9/23/14, 8:02 PM, Michael Paquier wrote:
pgsql-hackers is as well a mailing list where people have technical
discussions about features and patches, hence your question would be
more adapted for pgsql-novice or pgsql-general.
Let's be fair and get the details perfect if we're going to advis
Peter Geoghegan writes:
> On Fri, Sep 19, 2014 at 5:40 AM, Heikki Linnakangas
> wrote:
>> I think we should bite the bullet and break compatibility with 9.4beta2
>> format, even if we go with "my patch". In a jsonb object, it makes sense to
>> store all the keys first, like Tom did, because of ca
On Fri, Sep 19, 2014 at 5:40 AM, Heikki Linnakangas
wrote:
> I think we should bite the bullet and break compatibility with 9.4beta2
> format, even if we go with "my patch". In a jsonb object, it makes sense to
> store all the keys first, like Tom did, because of cache benefits, and the
> future p
On 09/23/2014 12:23 PM, Dmitry Dolgov wrote:
Hi all,
I'm faced with some troubles about the jsonb implementation, and I
hope I'll get little advice =)
If I understand correctly, an abstract function for jsonb modification
should have the following stages:
Jsonb -> JsonbValue -> Modification
On Wed, Sep 24, 2014 at 8:29 AM, Mingzhe Li wrote:
> Hi experts,
>
> I want to know what's the "core" function used in Postgres server? I am
> looking for something corresponding to main() in a simple C program. I want
> to know the file path and the function name. I am using Postgres 9.3.5,
> how
On Tue, Sep 23, 2014 at 4:29 PM, Mingzhe Li wrote:
> I want to know what's the "core" function used in Postgres server? I am
> looking for something corresponding to main() in a simple C program. I want
> to know the file path and the function name. I am using Postgres 9.3.5,
> however I assume th
On 2014-09-23 19:21:10 -0400, Robert Haas wrote:
> On Tue, Sep 23, 2014 at 6:54 PM, Andres Freund wrote:
> > I think it might be possible to construct some cases where the spinlock
> > performs worse than the lwlock. But I think those will be clearly in the
> > minority. And at least some of those
Hi experts,
I want to know what's the "core" function used in Postgres server? I am
looking for something corresponding to main() in a simple C program. I want
to know the file path and the function name. I am using Postgres 9.3.5,
however I assume the "core" function will be unchanged between dif
On Wed, Sep 24, 2014 at 8:23 AM, Robert Haas wrote:
> On Tue, Sep 23, 2014 at 3:04 PM, Alvaro Herrera
> wrote:
>> Alvaro Herrera wrote:
>> I will look into adding some testing mechanism for the union support
>> proc; with that I will just consider the patch ready for commit and will
>> push.
>
>
On Tue, Sep 23, 2014 at 3:04 PM, Alvaro Herrera
wrote:
> Alvaro Herrera wrote:
>> Heikki Linnakangas wrote:
>> > If you add a new datatype, and define b-tree operators for it, what
>> > is required to create a minmax opclass for it? Would it be possible
>> > to generalize the functions in brin_min
On Tue, Sep 23, 2014 at 6:54 PM, Andres Freund wrote:
> Am I understanding you correctly that you also measured context switches
> for spinlocks? If so, I don't think that's a valid comparison. LWLocks
> explicitly yield the CPU as soon as there's any contention while
> spinlocks will, well, spin.
On Tue, Sep 23, 2014 at 6:02 PM, Gregory Smith wrote:
> On 9/23/14, 10:31 AM, Robert Haas wrote:
>> I suggest we count these things:
>>
>> 1. The number of buffers the reclaimer has put back on the free list.
>> 2. The number of times a backend has run the clocksweep.
>> 3. The number of buffers p
On 2014-09-23 16:29:16 -0400, Robert Haas wrote:
> On Tue, Sep 23, 2014 at 10:55 AM, Robert Haas wrote:
> > But this gets at another point: the way we're benchmarking this right
> > now, we're really conflating the effects of three different things:
> >
> > 1. Changing the locking regimen around t
On 2014-09-24 00:27:25 +0300, Oskari Saarenmaa wrote:
> 23.09.2014, 15:18, Andres Freund kirjoitti:
> >On 2014-09-23 13:50:28 +0300, Oskari Saarenmaa wrote:
> >>23.09.2014, 00:01, Andres Freund kirjoitti:
> >>>The patches:
> >>>0001: The actual atomics API
> >>
> >>I tried building PG on Solaris 10
On 9/23/14, 10:31 AM, Robert Haas wrote:
I suggest we count these things:
1. The number of buffers the reclaimer has put back on the free list.
2. The number of times a backend has run the clocksweep.
3. The number of buffers past which the reclaimer has advanced the
clock sweep (i.e. the numbe
On Tue, Sep 23, 2014 at 5:43 PM, Tom Lane wrote:
> Robert Haas writes:
>> On Tue, Sep 23, 2014 at 4:29 PM, Robert Haas wrote:
>>> I did some more experimentation on this. Attached is a patch that
>>> JUST does #1, and, ...
>
>> ...and that was the wrong patch. Thanks to Heikki for point that o
Robert Haas writes:
> On Tue, Sep 23, 2014 at 4:29 PM, Robert Haas wrote:
>> I did some more experimentation on this. Attached is a patch that
>> JUST does #1, and, ...
> ...and that was the wrong patch. Thanks to Heikki for point that out.
> Second try.
But the results you gave in the previo
Peter Eisentraut writes:
> On 9/23/14 10:29 AM, Tom Lane wrote:
>> Votes or no votes, that's a horrible idea; it breaks the design goal
>> that users shouldn't need to remember the precise unit size when making
>> postgresql.conf entries.
> I think this is not historically correct.
> The origina
23.09.2014, 15:18, Andres Freund kirjoitti:
On 2014-09-23 13:50:28 +0300, Oskari Saarenmaa wrote:
23.09.2014, 00:01, Andres Freund kirjoitti:
The patches:
0001: The actual atomics API
I tried building PG on Solaris 10/Sparc using GCC 4.9.0 (buildfarm animal
dingo) with this patch but regressi
Thanks for reviewing this. I will spend some time looking at your
recommendations in detail and seeing what it would take to
implement them, and whether I agree that is better; but I wanted to
point out a couple things regarding the SPI interface where I'm not
sure you realize what's currently bei
On 9/23/14 1:13 AM, Stephen Frost wrote:
> To clarify- we'll simply swap from (essentially) floor() to ceil() for
> handling all GUC_with_unit to internal_unit conversions, document that,
> and note it in the release notes as a possible behavior change and move
> on.
That'll probably work, as long
On 9/23/14 10:29 AM, Tom Lane wrote:
> Votes or no votes, that's a horrible idea; it breaks the design goal
> that users shouldn't need to remember the precise unit size when making
> postgresql.conf entries.
I think this is not historically correct.
The original motivation was that you had to re
On Tue, Sep 23, 2014 at 4:29 PM, Robert Haas wrote:
> I did some more experimentation on this. Attached is a patch that
> JUST does #1, and, ...
...and that was the wrong patch. Thanks to Heikki for point that out.
Second try.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Ente
On Tue, Sep 23, 2014 at 3:05 PM, Greg Stark wrote:
> Fwiw I agree with TL2. The simplest, least surprising behaviour to explain
> to users is to say we round to nearest and if that means we rounded to zero
> (or another special value) we throw an error. We could list the minimum
> value in pg_set
All,
Robert and I had a great discussion on the phone where we were both
able to voice our concerns and feelings about RLS being pushed. By
way of summary, we agree that it was pushed ahead of its time and that
it should have waited for at least another review, which likely would
have h
On Tue, Sep 23, 2014 at 10:55 AM, Robert Haas wrote:
> But this gets at another point: the way we're benchmarking this right
> now, we're really conflating the effects of three different things:
>
> 1. Changing the locking regimen around the freelist and clocksweep.
> 2. Adding a bgreclaimer proce
Re: Tom Lane 2014-09-23 <15155.1411493...@sss.pgh.pa.us>
> Robert Haas writes:
> > On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg wrote:
> >> Can we have "EXPLAIN (timing off)" in 9.4+ hide the "Planning time"
> >> line? That would even be backwards compatible with 9.x where it would
> >> be a n
On 09/22/2014 08:23 PM, Amit Kapila wrote:
> Who decides if the patch is adequately reviewed?
>
> Author, Committer or Reviewer? In CF, that is comparatively clear
> that once Reviewer is satisfied, he marks the patch as
> Ready For Committer and then Committer picks up and if he is satisfied
> w
On 09/23/2014 08:51 PM, Tom Lane wrote:
Robert Haas writes:
On Tue, Sep 23, 2014 at 12:46 PM, Heikki Linnakangas
wrote:
Now, how do we make the tuplestores work similarly? Here's what I think we
should do:
Add a new p_tableref_hook function pointer, similar to p_paramref_hook.
Whenever the p
On 09/23/2014 09:15 PM, Fabien COELHO wrote:
So I'm inclined to reject rather than put in something that may cause
surprises down the road. The usefulness doesn't seem great enough to
take that risk.
Marked as "Returned with feedback".
If you reject it, you can also remove the gaussian and e
Fwiw I agree with TL2. The simplest, least surprising behaviour to explain
to users is to say we round to nearest and if that means we rounded to zero
(or another special value) we throw an error. We could list the minimum
value in pg_settings and maybe in documentation.
--
greg
Alvaro Herrera wrote:
> Heikki Linnakangas wrote:
> > If you add a new datatype, and define b-tree operators for it, what
> > is required to create a minmax opclass for it? Would it be possible
> > to generalize the functions in brin_minmax.c so that they can be
> > reused for any datatype (with
On 9/23/14, 9:00 AM, Andres Freund wrote:
I also think committers need to be much more careful when committing
patches which they (or their employer) appear to have a business
interest in. Rushing ahead to commit the patch of somebody 'unrelated'
leaves a completely different taste than committ
Hi,
I've previously started two threads about replication identifiers. Check
http://archives.postgresql.org/message-id/20131114172632.GE7522%40alap2.anarazel.de
and
http://archives.postgresql.org/message-id/20131211153833.GB25227%40awork2.anarazel.de
.
The've also been discussed in the course of
So my opinion is that this small modulo operator patch is both useful and
harmless, so it should be committed.
You've really failed to make that case --- in particular, AFAICS there is
not even consensus on the exact semantics that the operator should have.
There is. Basically whatever with
On Tue, Sep 23, 2014 at 1:32 PM, Tom Lane wrote:
> Robert Haas writes:
>> On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg wrote:
>>> Can we have "EXPLAIN (timing off)" in 9.4+ hide the "Planning time"
>>> line? That would even be backwards compatible with 9.x where it would
>>> be a no-op.
>
>>
Robert Haas writes:
> On Tue, Sep 23, 2014 at 12:46 PM, Heikki Linnakangas
> wrote:
>> Now, how do we make the tuplestores work similarly? Here's what I think we
>> should do:
>>
>> Add a new p_tableref_hook function pointer, similar to p_paramref_hook.
>> Whenever the parser sees a RangeVar tha
Robert Haas writes:
> On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg wrote:
>> Can we have "EXPLAIN (timing off)" in 9.4+ hide the "Planning time"
>> line? That would even be backwards compatible with 9.x where it would
>> be a no-op.
> I don't think that'll work becuase:
> /* check th
* Ants Aasma:
> CRC has exactly one hardware implementation in general purpose CPU's
I'm pretty sure that's not true. Many general purpose CPUs have CRC
circuity, and there must be some which also expose them as
instructions.
> and Intel has a patent on the techniques they used to implement
> i
On Tue, Sep 23, 2014 at 12:46 PM, Heikki Linnakangas
wrote:
> On 09/15/2014 05:25 PM, Kevin Grittner wrote:
> Now, how do we make the tuplestores work similarly? Here's what I think we
> should do:
>
> Add a new p_tableref_hook function pointer, similar to p_paramref_hook.
> Whenever the parser se
On 09/15/2014 05:25 PM, Kevin Grittner wrote:
Tom Lane wrote:
Heikki Linnakangas writes:
On 08/30/2014 12:15 AM, Kevin Grittner wrote:
If we were to go with the hooks as you propose, we would still need
to take the information from TriggerData and put it somewhere else
for the hook to refer
On Sat, Sep 20, 2014 at 4:13 PM, Christoph Berg wrote:
> there's another problem in this area: 9.4 adds "Planning time" to the
> EXPLAIN output. If you don't want to see that, you need to use (costs
> off), but this, well, also disables the costs. If you are running
> regression tests to actually
Hi all,
I'm faced with some troubles about the jsonb implementation, and I hope
I'll get little advice =)
If I understand correctly, an abstract function for jsonb modification
should have the following stages:
Jsonb -> JsonbValue -> Modification -> JsonbValue -> Jsonb
One can convert the *Jsonb
On 09/23/2014 06:23 PM, Andrew Dunstan wrote:
On 09/23/2014 10:29 AM, Tom Lane wrote:
Robert Haas writes:
Three people have voted for making it an *error* to supply a value
that needs to be rounded, instead of changing the rounding behavior.
Votes or no votes, that's a horrible idea; it brea
On Tue, Sep 23, 2014 at 1:30 AM, Tom Lane wrote:
> David Johnston writes:
> > My original concern was things that are rounded to zero now will not be
> in
> > 9.5 if the non-error solution is chosen. The risk profile is extremely
> > small but it is not theoretically zero.
>
> This is exactly t
On Tue, Sep 23, 2014 at 11:23 AM, Dave Page wrote:
> On Tue, Sep 23, 2014 at 4:19 PM, Robert Haas wrote:
>> On Tue, Sep 23, 2014 at 11:16 AM, Dave Page wrote:
>>> Regardless of what Robert may feel, review should only generally be
>>> *expected* during a commitfest, but it can be done at any tim
On 09/23/2014 11:19 AM, Robert Haas wrote:
On Tue, Sep 23, 2014 at 11:16 AM, Dave Page wrote:
Regardless of what Robert may feel, review should only generally be
*expected* during a commitfest, but it can be done at any time.
Committers are free to commit at any time. The process was never
int
On 2014-09-23 16:16:18 +0100, Dave Page wrote:
> On Tue, Sep 23, 2014 at 2:00 PM, Andres Freund wrote:
> > On 2014-09-23 13:23:32 +0100, Dave Page wrote:
> >> Just to be clear here, the *only* issue we should even be discussing
> >> is whether the patch should or should not have been committed in
On 09/23/2014 10:29 AM, Tom Lane wrote:
Robert Haas writes:
Three people have voted for making it an *error* to supply a value
that needs to be rounded, instead of changing the rounding behavior.
Votes or no votes, that's a horrible idea; it breaks the design goal
that users shouldn't need to
On Tue, Sep 23, 2014 at 4:19 PM, Robert Haas wrote:
> On Tue, Sep 23, 2014 at 11:16 AM, Dave Page wrote:
>> Regardless of what Robert may feel, review should only generally be
>> *expected* during a commitfest, but it can be done at any time.
>> Committers are free to commit at any time. The proc
On Tue, Sep 23, 2014 at 11:16 AM, Dave Page wrote:
> Regardless of what Robert may feel, review should only generally be
> *expected* during a commitfest, but it can be done at any time.
> Committers are free to commit at any time. The process was never
> intended to restrict what committers do or
On Tue, Sep 23, 2014 at 2:00 PM, Andres Freund wrote:
> On 2014-09-23 13:23:32 +0100, Dave Page wrote:
>> Just to be clear here, the *only* issue we should even be discussing
>> is whether the patch should or should not have been committed in the
>> face of those objections. As Josh has also noted
On Tue, Sep 23, 2014 at 10:31 AM, Robert Haas wrote:
> [ review ]
Oh, by the way, I noticed that this patch breaks pg_buffercache. If
we're going to have 128 lock partitions, we need to bump
MAX_SIMUL_LWLOCKS.
But this gets at another point: the way we're benchmarking this right
now, we're real
Hi,
On 2014-09-23 10:31:24 -0400, Robert Haas wrote:
> I suggest we count these things:
>
> 1. The number of buffers the reclaimer has put back on the free list.
> 2. The number of times a backend has run the clocksweep.
> 3. The number of buffers past which the reclaimer has advanced the clock
>
On Tue, Sep 23, 2014 at 10:29 AM, Tom Lane wrote:
> Robert Haas writes:
>> Three people have voted for making it an *error* to supply a value
>> that needs to be rounded, instead of changing the rounding behavior.
>
> Votes or no votes, that's a horrible idea; it breaks the design goal
> that use
On Fri, Sep 19, 2014 at 7:21 AM, Amit Kapila
wrote:
> Specific numbers of both the configurations for which I have
> posted data in previous mail are as follows:
>
> Scale Factor - 800
> Shared_Buffers - 12286MB (Total db size is 12288MB)
> Client and Thread Count = 64
> buffers_touched_freelist
Robert Haas writes:
> Three people have voted for making it an *error* to supply a value
> that needs to be rounded, instead of changing the rounding behavior.
Votes or no votes, that's a horrible idea; it breaks the design goal
that users shouldn't need to remember the precise unit size when mak
On Tue, Sep 23, 2014 at 1:23 AM, Stephen Frost wrote:
> * Tom Lane (t...@sss.pgh.pa.us) wrote:
>> Stephen Frost writes:
>> > To clarify- we'll simply swap from (essentially) floor() to ceil() for
>> > handling all GUC_with_unit to internal_unit conversions, document that,
>> > and note it in the
Robert,
* Robert Haas (robertmh...@gmail.com) wrote:
> I'd be happy to discuss this with Stephen, either in person, by phone,
> or over public or private email.
Please understand that I'm not ignoring you, the conversation, or the
concern. I was asked (by other community members) to not comment
On Sat, Sep 20, 2014 at 1:16 PM, Michael Paquier
wrote:
> On Fri, Sep 19, 2014 at 12:18 PM, Robert Haas wrote:
>> On Tue, Sep 16, 2014 at 2:19 PM, Michael Paquier
>> wrote:
>>> - A patch refactoring code for pg_stat_get_wal_senders and
>>> SyncRepReleaseWaiters as there is in either case duplica
* Tom Lane (t...@sss.pgh.pa.us) wrote:
> Fabien COELHO writes:
> > So my opinion is that this small modulo operator patch is both useful and
> > harmless, so it should be committed.
>
> You've really failed to make that case --- in particular, AFAICS there is
> not even consensus on the exact se
* Tom Lane (t...@sss.pgh.pa.us) wrote:
> Stephen Frost writes:
> > To be a bit more clear- why is it safe to change the contents if the
> > equal() function returns 'false'? I'm guessing the answer is
> > "locking"- whereby such a change would imply a lock was acquired on
> > the relation
Fabien COELHO writes:
> So my opinion is that this small modulo operator patch is both useful and
> harmless, so it should be committed.
You've really failed to make that case --- in particular, AFAICS there is
not even consensus on the exact semantics that the operator should have.
So I'm incli
On Tue, Sep 23, 2014 at 9:14 AM, Robert Haas wrote:
> On Tue, Sep 23, 2014 at 12:38 AM, David G Johnston
> wrote:
> > I'm of a mind to agree that this shouldn't have been committed...but I'm
> not
> > seeing where Stephen has done sufficient wrong to justify crucifixion.
> > Especially since eve
Stephen Frost writes:
> To be a bit more clear- why is it safe to change the contents if the
> equal() function returns 'false'? I'm guessing the answer is
> "locking"- whereby such a change would imply a lock was acquired on
> the relation by another backend, which shouldn't be possible
On Tue, Sep 23, 2014 at 9:09 AM, Andres Freund
wrote:
> On 2014-09-22 21:38:17 -0700, David G Johnston wrote:
> > Robert Haas wrote
> > > It's difficult to imagine a more flagrant violation of process than
> > > committing a patch without any warning and without even *commenting*
> > > on the fac
Hello Stephen,
But this is not convincing. Adding a unary function with a clean
syntax indeed requires doing something with the parser.
Based on the discussion so far, it sounds like you're coming around to
agree with Robert (as I'm leaning towards also) that we'd be better off
building a rea
On Tue, Sep 23, 2014 at 12:38 AM, David G Johnston
wrote:
> I'm of a mind to agree that this shouldn't have been committed...but I'm not
> seeing where Stephen has done sufficient wrong to justify crucifixion.
> Especially since everything is being done publicly and you are one of the
> many peopl
Stephen Frost wrote:
> pgstat_get_backend_current_activity()
> Used to decide if the current activity string should be returned or
> not. In my view, this is a clear case which should be addressed
> through has_privs_of_role() instead of requiring the user to
> SET ROLE to each
On Tue, Sep 23, 2014 at 9:00 AM, Andres Freund wrote:
> I think it's obvious that a committer doesn't need to wait till some
> later commitfest to commit patches that have since gotten enough review
> or are uncontroversial. Neither is the case here.
Right. I mean, the occasionally-floated notio
On 2014-09-22 21:38:17 -0700, David G Johnston wrote:
> Robert Haas wrote
> > It's difficult to imagine a more flagrant violation of process than
> > committing a patch without any warning and without even *commenting*
> > on the fact that clear objections to commit were made on a public
> > mailin
On 2014-09-23 13:23:32 +0100, Dave Page wrote:
> Just to be clear here, the *only* issue we should even be discussing
> is whether the patch should or should not have been committed in the
> face of those objections. As Josh has also noted, the commitfest
> process was never meant to constrain what
Hi,
On 2014-09-23 14:47:33 +0200, didier wrote:
> Currently the value is hard code to NBuffers / 4 but ISTM that with
> bigger shared_buffer it's too much, ie even with a DB 10 to 20 time
> the memory size there's a lot of tables under this limit and nightly
> batch reports are trashing the shared
Hi,
Currently the value is hard code to NBuffers / 4 but ISTM that with
bigger shared_buffer it's too much, ie even with a DB 10 to 20 time
the memory size there's a lot of tables under this limit and nightly
batch reports are trashing the shared buffers cache as if there's no
tomorrow.
regards,
On 22 September 2014 19:17, Heikki Linnakangas wrote:
> On 09/22/2014 04:45 PM, Tom Lane wrote:
> > Heikki Linnakangas writes:
> >> On 09/22/2014 07:47 AM, Rajeev rastogi wrote:
> >>> So my proposal is to skip the condition check on the first scan key
> condition for every tuple.
> >
> >> The sa
On Tue, Sep 23, 2014 at 4:25 AM, Robert Haas wrote:
> On Mon, Sep 22, 2014 at 7:22 PM, Peter Geoghegan wrote:
>> On Mon, Sep 22, 2014 at 4:02 PM, Andres Freund wrote:
>>> This patch has been pushed in a clear violation of established policy.
>>>
>>> Fundamental pieces of the patch have changed *
Heikki,
* Heikki Linnakangas (hlinnakan...@vmware.com) wrote:
> Some random comments after a quick read-through of the patch:
Glad you were able to find a bit of time to take a look, thanks!
> * Missing documentation. For a major feature like this, reference
> pages for the CREATE/DROP POLICY st
Hi,
On 2014-09-23 13:50:28 +0300, Oskari Saarenmaa wrote:
> 23.09.2014, 00:01, Andres Freund kirjoitti:
> >I've finally managed to incorporate (all the?) feedback I got for
> >0.5. Imo the current version looks pretty good.
> >
> >Most notable changes:
> >* Lots of comment improvements
> >* code m
On 2014-09-23 13:50:28 +0300, Oskari Saarenmaa wrote:
> 23.09.2014, 00:01, Andres Freund kirjoitti:
> >I've finally managed to incorporate (all the?) feedback I got for
> >0.5. Imo the current version looks pretty good.
> >
> >Most notable changes:
> >* Lots of comment improvements
> >* code moved
23.09.2014, 00:01, Andres Freund kirjoitti:
I've finally managed to incorporate (all the?) feedback I got for
0.5. Imo the current version looks pretty good.
Most notable changes:
* Lots of comment improvements
* code moved out of storage/ into port/
* separated the s_lock.h changes into its own
Some random comments after a quick read-through of the patch:
* Missing documentation. For a major feature like this, reference pages
for the CREATE/DROP POLICY statements are not sufficient. We'll need a
whole new chapter for this.
* In CREATE POLICY, the "USING" and "WITH CHECK" keywords ar
1 - 100 of 103 matches
Mail list logo