Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wAldN-000Iy5-39 for pgsql-hackers@arkaria.postgresql.org; Thu, 09 Apr 2026 09:25:10 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1wAldM-004scw-0T for pgsql-hackers@arkaria.postgresql.org; Thu, 09 Apr 2026 09:25:09 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wAldL-004sco-2d for pgsql-hackers@lists.postgresql.org; Thu, 09 Apr 2026 09:25:08 +0000 Received: from mail-pj1-x1035.google.com ([2607:f8b0:4864:20::1035]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.98.2) (envelope-from ) id 1wAldK-000000009ct-328y for pgsql-hackers@postgresql.org; Thu, 09 Apr 2026 09:25:08 +0000 Received: by mail-pj1-x1035.google.com with SMTP id 98e67ed59e1d1-35d99031e4eso495089a91.1 for ; Thu, 09 Apr 2026 02:25:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1775726704; cv=none; d=google.com; s=arc-20240605; b=VfzYnYRr9HGmRoS1gVopgmUXaLnThkV+6BrCTokx07tAU8btreCgff5erlTPFEQJM+ cKRyfTsqrRPGFcMIDRBZkc78tK42aGdh/PUOeQ707JGwpPe99mp2VM2bR1ogRXUjRFY4 nLkVA8sTY7Zs/ae8rEWrlR+iC+Rbp9Y+LUOD6p9uceuzMRat2fgk3EQuYe7S38GOZBv6 Ygm7MxD0FV+pQm6P7ybjSwXlE0FJ2AD7U8uoKgU2IYx+IKryWff81FuakblxEMyUAv+R ill6dW/rlyFbrH5VbE+cmIDxVAzRXYiJSwItLNTh8wtSe5GRm2IIuPWAL60NkVaxMmVh gSNw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=aZi9vMkYGpCYCzuqHlp9E4UUW9yPsPfwPSHKIXAO9Ig=; fh=o9Vk88D19sr5R+zRjcsqU7T02a8vJFoQ6ABlVNsxLhw=; b=PzeZDABfoi+PGg95RdhWCxHTR2TX1U3h0BPWRLmfqJp/2wpMUGiQOZ24OfWRDeudYu XsFPn0Gue17VodK9WtKq9lHdUaIG+XienIAp6m7bQWhSz4rcCwUkDzHyqB/Qm9WTk84t wRCRSWFNNUBRocYmKyFVpR3UKYP5/ENBuvFxGPdrI6xxa+KvwmuOgUlJwQ1xWpygkooU T4P6LzTP7pOJA0TnJ84GSQXt+oS57NXFouRXZiexvKn/DkhfhvK56rtYrQ4ee0hdbFov 372ldDiMTjHpCiBtzoGl8LU959wPsWESB77AwdOD5zlKCJFScb9RDGOZiW+6hy40xAdA bv3g==; darn=postgresql.org ARC-Authentication-Results: i=1; mx.google.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775726704; x=1776331504; darn=postgresql.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=aZi9vMkYGpCYCzuqHlp9E4UUW9yPsPfwPSHKIXAO9Ig=; b=egwRGmd4AMEzAiCskBpsaRdEQo7pfCgxp9hSe86pW0jNg7WC1QzxkjpYw2wWY531Ku 6wyJiqFP4SSUbiljYTK2H6xtlUESmPLzX9p2JdASNDpl7d+RN6FuOBTVOySaLhgx3bf6 8+OhKQ5/GbX5U/wbFzC4biHCIqf5+RGo8xNe289ZldglGcxQubSOmsBgc6+dt5lxtf7s T8xjPakTUrZXfqY4kyiKtN3dvO8qU+tZj89rpU39YIDy+r+7Asp7iKYcLYYDvnfT8NaP 3Ze8WmgQ71c9I14FAPY9QdjRoBr1qPz+nWrxAz9Ds8K4411XYvWE4T1gQQ0dJ4pUisLS 2llg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775726704; x=1776331504; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=aZi9vMkYGpCYCzuqHlp9E4UUW9yPsPfwPSHKIXAO9Ig=; b=AQuHC9T2q7mFttuLUXLuNZwPW3WdYPdgCkp86+2nbwEMk2isG8OKKBZEIQqi9j/9AU oMYGxnZkpFlr5Ufak7vKL1j73l4UMPCQurl2Fu4elJnbIdQI9l1g/iwZJqHC+B9nWiAb fP6hc0WtxSl0jJ/mGsbWqDUHnc4kqmZYjiQoCrcZFq9VWZXjDCi/hjVgtcvk0O9IADPF VudTCvEqen44USgi9f3r1Xv6JjRAA1MFayc+F2DrA2MzliNbZgIbhTDbvZCsbMMuTD6i 0rCs3zTj/GWIQLBr8dxC02JIpPmPNd22DXZ2TqKILaxkpBfAD1vVQNhXTHnno/3lTO7S 5EuQ== X-Forwarded-Encrypted: i=1; AJvYcCUz5o6uJQB1r1kI4RVpeVf45wTlcw05zlwShmIyKV7LgLdP8GTI761p1TNQ2gtoySH32EeJJFjHtMzPsD4Z@postgresql.org X-Gm-Message-State: AOJu0YwZ5+Ne2iJGp9HRn6gxjoBsTA9zDuzLSdAXdD6iRduFlgSx01cY HPwyf/ZzVTOSzSdws963HKqw/A2o8Qn/crbzH6JYGpsEYrKM7gk33ZHlVj7E/qeOM9OMwgzUjMY fUtHtaYdP6n4K5U/+aTfFRADRM32NLw0= X-Gm-Gg: AeBDieuifS+BhZl2mpLg+ZWxABMthkpcCHsPpIRAIvaxwVwf1rxjsk1W41rDY2iJ6jd Q0TP4sObKM2Ab74/HjV0wPggYzDEL5Z+2cSctMBnygLgWR3sHoNugxB6ydXNxqdYNqodMrICKeo 8u5WAX9Lv9M1MAwKHgHvfGMlbMcm4d2jOgzP7UcneASJGtraclgwl1FBUzJuXXy1qrAhcnscxc+ EHVWHx5nICENwBImQEC9aihHtoJmMI126n7zaUxAdhFSUIKQJUrJPgO8leIVpMbpsfWN4uk3dZ7 PCQH6i3aaMgiDmx3x9xg8OQUccSagnLtTgXyduNSzoCfVPRKGAWLi/TYW1J2ldW9 X-Received: by 2002:a17:90b:2fcc:b0:35b:9d3b:34cb with SMTP id 98e67ed59e1d1-35e35774202mr2754814a91.8.1775726704538; Thu, 09 Apr 2026 02:25:04 -0700 (PDT) MIME-Version: 1.0 References: <2BE661BA-D909-4093-BF78-DB9B0C099337@gmail.com> <77FA04FE-1F84-4DA1-8855-8BBFD8CC889A@gmail.com> <72AA2663-B642-4FB1-BDC2-5FAFF2D2DF15@gmail.com> <8561287B-19F1-421E-959D-99F4593CFF54@gmail.com> In-Reply-To: From: Amit Langote Date: Thu, 9 Apr 2026 18:24:47 +0900 X-Gm-Features: AQROBzDfEiDCffbZP4h2MZtb1YaL479Og51L1B7NpNQXm9XRZ4wf3b0OghOyAdw Message-ID: Subject: Re: Eliminating SPI / SQL from some RI triggers - take 3 To: jie wang Cc: Chao Li , Evan Montgomery-Recht , PostgreSQL-development Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk On Thu, Apr 9, 2026 at 6:22=E2=80=AFPM jie wang wrot= e: > Hi, > > I took a glance at the patch, overall looks good to me. A nitpick on 0001= : > > + bool firing_batch_callbacks; /* true when in > + = * FireAfterTriggersBatchCallbacks() */ > > Looks like a typo in the comment. The function name is FireAfterTriggerBa= tchCallbacks, no =E2=80=9Cs=E2=80=9D after Trigger. Thanks, I've fixed the typo in my local tree. --=20 Thanks, Amit Langote