Re: ecpg command does not warn COPY ... FROM STDIN; - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: ecpg command does not warn COPY ... FROM STDIN;
Date
Msg-id 7db9ef7d-1d55-4ddc-8837-d383ae723338@oss.nttdata.com
Whole thread Raw
In response to RE: ecpg command does not warn COPY ... FROM STDIN;  ("Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>)
Responses Re: ecpg command does not warn COPY ... FROM STDIN;
List 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




pgsql-hackers by date:

Previous
From: Jeremy Schneider
Date:
Subject: Re: llvm dependency and space concerns
Next
From: Ryo Kanbayashi
Date:
Subject: Re: ecpg command does not warn COPY ... FROM STDIN;