Re: ecpg command does not warn COPY ... FROM STDIN;
Fujii Masao <masao.fujii@oss.nttdata.com>
From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Ryo Kanbayashi <kanbayashi.dev@gmail.com>,
"pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>
Cc: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>,
Tom Lane <tgl@sss.pgh.pa.us>
Date: 2025-01-12T03:56:48Z
Lists: pgsql-hackers
On 2025/01/12 2:04, Ryo Kanbayashi wrote: > I wrote a patch for release v13 - v17 additionally and tested it for > each release branch :) > As a result, two patch is needed for this fix. Thanks for the patches! Barring any objections, I plan to commit them with the following commit log. ------------------- ecpg: Restore detection of unsupported COPY FROM STDIN. The ecpg command includes code to warn about unsupported COPY FROM STDIN statements in input files. However, since commit 3d009e45bd, this functionality has been broken due to a bug introduced in that commit, causing ecpg to fail to detect the statement. This commit resolves the issue, restoring ecpg's ability to detect COPY FROM STDIN and issue a warning as intended. Back-patch to all supported versions. Author: Ryo Kanbayashi Reviewed-by: Hayato Kuroda, Tom Lane Discussion: https://postgr.es/m/CANOn0Ez_t5uDCUEV8c1YORMisJiU5wu681eEVZzgKwOeiKhkqQ@mail.gmail.com ------------------- > check_patches.sh -> utility script for testing above two patches on > each target branches Should we add a regression test to ensure ecpg correctly reports errors and warnings, including the warning for COPY FROM STDIN? This could help catch similar bugs more effectively. If agreed, we could create this as a separate patch. Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION
Commits
-
ecpg: Restore detection of unsupported COPY FROM STDIN.
- 226c9048d734 13.19 landed
- d06ab3c0c117 14.16 landed
- 71ef47cf0f20 15.11 landed
- 5c7c34db2511 16.7 landed
- ba2dbedd5396 17.3 landed
- 94b914f601bd 18.0 landed
-
plpgsql: pure parser and reentrant scanner
- 7b27f5fd36cb 18.0 cited
-
Add support for piping COPY to/from an external program.
- 3d009e45bde2 9.3.0 cited