diff options
author | David Rowley <drowley@postgresql.org> | 2023-02-20 16:18:27 +1300 |
---|---|---|
committer | David Rowley <drowley@postgresql.org> | 2023-02-20 16:18:27 +1300 |
commit | 2cb82e2acfba069d00c6bd253d58df03d315672a (patch) | |
tree | 12d9873bb0cca1dd03a4e638f0318eb3c8287b45 /src/backend/executor/nodeFunctionscan.c | |
parent | de2aca288569fd0cabb425c0858e92e2c8c938cb (diff) | |
download | postgresql-2cb82e2acfba069d00c6bd253d58df03d315672a.tar.gz postgresql-2cb82e2acfba069d00c6bd253d58df03d315672a.zip |
Speedup and increase usability of set proc title functions
The setting of the process title could be seen on profiles of very
fast-to-execute queries. In many locations where we call
set_ps_display() we pass along a string constant, the length of which is
known during compilation. Here we effectively rename set_ps_display() to
set_ps_display_with_len() and then add a static inline function named
set_ps_display() which calls strlen() on the given string. This allows
the compiler to optimize away the strlen() call when dealing with
call sites passing a string constant. We can then also use memcpy()
instead of strlcpy() to copy the string into the destination buffer.
That's significantly faster than strlcpy's byte-at-a-time way of
copying.
Here we also take measures to improve some code which was adjusting the
process title to add a " waiting" suffix to it. Call sites which require
this can now just call set_ps_display_suffix() to add or adjust the suffix
and call set_ps_display_remove_suffix() to remove it again.
Reviewed-by: Andres Freund
Discussion: https://postgr.es/m/CAApHDvocBvvk-0gWNA2Gohe+sv9fMcv+fK_G+siBKJrgDG4O7g@mail.gmail.com
Diffstat (limited to 'src/backend/executor/nodeFunctionscan.c')
0 files changed, 0 insertions, 0 deletions