Re: TODO: Split out pg_resetxlog output into pre- and post-sections - Mailing list pgsql-hackers

On Thu, Nov 28, 2013 at 12:11 PM, Rajeev rastogi
<rajeev.rastogi@huawei.com> wrote:
> On 28 November 2013, Amit Kapila Wrote:
>> >> Further Review of this patch:
>> >> b. why to display 'First log segment after reset' in 'Currrent
>> >> pg_control values' section as now the same information
>> >>     is available in new section "Values to be used after reset:" ?
>> >
>> > May not be always. Suppose if user has given new value of seg no and
>> TLI, then it will be different.
>> > Otherwise it will be same.
>> > So now I have changed the code in such way that the value of XLOG
>> > segment # read from checkpoint record gets printed as part of current
>> > value and any further new value gets printed in Values to be reset
>> > (This will be always at-least one plus the current value). Because of
>> following code in FindEndOfXLOG
>> >                         xlogbytepos = newXlogSegNo *
>> ControlFile.xlog_seg_size;
>> >                         newXlogSegNo = (xlogbytepos + XLogSegSize - 1)
>> / XLogSegSize;
>> >                         newXlogSegNo++;
>>
>> It can be different, but I don't think we should display it in both
>> sections because:
>> a. it doesn't belong to control file.
>> b. if you carefully look at original link
>> (http://www.postgresql.org/message-id/1283277511-sup-2152@alvh.no-
>> ip.org),
>>     these values are not getting displayed in pg_control values section.
>>
>> So I suggest it is better to remove it from pg_control values section.
>
> Done as per suggestion.

I have done few more cosmetic changes in your patch, please find the
updated patch attached with this mail.
Kindly check once whether changes are okay.

With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com