Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-06 Thread Robert Haas
On Sat, Mar 4, 2023 at 3:33 AM Drouvot, Bertrand wrote: > Indeed, for gistxlogDelete, that's the other way around (as > compare to what the commit message says). Woops. Good point. -- Robert Haas EDB: http://www.enterprisedb.com

Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-04 Thread Drouvot, Bertrand
Hi, On 3/3/23 6:53 PM, Robert Haas wrote: On Fri, Mar 3, 2023 at 11:28 AM Drouvot, Bertrand wrote: Thanks for having looked at it! +1. Committed. Thanks! Not a big deal, but the commit message that has been used is not 100% accurate. Indeed, for gistxlogDelete, that's the other way

Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-03 Thread Robert Haas
On Fri, Mar 3, 2023 at 11:28 AM Drouvot, Bertrand wrote: > Thanks for having looked at it! +1. Committed. -- Robert Haas EDB: http://www.enterprisedb.com

Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-03 Thread Drouvot, Bertrand
Hi, On 3/3/23 12:30 PM, Amit Kapila wrote: On Thu, Mar 2, 2023 at 6:35 PM Drouvot, Bertrand wrote: On 1/6/23 11:05 AM, Drouvot, Bertrand wrote: Hi hackers, Please find attached a patch to $SUBJECT. The wrong comments have been discovered by Robert in [1]. Submitting this here as a

Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-03 Thread Amit Kapila
On Thu, Mar 2, 2023 at 6:35 PM Drouvot, Bertrand wrote: > > On 1/6/23 11:05 AM, Drouvot, Bertrand wrote: > > Hi hackers, > > > > Please find attached a patch to $SUBJECT. > > > > The wrong comments have been discovered by Robert in [1]. > > > > Submitting this here as a separate thread so it does

Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-03-02 Thread Drouvot, Bertrand
: [PATCH v2] Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete gistxlogDelete claims that OffsetNumbers are in payload of blk 0, which is not as they follow the main payload. xl_heap_freeze_page claims that freeze plans and offset numbers follow, but they don't: they're

Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete

2023-01-06 Thread Drouvot, Bertrand
Hi hackers, Please find attached a patch to $SUBJECT. The wrong comments have been discovered by Robert in [1]. Submitting this here as a separate thread so it does not get lost in the logical decoding on standby thread. [1]: