Page MenuHomeFreeBSD

Add setproctitle_fast() patches to PostgreSQL 9.5, 9.6, 10 ports.
Needs ReviewPublic

Authored by munro_ip9.org on Jul 12 2018, 5:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 12:26 AM
Unknown Object (File)
Tue, Apr 16, 10:20 PM
Unknown Object (File)
Thu, Mar 28, 8:23 PM
Unknown Object (File)
Feb 5 2024, 8:22 PM
Unknown Object (File)
Feb 5 2024, 7:08 PM
Unknown Object (File)
Jan 12 2024, 11:04 PM
Unknown Object (File)
Jan 9 2024, 5:47 AM
Unknown Object (File)
Dec 22 2023, 1:30 AM

Details

Reviewers
girgen
Summary

In https://reviews.freebsd.org/D16111 we got setproctitle_fast(), a way to speed up PostgreSQL's frequent process title updates on FreeBSD. I can get the PostgreSQL side of that committed to PostgreSQL 12, but it's not PostgreSQL project policy to backpatch new features to older releases (only bug fixes). How about carrying patches for the stable PostgreSQL releases in FreeBSD ports? Then 12-CURRENT users can benefit from the speed-up and provide feedback now. This covers PostgreSQL 9.5, 9.6 and 10. We can take care of PostgreSQL 11 when it's released, quite soon.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Has this been submitted upstream?

Looks good. We should probably also change the default configuration if setproctitle_fast exists. When was it introduced, in which OSVERSION?
files/patch-src_backend_utils_misc_postgresql.conf.sample turns setproctitle off.

Looks good. We should probably also change the default configuration if setproctitle_fast exists. When was it introduced, in which OSVERSION?
files/patch-src_backend_utils_misc_postgresql.conf.sample turns setproctitle off.

I'm not sure... isn't the important question whether the postgres binary was built with this option? If you're using a pkg compiled for 11.0 and you're running on 11.3, then your postgres binary doesn't use setproctitle_fast(), even though the OS has it. Would it make sense to check if "nm /path/to/postgres | grep setproctitle_fast"?

BTW I just pushed this change to PostgreSQL master (PostgreSQL 12-to-be).

Would it make sense to check if "nm /path/to/postgres | grep setproctitle_fast"?

I guess that should be "nm -D ...", since symbols might be stripped.