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 1nFxaI-00088I-N9 for pgsql-hackers@arkaria.postgresql.org; Fri, 04 Feb 2022 12:21:02 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1nFxaH-0001h5-JR for pgsql-hackers@arkaria.postgresql.org; Fri, 04 Feb 2022 12:21:01 +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 1nFxaH-0001gw-7m for pgsql-hackers@lists.postgresql.org; Fri, 04 Feb 2022 12:21:01 +0000 Received: from mail-yb1-xb2f.google.com ([2607:f8b0:4864:20::b2f]) by makus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1nFxaE-0008Ot-SB for pgsql-hackers@lists.postgresql.org; Fri, 04 Feb 2022 12:21:00 +0000 Received: by mail-yb1-xb2f.google.com with SMTP id i10so18029957ybt.10 for ; Fri, 04 Feb 2022 04:20:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=uBSHdkItZzfiggAz0r2ujZTZx7vNSTCCCOsXJQWNt3E=; b=eyprhNdav1sdEC5uPqB/9XCx5HyViFAVyySxNWwKjBEZEMo6um4POxowiXNGzohK7F aC7Rhip0IAQuZvQjm7iemWXPx/NutAvJYJ7sBgc3n0qaFmhMcEc/uXfX7OUglUsHemZP aUqM//rEssI1wbU88LkjPXayZQMfPHb1PA5uqFzDBfwJNG73DrAjArFNPiNtXUP7kL7S DVvTQt2ADguCRjjnJMFsz1ZF4o0ZeHcm39Je1RiVD4OinIqlos6A7YHBqo79vBWgW3rs ChGdKHuEVCebxcNU5FhVKECuH5/KKAYQVhnEotv4uWodkMCY141idWemUWHjrdM8HzRb rToQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=uBSHdkItZzfiggAz0r2ujZTZx7vNSTCCCOsXJQWNt3E=; b=rbbtLrNQpwmavKeMk2owMAzpmitOkckKCSc6UMiTxNSucjevcYL27CNutoWQErw9jH GvInnVR4kmAYy9NpFhl6xzCLLtV7vL6GbG5uSgWjbIvsNgpqe3w9zTK3L0yBHQhryWOD UKgZ0NtQSSc3uU9q8kfOeA/o5iufKpNch2Jw63zk7r0bAsH7kASXkS0XgAcbb1uxoCkQ R8t4VwuTpD7F77vF6tOASE/MW/SK6LTvaJ434upa3BL5RYmAWRZS0OlIyU99bMOLinCQ YQDu3CHbKU7JjidDHiLhy+aaZTeZtJhhbM7N+ihqV4vlWlECd/my3w9mGwLvzYsubaws 6Jug== X-Gm-Message-State: AOAM532xGCZelQuIBvkkvrPf9nvrxuVQOZDipeXEgUDAS+zX41uroAg0 SHSDOJoexgLb5BdDpVJzYSO3j9iZOncCGMm4RwQ= X-Google-Smtp-Source: ABdhPJyrdSNh+l0F3rDD4+EFWWPMvkhSbYWPUyZrXFQTsZyo1Cr1u1wn58/fF/scu0ebZwTNbvUtLsKOKP/Kx/XwD7c= X-Received: by 2002:a25:df89:: with SMTP id w131mr2359279ybg.692.1643977257885; Fri, 04 Feb 2022 04:20:57 -0800 (PST) MIME-Version: 1.0 References: <885288.1641420714@sss.pgh.pa.us> <137797.1643856498@sss.pgh.pa.us> In-Reply-To: From: Amit Kapila Date: Fri, 4 Feb 2022 17:50:46 +0530 Message-ID: Subject: Re: Bugs in pgoutput.c To: Tom Lane Cc: PostgreSQL Hackers , =?UTF-8?B?SG91LCBaaGlqaWUv5L6vIOW/l+adsA==?= Content-Type: text/plain; charset="UTF-8" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Thu, Feb 3, 2022 at 5:24 PM Amit Kapila wrote: > > On Thu, Feb 3, 2022 at 8:18 AM Tom Lane wrote: > > > > Amit Kapila writes: > > > Tom, is it okay for you if I go ahead with this patch after some testing? > > > > I've been too busy to get back to it, so sure. > > > > Thanks. I have tested the patch by generating an invalidation message > for table DDL before accessing the syscache in > logicalrep_write_tuple(). I see that it correctly invalidates the > entry and rebuilds it for the next operation. I couldn't come up with > some automatic test for it so used the debugger to test it. I have > made a minor change in one of the comments. I am planning to push this > tomorrow unless there are comments or suggestions. > Pushed! -- With Regards, Amit Kapila.