Re: A GUC variable to replace PGBE_ACTIVITY_SIZE - Mailing list pgsql-patches

From Heikki Linnakangas
Subject Re: A GUC variable to replace PGBE_ACTIVITY_SIZE
Date
Msg-id 4860F0A8.3060303@enterprisedb.com
Whole thread Raw
In response to A GUC variable to replace PGBE_ACTIVITY_SIZE  (Thomas Lee <tom@vector-seven.com>)
Responses Re: [UPDATED] A GUC variable to replace PGBE_ACTIVITY_SIZE
List pgsql-patches
I'd like to see some quick testing of this thing mentioned in the comments:

>     /*
>      * XXX if pgstat_track_activity_query_size is really large,
>      * it might be best to use strcpy not memcpy for copying the
>      * activity string?
>      */

If we make it a GUC, there will be people running with "really large"
pgstat_track_activity_query_size settings. In fact I wouldn't be
surprised if people routinely cranked it up to the 10-100 KB range, just
in case.

It's going to be platform-dependent, for sure, but some quick
micro-benchmarks of where the break-even point between memcpy and strcpy
lies would be nice.

--
   Heikki Linnakangas
   EnterpriseDB   http://www.enterprisedb.com

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [HACKERS] Patch for Prevent pg_dump/pg_restore from being affected by statement_timeout
Next
From: Thomas Lee
Date:
Subject: Re: [UPDATED] A GUC variable to replace PGBE_ACTIVITY_SIZE