On Sat, Jun 13, 2026 at 6:07 PM Sami Imseih <[email protected]> wrote:

> This comment is not needed.
>
> But I think we should add a comment explaining the condition here.
>

Done, replaced with a short comment above the condition and
dropped the inner block comment.

Should we just add tests for all the output types?
>

Added. v2 has column-reorder tests for text and csv (before the
JSON section) plus subset and reorder tests for JSON in the JSON
section.

Thanks for the review, Sami. And thanks Andrew for the +1.

v2 attached.

Attachment: v2-0001-COPY-TO-FORMAT-JSON-respect-column-list-order.patch
Description: Binary data

Reply via email to