Re: proposal: psql command \graw

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: proposal: psql command \graw
Date: 2017-08-24 07:20:10
Message-ID: CAFj8pRBmB-++d8Xi=i=moLhj5q=6Tg0N62ogPhE0OeUaK72GBw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2017-08-24 8:53 GMT+02:00 Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>:

>
> "column_header" is somehow redundant with "tuples_only". Use the
>>> existing one instead of adding a new one?
>>>
>>
>> It is different - a result of tuples_only is just tuples - not column
>> names, not title, footer. I needed new special flag for enhancing
>> tuples_only to print column names
>>
>
> I do not understand. If you keep the special print_raw function, it can
> use tuples_only as true for without column names, and false for with column
> names?

yes - in this case you have true.

>
>
> More generally, ISTM that the same effect could be achieved without
>>> adding a new print function, but by setting more options (separator,
>>> ...) and calling an existing print function. If so, I think it would
>>> reduce the code size.
>>>
>>
>> Maybe, maybe not. removing PRINT_RAW you need to enhance PRINT_UNALIGNED
>> to
>> use one shot parameters and you have to teach it to print column names in
>> tuples_only mode. The code's length will be same. The form of this patch
>> is
>> not final.
>>
>
> Hmmm. Ok. It depends on the change implication on the print unaligned
> function.

It is open - I'll wait to winter commitfest for some other ideas, tests,
comments - it is topic for PostgreSQL 11, and then there are a time for
discussion

Now, I'll be happy if some other people will test it with larger set of
applications and send me a feedback.

Regards

Pavel

>
>
> --
> Fabien.
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Masahiko Sawada 2017-08-24 07:27:05 Re: Quorum commit for multiple synchronous replication.
Previous Message Masahiko Sawada 2017-08-24 07:12:29 Re: pgbench: Skipping the creating primary keys after initialization