Revert "Refactor CopyReadAttributes{CSV,Text}() to use a callback in COPY FROM"
This reverts commit 95fb5b49024, for reasons similar to what led to 1aa8324b81fa. In this case, the callback was called once per row, which is less worse than the previous callback introduced for COPY TO called once per argument for each row, still the patch set discussed to plug in custom routines to the COPY paths would be able to know which subroutine to use depending on its CopyFromState, so this led to a suboptimal approach at the end. For now, this part is reverted to consider better which approach to use. Discussion: https://postgr.es/m/20240206014125.qofww7ew3dx3v...@awork3.anarazel.de Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/06bd311bce24083c76d9741ae89c98750aaf4b41 Modified Files -------------- src/backend/commands/copyfrom.c | 5 ----- src/backend/commands/copyfromparse.c | 24 ++++++++++++++++-------- src/include/commands/copyfrom_internal.h | 17 ----------------- 3 files changed, 16 insertions(+), 30 deletions(-)