Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pY8H7-0003yP-J2 for pgsql-hackers@arkaria.postgresql.org; Fri, 03 Mar 2023 16:28:53 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1pY8H6-0007Hy-El for pgsql-hackers@arkaria.postgresql.org; Fri, 03 Mar 2023 16:28:52 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pY8H6-0007Hp-5R for pgsql-hackers@lists.postgresql.org; Fri, 03 Mar 2023 16:28:52 +0000 Received: from mail-wm1-x331.google.com ([2a00:1450:4864:20::331]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1pY8H3-0005Dv-K8 for pgsql-hackers@lists.postgresql.org; Fri, 03 Mar 2023 16:28:51 +0000 Received: by mail-wm1-x331.google.com with SMTP id t25-20020a1c7719000000b003eb052cc5ccso4309118wmi.4 for ; Fri, 03 Mar 2023 08:28:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1677860928; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=EbI3M9L8ParYU8mpv9jP4fAV8cQ/WfGLzMp3D9XCmWo=; b=ips4qXvzZYlX9kVfBK7EDI8aHCRIJYJKbskLa9fObx62A3BC0ds7Ro5dBPsHRjU2TS 50s7BdFnPyb57kdX1fl5uEd8A6VGZObNyjyfWW+5h5v77+ifWys8oqENJjPPqoH2Mjte se6U+XJgIu56BbxbNosHesfly15R/vnt9S3JORVm/JlbfRnZQRkDTTXjt9klJIQK44dm VQS6FjgC3vrLOzIzBhReymy7HzhhcFRPnG0+9VAUXtV8RxEmDYcjXJ6VmwK2SmntP4oa V4c23JYA9SGPzd1/HGnPvGDDjOsxvjl+zcYr2+h0C2h5lqvYs5+N0Vf14Tmy9ADSTvRo OxpQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1677860928; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=EbI3M9L8ParYU8mpv9jP4fAV8cQ/WfGLzMp3D9XCmWo=; b=1Io3FXX3x6rabXv2easRTf07t9AJ27cA4P8ae8rNE3l0zmBEVxX4voykPakO5BTSAM ZZQKpLNE/T5OcAlLNfFvL01ExOPzijEGodrh1zgt1zUsxgSyQ3mPo/oeKXHyNPr0dDfs yBjYd+JOjyHAiZLcb/8uMAnRq8fj6owwUcQlad4WtpoUSY0Tj5VdBdN/UqvyUOjlUOdQ 0bQvd252Esc3kEonXitBThp4P3IBVWFJ7anUTBtoYMy4KJxpKA0wuwTkye6jNaOh/iKI 1zFdi6t9pCpl27U9l7yG1AxIZChx9tqMpT/GBIV8c4dtzODBTfpaMbDOojUzEBGoKkqI zKUA== X-Gm-Message-State: AO0yUKV+uNm6Pm4rLJouSd4zlJddMH1sXPg2Fc5LuIi2iG4bjeOMGcOR Obg+dxzQkivDepQIBhstO4Q= X-Google-Smtp-Source: AK7set8UNYf4ucXZhzOjfNS2105I8Ifxc1JnhltOe+UgID7wk69dwKfNdqYLXwD8gQqsYccViCIXBA== X-Received: by 2002:a05:600c:4448:b0:3eb:3254:780d with SMTP id v8-20020a05600c444800b003eb3254780dmr2201761wmn.34.1677860928549; Fri, 03 Mar 2023 08:28:48 -0800 (PST) Received: from [192.168.2.139] ([54.239.6.189]) by smtp.gmail.com with ESMTPSA id h22-20020a05600c351600b003daf6e3bc2fsm10788891wmq.1.2023.03.03.08.28.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 03 Mar 2023 08:28:48 -0800 (PST) Message-ID: Date: Fri, 3 Mar 2023 17:28:47 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 Subject: Re: Fix comments in gistxlogDelete, xl_heap_freeze_page and xl_btree_delete Content-Language: en-US To: Amit Kapila Cc: PostgreSQL Hackers , Robert Haas References: <80db7836-4415-d54a-64c3-66b88b1430e7@gmail.com> <86b7297e-13c0-dca0-a723-c687e3878608@gmail.com> From: "Drouvot, Bertrand" In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk 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 separate thread so it does not get lost in the logical decoding >>> on standby thread. >>> >>> [1]: https://www.postgresql.org/message-id/CA%2BTgmoYTTsxP8y6uknZvCBNCRq%2B1FJ4zGbX8Px1TGW459fGsaQ%40mail.gmail.com >>> >>> Looking forward to your feedback, >>> >>> Regards, >>> >> >> It looks like I did not create a CF entry for this one: fixed with [1]. >> >> Also, while at it, adding a commit message in V2 attached. >> > > LGTM. > Thanks for having looked at it! Regards, -- Bertrand Drouvot PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com