On Thu, Sep 14, 2017 at 11:00 PM, Craig Ringer wrote:
> On 14 September 2017 at 19:57, Ashutosh Sharma
> wrote:
>
>>
>>
>> Are you planning to work on the review comments from Robert, Moon
>> Insung and supply the new patch. I just had a quick glance into this
>> mail thread (after a long time) a
On Sat, Oct 14, 2017 at 2:47 PM, Peter Geoghegan wrote:
> I am working on an experimental version of pg_filedump, customized to
> output XML that can be interpreted by an open source hex editor. The
> XML makes the hex editor produce color coded, commented
> tags/annotations for any given heap or
On 10/14/2017 11:47 PM, Peter Geoghegan wrote:
> On Sat, Oct 14, 2017 at 10:58 AM, Robert Haas wrote:
>> I think it's perfectly sensible to view those 2 bits as making up a
>> 2-bit field with 4 states rather than displaying each bit
>> individually, but you obviously disagree. Fair enough.>
> I
On Sat, Oct 14, 2017 at 10:58 AM, Robert Haas wrote:
> I think it's perfectly sensible to view those 2 bits as making up a
> 2-bit field with 4 states rather than displaying each bit
> individually, but you obviously disagree. Fair enough.
I guess it is that simple.
> I can think of two possibl
On Fri, Oct 13, 2017 at 4:36 PM, Peter Geoghegan wrote:
> No, I'm arguing that they're just bits. Show the bits, rather than
> interpreting what is displayed. Document that there are other logical
> states that are represented as composites of contradictory/mutually
> exclusive states.
/me shrugs
On Fri, Oct 13, 2017 at 1:02 PM, Robert Haas wrote:
>> I don't think it's our place to "interpret" the bits. Are we *also*
>> going to show HEAP_XMIN_FROZEN when xmin is physically set to
>> FrozenTransactionId?
>
> No, of course not. We're talking about how to display the 256 and 512
> bits of t
On Thu, Oct 12, 2017 at 7:35 PM, Peter Geoghegan wrote:
> On Tue, Aug 15, 2017 at 10:54 AM, Robert Haas wrote:
>>> Or at least make the filtering optional.
>>
>> I don't think "filtering" is the right way to think about it. It's
>> just labeling each combination of bits with the meaning appropri
On Tue, Aug 15, 2017 at 10:54 AM, Robert Haas wrote:
>> Or at least make the filtering optional.
>
> I don't think "filtering" is the right way to think about it. It's
> just labeling each combination of bits with the meaning appropriate to
> that combination of bits.
I do. -1 to not just showin
On 14 September 2017 at 19:57, Ashutosh Sharma
wrote:
>
> Are you planning to work on the review comments from Robert, Moon
> Insung and supply the new patch. I just had a quick glance into this
> mail thread (after a long time) and could understand Robert's concern
> till some extent. I think,
Hi Craig,
On Thu, Aug 17, 2017 at 5:50 AM, Craig Ringer wrote:
> On 16 August 2017 at 23:14, Robert Haas wrote:
>>
>> On Tue, Aug 15, 2017 at 4:36 PM, Tomas Vondra
>> wrote:
>> > You might say that people investigating issues in this area of code
>> > should
>> > be aware of how HEAP_XMIN_FROZE
On 16 August 2017 at 23:14, Robert Haas wrote:
> On Tue, Aug 15, 2017 at 4:36 PM, Tomas Vondra
> wrote:
> > You might say that people investigating issues in this area of code
> should
> > be aware of how HEAP_XMIN_FROZEN is defined, and perhaps you're right ...
>
> Yes, I think that's what I wo
On Tue, Aug 15, 2017 at 4:36 PM, Tomas Vondra
wrote:
> You might say that people investigating issues in this area of code should
> be aware of how HEAP_XMIN_FROZEN is defined, and perhaps you're right ...
Yes, I think that's what I would say. I mean, if you happen to NOT
know that committed|inv
On 16 August 2017 at 03:42, Tomas Vondra
wrote:
>
>
> On 08/15/2017 07:54 PM, Robert Haas wrote:
>
>> On Tue, Aug 15, 2017 at 9:59 AM, Tomas Vondra
>> wrote:
>>
>>> I don't think so -- the "committed" and "invalid" meanings are
effectively canceled when the "frozen" mask is present.
>>
mation that is presented to the user should inform us
which flags was entered.
Regards.
Moon
> -Original Message-
> From: pgsql-hackers-ow...@postgresql.org
> [mailto:pgsql-hackers-ow...@postgresql.org] On Behalf Of Tomas Vondra
> Sent: Wednesday, August 16, 2017 5:36 AM
> To
On 08/15/2017 09:55 PM, Robert Haas wrote:
On Tue, Aug 15, 2017 at 3:42 PM, Tomas Vondra
wrote:
What I think we should not do is interpret the bitmasks (omitting some of
the information) assuming all the bits were set correctly.
I'm still confused. HEAP_XMIN_COMMITTED|HEAP_XMIN_ABORTED ==
On Tue, Aug 15, 2017 at 3:42 PM, Tomas Vondra
wrote:
> What I think we should not do is interpret the bitmasks (omitting some of
> the information) assuming all the bits were set correctly.
I'm still confused. HEAP_XMIN_COMMITTED|HEAP_XMIN_ABORTED ==
HEAP_XMIN_FROZEN. Nobody is proposing to omi
On 08/15/2017 07:54 PM, Robert Haas wrote:
On Tue, Aug 15, 2017 at 9:59 AM, Tomas Vondra
wrote:
I don't think so -- the "committed" and "invalid" meanings are
effectively canceled when the "frozen" mask is present.
I mean, "committed" and "invalid" contradict each other...
FWIW I agree wit
On Tue, Aug 15, 2017 at 9:59 AM, Tomas Vondra
wrote:
>> I don't think so -- the "committed" and "invalid" meanings are
>> effectively canceled when the "frozen" mask is present.
>>
>> I mean, "committed" and "invalid" contradict each other...
>
> FWIW I agree with Craig - the functions should outp
On Tue, Aug 15, 2017 at 10:59 PM, Tomas Vondra
wrote:
>
>
> On 08/15/2017 03:24 PM, Robert Haas wrote:
>>
>> On Mon, Aug 14, 2017 at 9:59 PM, Craig Ringer
>> wrote:
>>>
>>> The bits are set, those macros just test to exclude the special meaning
>>> of
>>> both bits being set at once to mean "froz
On 08/15/2017 03:24 PM, Robert Haas wrote:
On Mon, Aug 14, 2017 at 9:59 PM, Craig Ringer wrote:
The bits are set, those macros just test to exclude the special meaning of
both bits being set at once to mean "frozen".
I was reluctant to filter out HEAP_XMIN_COMMITTED and HEAP_XMIN_INVALID
wh
On Mon, Aug 14, 2017 at 9:59 PM, Craig Ringer wrote:
> The bits are set, those macros just test to exclude the special meaning of
> both bits being set at once to mean "frozen".
>
> I was reluctant to filter out HEAP_XMIN_COMMITTED and HEAP_XMIN_INVALID
> when we detect that it's frozen, because
On 15 August 2017 at 09:11, Moon Insung
wrote:
> Dear Craig Ringer
>
>
>
> Frist, thank you for implementing the necessary function.
>
>
>
> but, i have some question.
>
>
>
> question 1) vacuum freeze hint bits
>
> If run a vacuum freeze, bits in the infomask will be 0x0300.
>
> in this case, if
ouhaud; Pavan Deolasee; Álvaro Herrera; Peter
Eisentraut; Masahiko Sawada; abhijit Menon-Sen; Peter Geoghegan
Subject: Re: [HACKERS] [PATCH] pageinspect function to decode infomasks
On 20 Jul. 2017 19:09, "Ashutosh Sharma" mailto:ashu.coe...@gmail.com> > wrote:
I had
On 20 July 2017 at 19:09, Ashutosh Sharma wrote:
> I had a quick look into this patch and also tested it and following
> are my observations.
>
> 1) I am seeing a server crash when passing any non meaningful value
> for t_infomask2 to heap_infomask_flags().
>
> postgres=# SELECT heap_infomask_fla
On 20 Jul. 2017 19:09, "Ashutosh Sharma" wrote:
I had a quick look into this patch and also tested it and following
are my observations.
Thanks very much.
I'll expand the tests to cover various normal and nonsensical masks and
combinations and fix the identified issues.
This was a quick morni
I had a quick look into this patch and also tested it and following
are my observations.
1) I am seeing a server crash when passing any non meaningful value
for t_infomask2 to heap_infomask_flags().
postgres=# SELECT heap_infomask_flags(2816, 3);
server closed the connection unexpectedly
This
On Thu, Jul 20, 2017 at 3:13 PM, Julien Rouhaud wrote:
> On Thu, Jul 20, 2017 at 5:44 AM, Peter Geoghegan wrote:
>> On Wed, Jul 19, 2017 at 8:33 PM, Craig Ringer wrote:
>>> That's silly, so here's a patch to teach pageinspect how to decode infomasks
>>> to a human readable array of flag names.
>
On Thu, Jul 20, 2017 at 5:44 AM, Peter Geoghegan wrote:
> On Wed, Jul 19, 2017 at 8:33 PM, Craig Ringer wrote:
>> That's silly, so here's a patch to teach pageinspect how to decode infomasks
>> to a human readable array of flag names.
>>
>> Example:
>>
>> SELECT t_infomask, t_infomask2, flags
>>
On Wed, Jul 19, 2017 at 8:33 PM, Craig Ringer wrote:
> That's silly, so here's a patch to teach pageinspect how to decode infomasks
> to a human readable array of flag names.
>
> Example:
>
> SELECT t_infomask, t_infomask2, flags
> FROM heap_page_items(get_raw_page('test1', 0)),
> LATERAL hea
On 20 July 2017 at 11:33, Craig Ringer wrote:
> Hi
>
> Whenever I'm debugging some kind of corruption incident, possible
> visibility bug, etc, I always land up staring at integer infomasks or using
> a SQL helper function to decode them.
>
> That's silly, so here's a patch to teach pageinspect h
30 matches
Mail list logo