On Wed, Mar 26, 2025 at 8:28 PM Sutou Kouhei <kou@clear-code.com> wrote:
The attached v39 patch set uses the followings:
0001: Create copyto_internal.h and change COPY_XXX to COPY_SOURCE_XXX and COPY_DEST_XXX accordingly. (Same as 1. in your suggestion) 0002: Support custom format for both COPY TO and COPY FROM. (Same as 2. in your suggestion) 0003: Expose necessary helper functions such as CopySendEndOfRow() and add CopyFromSkipErrorRow(). (3. + 4. in your suggestion) 0004: Define handler functions for built-in formats. (Not included in your suggestion) 0005: Documentation. (WIP) (Same as 5. in your suggestion)
I don't think this module should be responsible for testing the validity of "qualified names in a string literal" behavior. Having some of the tests use a schema qualification, and I'd suggest explicit double-quoting/case-folding, wouldn't hurt just to demonstrate it's possible, and how extensions should be referenced, but definitely don't need tests to prove the broken cases are indeed broken. This relies on an existing API that has its own tests. It is definitely pointlessly redundant to have 6 tests that only differ from 6 other tests in their use of a schema qualification.
I prefer keeping 0002 and 0004 separate. In particular, keeping the design choice of "unqualified internal format names ignore search_path" should stand out as its own commit.
