Re: [PATCH] Use $ parameters as replacement characters for pg_stat_statements

Поиск
Список
Период
Сортировка
От Lukas Fittl
Тема Re: [PATCH] Use $ parameters as replacement characters for pg_stat_statements
Дата
Msg-id CAP53PkwDGqaafZ_qtqu99yqOiXYT05fXE4Gzcm+=0GX-yBu7MA@mail.gmail.com
обсуждение исходный текст
Ответ на [HACKERS] [PATCH] Use $ parameters as replacement characters for pg_stat_statements  (Lukas Fittl <lukas@fittl.com>)
Список pgsql-hackers
On Mon, Mar 27, 2017 at 8:24 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
Pushed with minor adjustments.

Excellent - thanks for your review and all the discussion here!
 
The main non-cosmetic thing I did was to replace the floor(log10())
business with plain constant "10" as I suggested before.  That's
what we do in other places --- see int4out for an example --- and
frankly I did not feel that a small space savings in a transient
string buffer was worth the intellectual effort to verify whether
that calculation was correct or not, never mind whatever runtime
cycles it would take.  I don't believe the argument that it's safer
your way: if you had an off-by-one thinko in the calculation, or even
just roundoff error in the log10() call, it could result in an actual
reachable buffer overrun, because there's no safety margin.

Makes sense, guess I was overthinking this one.

Best,
Lukas 

--
Lukas Fittl

Skype: lfittl
Phone: +1 415 321 0630

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: Potential data loss of 2PC files
Следующее
От: Stas Kelvich
Дата:
Сообщение: Re: logical decoding of two-phase transactions