On Friday, November 23, 2012 10:10 PM Fujii Masao wrote:
On Fri, Nov 23, 2012 at 6:56 PM, Amit Kapila wrote:
>> When I remove postgresql.auto.conf, SET PERSISTENT failed.
>>
>>> =# SET PERSISTENT synchronous_commit = 'local';
>>> ERROR: failed to open "postgresql.auto.conf" file
>
>> There can be
On Wed, Nov 14, 2012 at 06:28:41PM -0500, Bruce Momjian wrote:
> On Wed, Nov 14, 2012 at 06:15:30PM -0500, Tom Lane wrote:
> > Bruce Momjian writes:
> > > On Wed, Nov 14, 2012 at 05:39:29PM -0500, Tom Lane wrote:
> > >> You would have been better off keeping the array and sorting it so you
> > >>
On Mon, Nov 19, 2012 at 12:11:26PM -0800, Jeff Janes wrote:
[ Sorry for the delay in replying.]
> On Wed, Nov 14, 2012 at 3:55 PM, Bruce Momjian wrote:
> > On Mon, Nov 12, 2012 at 10:29:39AM -0800, Jeff Janes wrote:
> >>
> >> Is turning off synchronous_commit enough? What about turning off fsy
Alvaro Herrera writes:
> So incorporating these ideas, the layout now looks like this:
> ./commands/seq_desc.c
> ./commands/dbase_desc.c
> ./commands/tablespace_desc.c
> ./catalog/storage_desc.c
> ./utils/cache/relmap_desc.c
> ./access/rmgrdesc/mxact_desc.c
> ./access/rmgrdesc/spgdesc.c
> ./acces
On 23 November 2012 22:52, Alvaro Herrera wrote:
> ./access/rmgrdesc/xact_desc.c
> ./access/rmgrdesc/heapdesc.c
Could we have either all underscores _ or none at all for the naming?
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training &
On Thu, Nov 15, 2012 at 7:05 PM, Jeff Janes wrote:
> On Wed, Nov 14, 2012 at 11:49 AM, Tom Lane wrote:
>> Jeff Janes writes:
>>> On Thu, Nov 8, 2012 at 9:50 PM, Tom Lane wrote:
There are at least three ways we could whack that mole: ...
* Keep a separate list (or data structure o
Robert Haas escribió:
> On Thu, Oct 25, 2012 at 4:25 AM, Simon Riggs wrote:
> > I'd put these in a separate directory to avoid annoyance. Transam is
> > already too large.
>
> +1. A further advantage of that is that it means that that everything
> in that new directory will be intended to end up
Alvaro Herrera wrote:
> Are you posting an updated patch?
Well, there wasn't exactly a consensus on what should change, so I'll
throw some initial review comments out even though I still need to
check some things in the code and do more testing.
The patch applied cleanly, compiled without warnin
On Wed, 2012-11-21 at 11:46 -0800, Jeff Janes wrote:
> Using gcc (GCC) 4.4.6 20120305 (Red Hat 4.4.6-4), I get dozens of
> warnings, all apparently coming from somewhere in the MemSet macro.
OK, reverted.
It looks like they dialed it down to a more useful volume in GCC 4.5,
but that doesn't rea
Pavel Stehule writes:
> * some wrong in deparsing - doesn't support constraints
>
> postgres=# alter table a add column bbbsss int check (bbbsss > 0);
> NOTICE: event: ddl_command_start, context: TOPLEVEL, tag: ALTER
> TABLE, operation: ALTER, type: TABLE, schema: , name:
> NOTICE: command:
>
On Wed, Nov 21, 2012 at 5:48 AM, Karl O. Pinc wrote:
>> On Fri, Sep 21, 2012 at 8:54 AM, Karl O. Pinc wrote:
>> > On 09/20/2012 12:24:49 PM, Karl O. Pinc wrote:
> OT:
> After looking at the code I found a number of "conflicting"
> option combinations are not tested for or rejected. I can't
>
On Friday, November 23, 2012 11:15 AM Pavan Deolasee wrote:
On Thu, Nov 22, 2012 at 2:05 PM, Amit Kapila wrote:
>>>Sorry, I haven't followed this thread at all, but the numbers (43171 and
>>>57920) in the last two runs of @mv-free-list for 32 clients look
>>>aberrations, no ? I wonder if
>>>
On 23.11.2012 19:55, Tom Lane wrote:
Heikki Linnakangas writes:
Committed, thanks.
Doesn't seem to have been pushed to master?
On purpose. Per commit message:
9.3 doesn't have this problem because XLogRecPtr is now a single 64-bit
integer, so XLogRecPtrIsInvalid() does the right thing. Ap
Heikki Linnakangas writes:
> Committed, thanks.
Doesn't seem to have been pushed to master?
regards, tom lane
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
Hello
2012/11/22 Dimitri Fontaine :
> Hi,
>
> Thank you Pavel for reviewing my patch! :)
>
> Pavel Stehule writes:
>> * issues
>>
>> ** missing doc
>
> Yes. I wanted to reach an agreement on why we need the de parsing for.
> You can read the Last comment we made about it at the following link, I
On 09.11.2012 15:28, Fujii Masao wrote:
On Fri, Nov 9, 2012 at 4:06 AM, Alvaro Herrera wrote:
Fujii Masao escribió:
On Fri, Oct 19, 2012 at 10:29 PM, Fujii Masao wrote:
However, I've forgotten to treat other three portions in
walsender.c and syncrep.c also does XLogRecPtrIsInvalid(). This
On Thu, Nov 15, 2012 at 07:05:00PM -0800, Jeff Janes wrote:
> On Wed, Nov 14, 2012 at 11:49 AM, Tom Lane wrote:
> > Jeff Janes writes:
> >> On Thu, Nov 8, 2012 at 9:50 PM, Tom Lane wrote:
> >>> There are at least three ways we could whack that mole: ...
> >>>
> >>> * Keep a separate list (or dat
On Fri, Nov 23, 2012 at 6:56 PM, Amit Kapila wrote:
>> When I remove postgresql.auto.conf, SET PERSISTENT failed.
>>
>> =# SET PERSISTENT synchronous_commit = 'local';
>> ERROR: failed to open "postgresql.auto.conf" file
>
> There can be 2 ways to handle this, either we recreate the
> "postgresql
"Cyril VELTER" writes:
>I follow up on my previous message. Just got two more crash today very
> similar to the first ones :
> PANIC: could not write to log file 118, segment 237 at offset 2686976,
> length 5578752: Invalid argument
> STATEMENT: COMMIT
> PANIC: could not write to log fil
Alvaro Herrera writes:
> If the bgworker developer gets really tense about this stuff (or
> anything at all, really), they can create a completely new sigmask and
> do sigaddset() etc. Since this is all C code, we cannot keep them from
> doing anything, really; I think what we need to provide her
Jan,
Are you posting an updated patch?
--
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/m
Henning,
On 11/23/2012 03:17 PM, "Henning Mälzer" wrote:
> Can somebody help me?
Sure, but you might get better answers on the -hackers mailing list. I'm
redirecting there. The cluster-hackers one is pretty low volume and low
subscribers, I think.
> Question:
> What would be the loss if i cut NO
Merlin Moncure wrote:
> Kevin Grittner wrote:
>> Merlin Moncure wrote:
>>
>>> Hm, I bet it's possible (although probably not easy) to deduce
>>> volatility from the function body...maybe through the validator.
>>> If you could do that (perhaps warning in cases where you can't),
>>> then the perfo
On Sat, Nov 17, 2012 at 2:48 AM, Phil Sorber wrote:
> On Thu, Nov 15, 2012 at 10:55 PM, Michael Paquier
> wrote:
> > On Fri, Nov 16, 2012 at 12:34 PM, Phil Sorber wrote:
> >> On Thu, Nov 15, 2012 at 9:23 PM, Michael Paquier
> >> wrote:
> >> > 3) Having an output close to what ping actually doe
On Thu, Nov 22, 2012 at 06:09:20PM +0800, Chen Huajun wrote:
> When use a struct variable whose name length is very very long such as 12KB
> in .pgc source,
> ecpg will core dump because of buffer overflow if precompile the .pgc file.
How on earth did you run into this? :)
I absolutely agree tha
On 15.11.2012 17:16, Heikki Linnakangas wrote:
On 15.11.2012 16:55, Tom Lane wrote:
Heikki Linnakangas writes:
This is a fairly general issue, actually. Looking around, I can see at
least two similar cases in existing code, with BasicOpenFile, where we
will leak file descriptors on error:
Um,
>>Shouldn't that data be in the shared buffers if not the OS cache and hence
approximately same IO will be required?
>I don't think so as the data in OS cache or PG Shared buffers doesn't have
any direct relation, >OS can flush its buffers based on its scheduler
algorithm.
>Let us try to se
On Fri, Nov 23, 2012 at 6:30 AM, Fujii Masao wrote:
> On Thu, Nov 22, 2012 at 10:05 PM, Boszormenyi Zoltan wrote:
>> 2012-11-22 12:44 keltezéssel, Magnus Hagander írta:
Also, a question was buried in the other review which is - are we OK
to remove the requiressl parameter. Both
Hello
2012/11/23 Asif Rehman :
> Hi,
>
> I forgot to add documentation changes in the earlier patch. In the attached
> patch, I have added documentation as well as fixed other issues you raised.
>
ok
so
* applied and compiled cleanly
* All 133 tests passed.
* there are doc
* there are necessary
On Thursday, November 22, 2012 10:09 PM Fujii Masao wrote:
> On Thu, Nov 22, 2012 at 9:38 PM, Amit kapila
> wrote:
> > Patch to implement SET PERSISTENT command is attached with this mail.
> > Now it can be reviewed.
>
> I got the following compile warnings:
> xact.c:1847: warning: implicit decla
2012-11-23 06:30 keltezéssel, Fujii Masao írta:
On Thu, Nov 22, 2012 at 10:05 PM, Boszormenyi Zoltan wrote:
2012-11-22 12:44 keltezéssel, Magnus Hagander írta:
Also, a question was buried in the other review which is - are we OK
to remove the requiressl parameter. Both these patches do so, bec
Hi,
I forgot to add documentation changes in the earlier patch. In the attached
patch, I have added documentation as well as fixed other issues you raised.
Regards,
--Asif
On Thu, Nov 22, 2012 at 10:47 PM, Asif Rehman wrote:
> Thanks for the review Pavel. I have taken care of the points you rai
32 matches
Mail list logo