Re: [HACKERS] show xl_prev in xlog.c errcontext

2016-04-04 Thread Michael Paquier
On Tue, Apr 5, 2016 at 6:14 AM, Alvaro Herrera  wrote:
> Michael Paquier wrote:
>> And both?
>
> I couldn't see much point of doing this, so I didn't.  If you have a
> rationale for it, let's hear it.

Now that I think on it, it does not actually matter to print both.
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] show xl_prev in xlog.c errcontext

2016-04-04 Thread Alvaro Herrera
Michael Paquier wrote:
> On Thu, Apr 16, 2015 at 3:25 PM, Heikki Linnakangas  wrote:
> > On 04/15/2015 11:35 PM, Alvaro Herrera wrote:
> >>
> >> I found this patch in my local repo that I wrote some weeks or months
> >> ago while debugging some XLog corruption problem: it was difficult to
> >> pinpoint what XLog record in a long sequence of WAL files was causing a
> >> problem, and the displaying the prev pointer in errcontext made finding
> >> it much easier -- I could correlate it with pg_xlogdump output, I think.
> >
> > Seems like a good idea, but why print the prev pointer, and not the location
> > of the record itself?

Makes sense -- pushed that way.

> And both?

I couldn't see much point of doing this, so I didn't.  If you have a
rationale for it, let's hear it.

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] show xl_prev in xlog.c errcontext

2015-04-15 Thread Michael Paquier
On Thu, Apr 16, 2015 at 3:25 PM, Heikki Linnakangas  wrote:
> On 04/15/2015 11:35 PM, Alvaro Herrera wrote:
>>
>> I found this patch in my local repo that I wrote some weeks or months
>> ago while debugging some XLog corruption problem: it was difficult to
>> pinpoint what XLog record in a long sequence of WAL files was causing a
>> problem, and the displaying the prev pointer in errcontext made finding
>> it much easier -- I could correlate it with pg_xlogdump output, I think.
>
>
> Seems like a good idea, but why print the prev pointer, and not the location
> of the record itself?

And both?
-- 
Michael


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] show xl_prev in xlog.c errcontext

2015-04-15 Thread Heikki Linnakangas

On 04/15/2015 11:35 PM, Alvaro Herrera wrote:

I found this patch in my local repo that I wrote some weeks or months
ago while debugging some XLog corruption problem: it was difficult to
pinpoint what XLog record in a long sequence of WAL files was causing a
problem, and the displaying the prev pointer in errcontext made finding
it much easier -- I could correlate it with pg_xlogdump output, I think.


Seems like a good idea, but why print the prev pointer, and not the 
location of the record itself?


- Heikki


--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers