Hi Amit, > -----Original Message----- > From: Amit Langote [mailto:langote_amit...@lab.ntt.co.jp] > Sent: Wednesday, March 09, 2016 4:29 PM > To: Kyotaro HORIGUCHI <horiguchi.kyot...@lab.ntt.co.jp> > Cc: robertmh...@gmail.com; amitlangot...@gmail.com; SPS ポクレ ヴィ > ナヤック(三技術) <poku...@pm.nttdata.co.jp>; pgsql- > hack...@postgresql.org; SPS 坂野 昌平(三技術) <ban...@nttdata.co.jp> > Subject: Re: [HACKERS] [PROPOSAL] VACUUM Progress Checker. > > > On 2016/03/08 18:19, Kyotaro HORIGUCHI wrote: > >> + WHEN 0 THEN 100::numeric(5, 2) > >> + ELSE ((S.param3 + 1)::numeric / S.param2 * > 100)::numeric(5, 2) > >> > >> This usage of numeric seems overkill to me. > > > > Hmm, how could this rather be written? > > OK, agreed about the overkill. Following might be better: > > + WHEN 0 THEN round(100.0, 2) > + ELSE round((S.param3 + 1) * 100.0 / S.param2, 2) +1
> Will update that patch. > > Thanks, > Amit > -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers