Amit Kapila <amit.kapil...@gmail.com> writes: > The main point for this improvement is that the handling for guc s_s_names > is not similar to what we do for other somewhat similar guc's and which > causes in-efficiency in non-hot code path (less used code).
This is not about efficiency, this is about correctness. The proposed v7 patch is flat out not acceptable, not now and not for 9.7 either, because it introduces a GUC assign hook that can easily fail (eg, through out-of-memory for the copy step). Assign hook functions need to be incapable of failure. I do not see any good reason why this one cannot satisfy that requirement, either. It just needs to make use of the "extra" mechanism to pass back an already-suitably-long-lived result from check_synchronous_standby_names. See check_timezone_abbreviations/ assign_timezone_abbreviations for a model to follow. You are going to need to find a way to package the parse result into a single malloc'd blob, though, because that's as much as guc.c can keep track of for an "extra" value. regards, tom lane -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers