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 1oKHrj-000827-PC for pgsql-hackers@arkaria.postgresql.org; Sat, 06 Aug 2022 11:21:11 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1oKHrh-0002r9-Lc for pgsql-hackers@arkaria.postgresql.org; Sat, 06 Aug 2022 11:21:09 +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 1oKHrh-0002mF-9H for pgsql-hackers@lists.postgresql.org; Sat, 06 Aug 2022 11:21:09 +0000 Received: from mail-oi1-x241.google.com ([2607:f8b0:4864:20::241]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1oKHra-0003jG-MQ for pgsql-hackers@lists.postgresql.org; Sat, 06 Aug 2022 11:21:08 +0000 Received: by mail-oi1-x241.google.com with SMTP id c185so5202366oia.7 for ; Sat, 06 Aug 2022 04:21:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:subject:references:in-reply-to:message-id:to:from:date :from:to:cc; bh=Qwil8abst6xpdWL/kqP5gc3m1qU9LoOA+DrjKqqkqSA=; b=N//7czLpNNo1CcFu/aXmERd3XiR9eL9r3Wi4qBergZUwh6MC703ahJRart6ffRP/Ah fXcesJ3lpDgrAmPMFbuE7ZnMu/g7jZm0cYM6hnbXqke26jalLEwE2s4uF+NU/Xej4NS6 aOpBEyNOI0pPpPCFIld3JUF9dUrVlTMtFqXWHZtpAhn0/o/gdpFitJeMWZYRwbdc1zOd x+tZpSX+Fu8yCrFKGjbLoE/3cNUAGVkDoJFZ5zOcE3gH/Ap6QoeSohamLZjpjL45VmqD QDRpdHZMFsi4aaxPRm5JltYwO2tYXQD61z7e87fbzP2ykzGcA+2Z7Tz5g9BE/I+RKamC 2nJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:subject:references:in-reply-to:message-id:to:from:date :x-gm-message-state:from:to:cc; bh=Qwil8abst6xpdWL/kqP5gc3m1qU9LoOA+DrjKqqkqSA=; b=DT47ngTLEwBhphrXywYXCuaUvHUaXtxElOyL+8hdM86ph7678s0I3+VgesR5aflcgo aaQmmOCj/gZv6sTxR9P4yMcjSzTMP32fgNNguu242P4xC8APmL9hZIykYkLgV+d0IPxW aZEhSmjjMwxQ2VV6QmCi68GiogyhhQS5iH5Z3fmYR26vRly4AnrK6GnifqpFHs8i9sof LhS/jge/aY/PX6s6r+N07XjNQsFbBFJJmw4fdj3+oz3Cphbe//Y0dTgPL60COe4keY1a AQIWMRo8iFXqRwwVuiq4PN6ov0Ox/tSDS+bAeNJqXKObg6aLIjMGz1i2NjPniVERgje0 a2gg== X-Gm-Message-State: ACgBeo1XDbTLCUijgwHedmA3zR3C8Durrmcr/lMNVeqk7b8HSbhICYI1 F5WDFXZRv0O6jszqiYthb6I8Imi8sTweaA== X-Google-Smtp-Source: AA6agR47bbPUrmRARsAo078/5uj9VBZ44t05QoOp6b9DhpnILMZwwmq/ZG5Z/aZX6KrOMies5udKSw== X-Received: by 2002:a05:6808:1412:b0:33b:5cb:5183 with SMTP id w18-20020a056808141200b0033b05cb5183mr7564349oiv.12.1659784860202; Sat, 06 Aug 2022 04:21:00 -0700 (PDT) Received: from [198.18.46.47] ([104.223.97.2]) by smtp.gmail.com with ESMTPSA id 1-20020aca0701000000b00334c2e81dfbsm1067741oih.0.2022.08.06.04.20.58 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 06 Aug 2022 04:20:59 -0700 (PDT) Date: Sat, 6 Aug 2022 19:20:25 +0800 From: Zhang Mingli To: PostgreSQL Hackers Message-ID: In-Reply-To: <920cf618-daed-4a1f-9984-c43eaaa8debe@Spark> References: <920cf618-daed-4a1f-9984-c43eaaa8debe@Spark> Subject: [Code Comments]enum COPY_NEW_FE is removed X-Readdle-Message-ID: aeec12a3-501e-449a-8d25-901c4ff706cf@Spark MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="62ee4e89_515f007c_1b7" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --62ee4e89_515f007c_1b7 Content-Type: multipart/alternative; boundary="62ee4e89_7545e146_1b7" --62ee4e89_7545e146_1b7 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Disposition: inline Hi, Enum COPY_NEW_FE is removed in commit 3174d69fb9. Should use COPY_FRONTEND instead. Issue exists on 15 and master. ``` typedef struct CopyFromStateData - StringInfo fe_msgbuf; /* used if copy_src == COPY_NEW_FE */ + StringInfo fe_msgbuf; /* used if copy_src == COPY_FRONTEND */ ``` Regards, Zhang Mingli --62ee4e89_7545e146_1b7 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline
Hi,

Enum COPY=5FNEW=5F=46E is removed in commit 3174d69fb9.

Should use COPY=5F=46RONTEND instead.

Issue exists on 15 and master.

=60=60=60
typedef struct Copy=46romStateData

- StringInfo fe=5Fmsgbuf; /* used if copy=5Fsrc =3D=3D COPY=5FNEW=5F=46E = */
+ StringInfo fe=5Fmsgbuf; /* used if copy=5Fsrc =3D=3D COPY=5F=46RONTEND = */

=60=60=60

Regards,
Zhang Mingli
--62ee4e89_7545e146_1b7-- --62ee4e89_515f007c_1b7 Content-Type: application/octet-stream Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="=?utf-8?Q?v-0001-Fix-code-comments-COPY=5FFRONTEND.patch?=" RnJvbSA1ZGE1MjgyY2MyOWU0NWJkYTM1YTZlMjlhZDhmODdkNjc0NWY0ODU2IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBNaW5nbGkgWmhhbmcgPGF2YW1pbmdsaUBnbWFpbC5jb20+CkRh dGU6IFNhdCwgNiBBdWcgMjAyMiAxOTowODo0MiArMDgwMApTdWJqZWN0OiBbUEFUQ0ggdm5dIEZp eCBjb2RlIGNvbW1lbnRzIENPUFlfRlJPTlRFTkQKCkNPUFlfTkVXX0ZFIGlzIHJlbW92ZWQgaW4g Y29tbWl0IDMxNzRkNjlmYjkuClVzZSBDT1BZX0ZST05URU5EIGluc3RlYWQuCgpCYWNrcGF0Y2gt dGhyb3VnaDogMTUKRGlzY3Vzc2lvbjogeHh4ClJldmlld2VkLWJ5OiB4eHgKLS0tCiBzcmMvaW5j bHVkZS9jb21tYW5kcy9jb3B5ZnJvbV9pbnRlcm5hbC5oIHwgMiArLQogMSBmaWxlIGNoYW5nZWQs IDEgaW5zZXJ0aW9uKCspLCAxIGRlbGV0aW9uKC0pCgpkaWZmIC0tZ2l0IGEvc3JjL2luY2x1ZGUv Y29tbWFuZHMvY29weWZyb21faW50ZXJuYWwuaCBiL3NyYy9pbmNsdWRlL2NvbW1hbmRzL2NvcHlm cm9tX2ludGVybmFsLmgKaW5kZXggM2RmMWM1YTk3Yy4uZTM3YzYwMzJhZSAxMDA2NDQKLS0tIGEv c3JjL2luY2x1ZGUvY29tbWFuZHMvY29weWZyb21faW50ZXJuYWwuaAorKysgYi9zcmMvaW5jbHVk ZS9jb21tYW5kcy9jb3B5ZnJvbV9pbnRlcm5hbC5oCkBAIC01OCw3ICs1OCw3IEBAIHR5cGVkZWYg c3RydWN0IENvcHlGcm9tU3RhdGVEYXRhCiAJLyogbG93LWxldmVsIHN0YXRlIGRhdGEgKi8KIAlD b3B5U291cmNlCWNvcHlfc3JjOwkJLyogdHlwZSBvZiBjb3B5IHNvdXJjZSAqLwogCUZJTEUJICAg KmNvcHlfZmlsZTsJCS8qIHVzZWQgaWYgY29weV9zcmMgPT0gQ09QWV9GSUxFICovCi0JU3RyaW5n SW5mbwlmZV9tc2didWY7CQkvKiB1c2VkIGlmIGNvcHlfc3JjID09IENPUFlfTkVXX0ZFICovCisJ U3RyaW5nSW5mbwlmZV9tc2didWY7CQkvKiB1c2VkIGlmIGNvcHlfc3JjID09IENPUFlfRlJPTlRF TkQgKi8KIAogCUVvbFR5cGUJCWVvbF90eXBlOwkJLyogRU9MIHR5cGUgb2YgaW5wdXQgKi8KIAlp bnQJCQlmaWxlX2VuY29kaW5nOwkvKiBmaWxlIG9yIHJlbW90ZSBzaWRlJ3MgY2hhcmFjdGVyIGVu Y29kaW5nICovCi0tIAoyLjM0LjEKCg== --62ee4e89_515f007c_1b7--