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 1lVbih-0008W3-Vx for pgsql-bugs@arkaria.postgresql.org; Sun, 11 Apr 2021 15:09:51 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1lVbig-0004F2-Ur for pgsql-bugs@arkaria.postgresql.org; Sun, 11 Apr 2021 15:09:50 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lVbig-0004Ev-P8 for pgsql-bugs@lists.postgresql.org; Sun, 11 Apr 2021 15:09:50 +0000 Received: from oss.nttdata.com ([49.212.34.109]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lVbie-0008S6-IF for pgsql-bugs@lists.postgresql.org; Sun, 11 Apr 2021 15:09:49 +0000 Received: from hnk.local (p2016084-ipbf2305funabasi.chiba.ocn.ne.jp [122.25.171.84]) by oss.nttdata.com (Postfix) with ESMTPSA id D52B862756; Mon, 12 Apr 2021 00:09:45 +0900 (JST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.102.3 at oss.nttdata.com Subject: Re: BUG #16931: source code problem about commit_ts To: Alvaro Herrera Cc: zoulx1982@163.com, pgsql-bugs@lists.postgresql.org References: <20210409224241.GA32072@alvherre.pgsql> From: Fujii Masao Message-ID: Date: Mon, 12 Apr 2021 00:09:45 +0900 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:78.0) Gecko/20100101 Thunderbird/78.9.1 MIME-Version: 1.0 In-Reply-To: <20210409224241.GA32072@alvherre.pgsql> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On 2021/04/10 7:42, Alvaro Herrera wrote: > On 2021-Mar-24, Fujii Masao wrote: > >> diff --git a/src/backend/access/rmgrdesc/committsdesc.c b/src/backend/access/rmgrdesc/committsdesc.c >> index 7ebd3d35ef..26bad44b96 100644 >> --- a/src/backend/access/rmgrdesc/committsdesc.c >> +++ b/src/backend/access/rmgrdesc/committsdesc.c >> @@ -38,31 +38,6 @@ commit_ts_desc(StringInfo buf, XLogReaderState *record) >> appendStringInfo(buf, "pageno %d, oldestXid %u", >> trunc->pageno, trunc->oldestXid); >> } >> - else if (info == COMMIT_TS_SETTS) > > You have not pushed this one, right? I think we should do it now. Thanks for the ping! Pushed! Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION