[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Column labels with --full
From: |
Shawn Wagner |
Subject: |
Re: Column labels with --full |
Date: |
Tue, 17 May 2022 23:28:25 -0700 |
I think it doesn't make much sense to combine --full and an aggregate
operation. SQL databases (Except for sqlite) give an error when you
try to use a non grouped column outside of an aggregate function; this
is the same situation.
Maybe document it as deprecated, print a warning about the
combination, and turn it into a full blown error in a few releases?
On Sun, May 15, 2022 at 2:17 PM Tim Rice <trice@posteo.net> wrote:
>
> Hi Dima,
>
> Hmm. I haven't really used the --full flag before.
>
> The example in the manual only shows it being used when the operation can be
> applied to one record at a time. Furthermore, the wording of the manual entry
> makes it unclear what the expected behavior should be when the operation does
> some form of aggregation (eg groubpy, sum, mean).
>
> It might be reasonable to add a caveat "Only use this for record-wise
> operations" to the manual? And for other operations, perhaps we could add a
> check which warns on stderr when --full is not being used as expected.
>
> That's my initial thoughts. Any other ideas about what should be changed here?
>
> ~ Tim
>