Use stack-allocated StringInfoDatas, where possible 6d0eba662 already did most of the changes, but some new ones snuck in just prior to that commit, so these got missed.
Having these short-lived StringInfoDatas on the stack rather than having them get palloc'd by makeStringInfo() is simply for performance as it saves doing a 2nd palloc. Since this code is new to v19, it makes sense to improve it now rather than wait until we branch as having v19 and v20 differ here just makes it harder to backpatch fixes in this area. Author: Bertrand Drouvot <[email protected]> Reviewed-by: David Rowley <[email protected]> Discussion: https://postgr.es/m/[email protected] Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/a63bbc811d41b3567eb37fe2636e660a852dbbf2 Modified Files -------------- src/backend/replication/logical/sequencesync.c | 39 ++++++++++++++------------ 1 file changed, 21 insertions(+), 18 deletions(-)
