Re: appendBinaryStringInfo stuff - Mailing list pgsql-hackers

From Tom Lane
Subject Re: appendBinaryStringInfo stuff
Date
Msg-id 1826407.1671695771@sss.pgh.pa.us
Whole thread Raw
In response to Re: appendBinaryStringInfo stuff  (David Rowley <dgrowleyml@gmail.com>)
Responses Re: appendBinaryStringInfo stuff
List pgsql-hackers
David Rowley <dgrowleyml@gmail.com> writes:
>   22.57%  postgres          [.] escape_json

Hmm ... shouldn't we do something like

-                appendStringInfoString(buf, "\\b");
+                appendStringInfoCharMacro(buf, '\\');
+                appendStringInfoCharMacro(buf, 'b');

and so on in that function?  I'm not convinced that this one
hotspot justifies inlining appendStringInfoString everywhere.

            regards, tom lane



pgsql-hackers by date:

Previous
From: "Drouvot, Bertrand"
Date:
Subject: Re: Minimal logical decoding on standbys
Next
From: Masahiko Sawada
Date:
Subject: Re: Force streaming every change in logical decoding