2016-01-19 13:42 GMT+01:00 Robert Haas <robertmh...@gmail.com>:
> On Mon, Jan 18, 2016 at 6:56 PM, Vitaly Burovoy
> <vitaly.buro...@gmail.com> wrote:
> > On 1/4/16, Robert Haas <robertmh...@gmail.com> wrote:
> >> On Mon, Jan 4, 2016 at 10:17 AM, Pavel Stehule <pavel.steh...@gmail.com
> >
> >> wrote:
> >>> [ new patch ]
> >>
> >> + case '-':
> >> + ereport(ERROR,
> >> + (errcode(ERRCODE_INVALID_PARAMETER_VALUE),
> >> + errmsg("size cannot be negative")));
> >>
> >> Why not? I bet if you copy any - sign to the buffer, this will Just
> Work.
> >
> > Hmm. The function's name is pg_size_bytes. How number of bytes can be
> > negative? How any length can be negative? If anyone insert '-' sign to
> > an argument, it is copy-paste error. I don't see any case where there
> > is possible negatives as input value.
> >
> > I prefer error message instead of getting all relations (by using
> > comparison from the initial letter) just because of copy-paste mistake
> > or incomplete checking of input values at app-level.
>
> Well, we just recently did this:
>
> commit 8a1fab36aba7506fcf4559c4ef95fcacdd0b439a
> Author: Robert Haas <rh...@postgresql.org>
> Date: Fri Nov 6 11:03:02 2015 -0500
>
> pg_size_pretty: Format negative values similar to positive ones.
>
> Previously, negative values were always displayed in bytes, regardless
> of how large they were.
>
> Adrian Vondendriesch, reviewed by Julien Rouhaud and myself
>
> Since we went to the trouble of making the handling of positive and
> negative values symmetric for that function, it seems it should be
> done here also. Otherwise, it is asymmetric.
>
the last patch (pg-size-bytes-08.patch
<http://www.postgresql.org/message-id/attachment/41204/pg-size-bytes-08.patch>)
at 2016-01-04 17:03:02
<http://www.postgresql.org/message-id/cafj8pra__oz8qwokqseqtcqea4dwe9_hw_vunzo9j_c7sl3...@mail.gmail.com/>
allows negative size
Regards
Pavel
> --
> Robert Haas
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
