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 1wTekR-000fYN-2T for pgsql-hackers@arkaria.postgresql.org; Sun, 31 May 2026 11:54:32 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.96) (envelope-from ) id 1wTekO-007ScP-2X for pgsql-hackers@arkaria.postgresql.org; Sun, 31 May 2026 11:54:29 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wTekO-007ScH-0l for pgsql-hackers@lists.postgresql.org; Sun, 31 May 2026 11:54:28 +0000 Received: from mail-lf1-x12e.google.com ([2a00:1450:4864:20::12e]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.98.2) (envelope-from ) id 1wTekL-00000000Pa9-2uD4 for pgsql-hackers@lists.postgresql.org; Sun, 31 May 2026 11:54:27 +0000 Received: by mail-lf1-x12e.google.com with SMTP id 2adb3069b0e04-5aa21fa024cso13621599e87.2 for ; Sun, 31 May 2026 04:54:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1780228463; cv=none; d=google.com; s=arc-20240605; b=WU8M2JJNKNn1DrEAF+dp6sv+oS4ZqQtpO/LCaPl46en8ygjsSuNVPsrDAhuEN3atf6 S0ddWKdnCBtbl2iSz4T0xmVPKT9oz5Sennkq4B5HKqoQjuLyFAWv0Xwuy04P2/KM58zg PyYwWT7g7EpIS5hl6xwlmrXmGQpblEq1kRnayYjIVs0h7PszaqtImBAlcxYanvu4ejuc KVfWtDCvbgxgfbElfbBWa1UFe9JvYmPTa/sDX2s/2/MfUurjrGDDWbPtaiJmqCA4CgJG JOHOIcilpknIkNpD10PKYs6ftoRAbmhHbUSa7lagNhvxz0i86AIzcc0pHbsamIT7v21z 525g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:dkim-signature; bh=iTc+qWKU+OnSLoE6HxXMUrxVCkvgxIXobB86wdtRBOs=; fh=JK5jfoiBNSIwLEx1HCf6Crhao+zG6Xfr6ogX7SB7PPA=; b=XeCuMAwtSfa4VE6+87iAqYOLY0VNQERc6mzdznA+U+Eo/+UkFaRrWzHONuKOSRPscP oWEO9a1Q8KfCsFSluBi8XISJPoxCaKJX7L4lLtsULQaq9jQcoPUBYHe1rLuXCnLYy1F/ tvH3STemCEUqzj0m8yGioNnZhGq9CzhFSA7zLXo+ugdIhv59a+Ekys8VDFD4rmz1lYgZ FvCfVXJdBuf4R0cUzpxQFAa9hYXSN3vl6RD8ZlrBffh8gU+pEtOhUt7B/fvP6DAZIaXF vJiWdfSQ3rH1F+JLpvZ6jkWqwR9NL1ejrq15kusQ332r1/xsyelwilxxE/PwU74QsM/i eUxg==; darn=lists.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=1780228463; x=1780833263; darn=lists.postgresql.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=iTc+qWKU+OnSLoE6HxXMUrxVCkvgxIXobB86wdtRBOs=; b=mXBaKDKiIZtSFPrehL90siGXmNkDowiy2MVN3xJcNaPL24rsOu59Qi9sjgYwR8XPEc Q6V6D58AN2igKiNsoQwN2wHgbbBoXB+zSSxWG/HH+GlcwcIgJy04R10AIgAgQ1kKyT22 yU++HmVYhDQwh7PcVWqtYkBPrLTGD0zWHyUY9QvrJOMFqfMCZCWv8qnJJFJrLUYPFiMG vJN3Rlahhx7a9taEC6iH+Pn39bcyaKifR3sfExKGSFl/fdJ9jGgtGLWEtrPoQvJVwZpQ FQ+a3kHKdAbpfQquGUKoBO+ylzhwqsQxKllsf8OwbV6XRbKryqAu9KB9DGjnEjc3yAuR A7BQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780228463; x=1780833263; h=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=iTc+qWKU+OnSLoE6HxXMUrxVCkvgxIXobB86wdtRBOs=; b=aqpNljFunE6xIQkuC0vCY87SAExDUQ7ptSJ39FUfwr3xLct+pJotEldcnZzHLMW8R+ sg9O5qUVJL9Nv3KKYjLjiW19et13HAG9Ti9HP/4FGv+j1BKfeIXV6+VD9KwUxQ16qTyf qHUtcTSdsr40gEIwWpdSfqY680jc7wVdT3T3FNMWtGIhjVrf8+7SNQCdoOQ9pgtCBKUv GtsGjal5nYf2+HeanEjWFJR04yQt+kswehm+DneJ7WMTtlAae81RzPqCZtDgYMtiRAPQ TPY99JcPNaqHBF32fSDKolgZDS1LlKls/RlQJtdSQgdK3xszfVAjh9wJhSCWQ9CxkQCq MC/A== X-Forwarded-Encrypted: i=1; AFNElJ9DJACmOWxX10K4k/+zR/AoytoOXgk//fEL6rEwm3/8l6XPLPC4+2QGZvmM2acM7mBzEOQ4W1B27EJKos5W@lists.postgresql.org X-Gm-Message-State: AOJu0YyFldzP3opf3k6KmdLgLzUkN6dWOBuMi8t7xbjoML45ZRV8uKLU v55E5k2fULqevcWPL1e+Gjv9bdMIsWQEaNFFB6KDTE0Fcoz6axAVAuWBq1vy8qbHotgmi2Z/ykk hGINjCxl/qyTRerfxCJS3KSUu+Q0uB50= X-Gm-Gg: Acq92OF0NxbMauSwpBhIbYfVkceMYtQ1SnIltAG7FWcZThRHrztsFZZfN7hdl8CcTuY iO/7KrXNLVIJIYM7VR9nHSoFnFouU6TUKXQRLkxvQeH7zGZcwjFMElW3kp33wqFHG0CkAm5dYZK vCJNhwY0M2x6Nqj+BbieR9boYkkvwqJFYzvIw77EZctyy9ctvAFYVRPzcRvEiO8sws9opchejVb Z5klTnXe94onXCxdit9O4vQpCd1MRIFL5BJyvTwQaUcOLQrqbbkeUb34Tu9iR6kGqHmAlCf8Xka Hero83Dc2wC5Mj5DC59685fFH5Df4EtSs1cK4sqt1dqtN1skZ+qlt0hfVh6nPABSRLSa18716av VL23IiPKGSQsyKTWT7ZyK X-Received: by 2002:a05:6512:124a:b0:5aa:4bb4:1e9c with SMTP id 2adb3069b0e04-5aa607add8bmr2284757e87.18.1780228462611; Sun, 31 May 2026 04:54:22 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Amit Kapila Date: Sun, 31 May 2026 04:54:08 -0700 X-Gm-Features: AVHnY4KuXOfCnsp8Gr1CAYkY03XyJA9DWDhzeXSurI68fyXGlQqEFM7rkJaEZSI Message-ID: Subject: Re: Proposal: Conflict log history table for Logical Replication To: Dilip Kumar Cc: Nisha Moond , vignesh C , Peter Smith , shveta malik , Masahiko Sawada , Bharath Rupireddy , PostgreSQL Hackers , shveta malik Content-Type: multipart/mixed; boundary="00000000000022385e06531bbd8c" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --00000000000022385e06531bbd8c Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Sat, May 30, 2026 at 1:12=E2=80=AFAM Dilip Kumar = wrote: > Few comments on 0001 and 0002 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D 1. + Oid subconflictlogrelid; /* Relid of the conflict log table. */ #ifdef CATALOG_VARLEN /* variable-length fields start here */ + /* + * Strategy for logging replication conflicts: + * 'log' - server log only, + * 'table' - conflict log table only, + * 'all' - both log and table. + */ + text subconflictlogdest BKI_FORCE_NOT_NULL; 'log' sounds redundant in the above two field names. I feel naming them as subconflictrelid and subconflictdest should be sufficient. 2. If you agree with the above, then let's make similar changes at other places in the patch. We can change alter_sub_conflictlogdestination to alter_sub_conflict_destination. Also, similar to AlterSubscription_refresh and AlterSubscription_refresh_seq, we can name this new function as AlterSubscription_conflict_dest. 3. Now, let's consider whether we should change the option name to conflict_data_destination instead of conflict_log_destination? The reason I am asking to consider this change is that one of the option values is 'log', so it sounded a bit odd to name the option as conflict_log_destination. If we change this then we can consider changing the name of Enum ConflictLogDest as well. Apart from above, I have made some changes in the attached. Kindly review and see which all can be incorporated in the next version. --=20 With Regards, Amit Kapila. --00000000000022385e06531bbd8c Content-Type: text/plain; charset="US-ASCII"; name="v43-0001_amit.1.patch.txt" Content-Disposition: attachment; filename="v43-0001_amit.1.patch.txt" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_mptq0wt10 ZGlmZiAtLWdpdCBhL3NyYy9iYWNrZW5kL2NhdGFsb2cvY2F0YWxvZy5jIGIvc3JjL2JhY2tlbmQv Y2F0YWxvZy9jYXRhbG9nLmMKaW5kZXggNDZkMjdlZDAyYTkuLmI3ZmNhYWY2Nzk2IDEwMDY0NAot LS0gYS9zcmMvYmFja2VuZC9jYXRhbG9nL2NhdGFsb2cuYworKysgYi9zcmMvYmFja2VuZC9jYXRh bG9nL2NhdGFsb2cuYwpAQCAtMjM0LDggKzIzNCw3IEBAIElzVG9hc3RDbGFzcyhGb3JtX3BnX2Ns YXNzIHJlbHR1cGxlKQogCiAvKgogICogSXNDb25mbGljdExvZ1RhYmxlQ2xhc3MKLSAqCQlUcnVl IGlmZiBGb3JtX3BnX2NsYXNzIHR1cGxlIHJlcHJlc2VudHMgYSBzdWJzY3JpcHRpb24tc3BlY2lm aWMKLSAqICAgICAgQ29uZmxpY3QgTG9nIFRhYmxlLgorICoJCVRydWUgaWZmIHBnX2NsYXNzIHR1 cGxlIHJlcHJlc2VudHMgYSBDb25mbGljdCBMb2cgVGFibGUuCiAgKgogICoJCURvZXMgbm90IHBl cmZvcm0gYW55IGNhdGFsb2cgYWNjZXNzZXMuCiAgKi8KZGlmZiAtLWdpdCBhL3NyYy9iYWNrZW5k L2NhdGFsb2cvaGVhcC5jIGIvc3JjL2JhY2tlbmQvY2F0YWxvZy9oZWFwLmMKaW5kZXggM2I2YTE2 M2MxODQuLjk2ZGVmMzAxOWIzIDEwMDY0NAotLS0gYS9zcmMvYmFja2VuZC9jYXRhbG9nL2hlYXAu YworKysgYi9zcmMvYmFja2VuZC9jYXRhbG9nL2hlYXAuYwpAQCAtMzI3LDcgKzMyNyw3IEBAIGhl YXBfY3JlYXRlKGNvbnN0IGNoYXIgKnJlbG5hbWUsCiAJCQllcmVwb3J0KEVSUk9SLAogCQkJCQko ZXJyY29kZShFUlJDT0RFX0lOU1VGRklDSUVOVF9QUklWSUxFR0UpLAogCQkJCQkgZXJybXNnKCJw ZXJtaXNzaW9uIGRlbmllZCB0byBjcmVhdGUgXCIlcy4lc1wiIiwKLQkJCQkJIGdldF9uYW1lc3Bh Y2VfbmFtZShyZWxuYW1lc3BhY2UpLCByZWxuYW1lKSwKKwkJCQkJCQlnZXRfbmFtZXNwYWNlX25h bWUocmVsbmFtZXNwYWNlKSwgcmVsbmFtZSksCiAJCQkJCSBlcnJkZXRhaWwoIkNvbmZsaWN0IHNj aGVtYSBtb2RpZmljYXRpb25zIGFyZSBjdXJyZW50bHkgZGlzYWxsb3dlZC4iKSkpOwogCX0KIApk aWZmIC0tZ2l0IGEvc3JjL2JhY2tlbmQvY2F0YWxvZy9wZ19wdWJsaWNhdGlvbi5jIGIvc3JjL2Jh Y2tlbmQvY2F0YWxvZy9wZ19wdWJsaWNhdGlvbi5jCmluZGV4IDkzNzkxMjEwZTM1Li4yYTI3ZjZm NDI2NiAxMDA2NDQKLS0tIGEvc3JjL2JhY2tlbmQvY2F0YWxvZy9wZ19wdWJsaWNhdGlvbi5jCisr KyBiL3NyYy9iYWNrZW5kL2NhdGFsb2cvcGdfcHVibGljYXRpb24uYwpAQCAtMTU2LDcgKzE1Niw4 IEBAIGNoZWNrX3B1YmxpY2F0aW9uX2FkZF9zY2hlbWEoT2lkIHNjaGVtYWlkKQogICogaXMgcmVh bGx5IGluYWRlcXVhdGUgZm9yIHRoYXQsIHNpbmNlIHRoZSBpbmZvcm1hdGlvbl9zY2hlbWEgY291 bGQgYmUKICAqIGRyb3BwZWQgYW5kIHJlbG9hZGVkIGFuZCB0aGVuIGl0J2xsIGJlIGNvbnNpZGVy ZWQgcHVibGlzaGFibGUuICBUaGUgYmVzdAogICogbG9uZy10ZXJtIHNvbHV0aW9uIG1heSBiZSB0 byBhZGQgYSAicmVsaXNwdWJsaXNoYWJsZSIgYm9vbCB0byBwZ19jbGFzcywKLSAqIGFuZCBkZXBl bmQgb24gdGhhdCBpbnN0ZWFkIG9mIE9JRCBjaGVja3MuCisgKiBhbmQgZGVwZW5kIG9uIHRoYXQg aW5zdGVhZCBvZiBPSUQgY2hlY2tzLiAgSXNDb25mbGljdExvZ1RhYmxlQ2xhc3MoKQorICogZXhj bHVkZXMgdGFibGVzIGluIGNvbmZsaWN0IHNjaGVtYS4KICAqLwogc3RhdGljIGJvb2wKIGlzX3B1 Ymxpc2hhYmxlX2NsYXNzKE9pZCByZWxpZCwgRm9ybV9wZ19jbGFzcyByZWx0dXBsZSkKZGlmZiAt LWdpdCBhL3NyYy9iYWNrZW5kL2NvbW1hbmRzL3N1YnNjcmlwdGlvbmNtZHMuYyBiL3NyYy9iYWNr ZW5kL2NvbW1hbmRzL3N1YnNjcmlwdGlvbmNtZHMuYwppbmRleCA4OGYyMmJiYjI4Ni4uYWYyNDcw ZjYwYzQgMTAwNjQ0Ci0tLSBhL3NyYy9iYWNrZW5kL2NvbW1hbmRzL3N1YnNjcmlwdGlvbmNtZHMu YworKysgYi9zcmMvYmFja2VuZC9jb21tYW5kcy9zdWJzY3JpcHRpb25jbWRzLmMKQEAgLTg0NSw5 ICs4NDUsOCBAQCBDcmVhdGVTdWJzY3JpcHRpb24oUGFyc2VTdGF0ZSAqcHN0YXRlLCBDcmVhdGVT dWJzY3JpcHRpb25TdG10ICpzdG10LAogCQlDU3RyaW5nR2V0VGV4dERhdHVtKENvbmZsaWN0TG9n RGVzdE5hbWVzW29wdHMuY29uZmxpY3Rsb2dkZXN0XSk7CiAKIAkvKgotCSAqIElmIGxvZ2dpbmcg dG8gYSB0YWJsZSBpcyByZXF1aXJlZCwgcGh5c2ljYWxseSBjcmVhdGUgaXQgbm93LiBXZSBjcmVh dGUKLQkgKiB0aGUgY29uZmxpY3QgbG9nIHRhYmxlIGhlcmUgc28gaXRzIHJlbGF0aW9uIE9JRCBj YW4gYmUgc3RvcmVkIHdoZW4KLQkgKiBpbnNlcnRpbmcgdGhlIHBnX3N1YnNjcmlwdGlvbiB0dXBs ZSBiZWxvdy4KKwkgKiBXZSBjcmVhdGUgdGhlIGNvbmZsaWN0IGxvZyB0YWJsZSBoZXJlLCBpZiBy ZXF1aXJlZCwgc28gdGhhdCBpdHMgcmVsYXRpb24KKwkgKiBPSUQgY2FuIGJlIHN0b3JlZCB3aGVu IGluc2VydGluZyB0aGUgcGdfc3Vic2NyaXB0aW9uIHR1cGxlIGJlbG93LgogCSAqLwogCWlmIChD T05GTElDVFNfTE9HR0VEX1RPX1RBQkxFKG9wdHMuY29uZmxpY3Rsb2dkZXN0KSkKIAkJbG9ncmVs aWQgPSBjcmVhdGVfY29uZmxpY3RfbG9nX3RhYmxlKHN1YmlkLCBzdG10LT5zdWJuYW1lLCBvd25l cik7CkBAIC04NzcsOCArODc2LDggQEAgQ3JlYXRlU3Vic2NyaXB0aW9uKFBhcnNlU3RhdGUgKnBz dGF0ZSwgQ3JlYXRlU3Vic2NyaXB0aW9uU3RtdCAqc3RtdCwKIAl9CiAKIAkvKgotCSAqIElmIGNv bmZsaWN0cyBhcmUgbG9ncyB0byB0YWJsZSBlc3RhYmxpc2ggYW4gaW50ZXJuYWwgZGVwZW5kZW5j eQotCSAqIGJldHdlZW4gdGhlIGNvbmZsaWN0IGxvZyB0YWJsZSBhbmQgdGhlIHN1YnNjcmlwdGlv bi4KKwkgKiBFc3RhYmxpc2ggYW4gaW50ZXJuYWwgZGVwZW5kZW5jeSBiZXR3ZWVuIGNvbmZsaWN0 IGxvZyB0YWJsZSBhbmQKKwkgKiBzdWJzY3JpcHRpb24uCiAJICoKIAkgKiBXZSB1c2UgREVQRU5E RU5DWV9JTlRFUk5BTCB0byBzaWduaWZ5IHRoYXQgdGhlIHRhYmxlJ3MgbGlmZWN5Y2xlIGlzCiAJ ICogc3RyaWN0bHkgdGllZCB0byB0aGUgc3Vic2NyaXB0aW9uLCBzaW1pbGFyIHRvIGhvdyBhIFRP QVNUIHRhYmxlIHJlbGF0ZXMKQEAgLTg4Nyw3ICs4ODYsNyBAQCBDcmVhdGVTdWJzY3JpcHRpb24o UGFyc2VTdGF0ZSAqcHN0YXRlLCBDcmVhdGVTdWJzY3JpcHRpb25TdG10ICpzdG10LAogCSAqIFRo aXMgZW5zdXJlcyB0aGUgY29uZmxpY3QgbG9nIHRhYmxlIGlzIGF1dG9tYXRpY2FsbHkgcmVhcGVk IGR1cmluZyBhCiAJICogRFJPUCBTVUJTQ1JJUFRJT04gdmlhIHBlcmZvcm1EZWxldGlvbigpLgog CSAqLwotCWlmIChDT05GTElDVFNfTE9HR0VEX1RPX1RBQkxFKG9wdHMuY29uZmxpY3Rsb2dkZXN0 KSkKKwlpZiAoT2lkSXNWYWxpZChsb2dyZWxpZCkpCiAJewogCQlPYmplY3RBZGRyZXNzIGNsdDsK IApkaWZmIC0tZ2l0IGEvc3JjL2JhY2tlbmQvY29tbWFuZHMvdGFibGVjbWRzLmMgYi9zcmMvYmFj a2VuZC9jb21tYW5kcy90YWJsZWNtZHMuYwppbmRleCAyZGNmNjRmYTUzYi4uMGEwY2EyYjg1MGUg MTAwNjQ0Ci0tLSBhL3NyYy9iYWNrZW5kL2NvbW1hbmRzL3RhYmxlY21kcy5jCisrKyBiL3NyYy9i YWNrZW5kL2NvbW1hbmRzL3RhYmxlY21kcy5jCkBAIC0yNDU3LDggKzI0NTcsOCBAQCB0cnVuY2F0 ZV9jaGVja19yZWwoT2lkIHJlbGlkLCBGb3JtX3BnX2NsYXNzIHJlbHR1cGxlKQogCSAqIHBnX2xh cmdlb2JqZWN0IGFuZCBwZ19sYXJnZW9iamVjdF9tZXRhZGF0YSB0byBiZSB0cnVuY2F0ZWQgYXMg cGFydCBvZgogCSAqIHBnX3VwZ3JhZGUsIGJlY2F1c2Ugd2UgbmVlZCB0byBjaGFuZ2UgaXRzIHJl bGZpbGVub2RlIHRvIG1hdGNoIHRoZSBvbGQKIAkgKiBjbHVzdGVyLCBhbmQgYWxsb3dpbmcgYSBU UlVOQ0FURSBjb21tYW5kIHRvIGJlIGV4ZWN1dGVkIGlzIHRoZSBlYXNpZXN0Ci0JICogd2F5IG9m IGRvaW5nIHRoYXQuICBXZSBhbHNvIGFsbG93IFRSVU5DQVRFIG9uIHRoZSBjb25mbGljdCBsb2cg dGFibGVzLAotCSAqIHRvIHBlcm1pdCB1c2VycyB0byBtYW51YWxseSBwcnVuZSB0aGVzZSBsb2dz IHRvIG1hbmFnZSBkaXNrIHNwYWNlLgorCSAqIHdheSBvZiBkb2luZyB0aGF0LiBXZSBhbHNvIGFs bG93IFRSVU5DQVRFIG9uIHRoZSBjb25mbGljdCBsb2cgdGFibGVzLAorCSAqIHRvIHBlcm1pdCB1 c2VycyB0byBtYW51YWxseSBwcnVuZSBjb25mbGljdCBkYXRhIHRvIG1hbmFnZSBkaXNrIHNwYWNl LgogCSAqLwogCWlmICghYWxsb3dTeXN0ZW1UYWJsZU1vZHMgJiYgSXNTeXN0ZW1DbGFzcyhyZWxp ZCwgcmVsdHVwbGUpICYmCiAJCSFJc0NvbmZsaWN0TG9nVGFibGVDbGFzcyhyZWx0dXBsZSkK --00000000000022385e06531bbd8c--