Lists: | pgsql-hackerspgsql-patches |
---|
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | GUC description cleanup |
Date: | 2006-10-26 17:47:24 |
Message-ID: | 1161884844.27820.17.camel@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
This patch makes the short_desc and extra_desc fields of the GUC table
more consistent: both text in both fields should be complete sentences
that begin with a capital letter and end in a period.
Note that this patch breaks the translations of these strings, so I
haven't applied it yet. Should I apply it now, or wait for 8.3 to
branch?
BTW, should pre_auth_delay be included in SHOW ALL? Its short_desc is
currently "No description available", so I think we should either add a
proper description or omit pre_auth_delay from SHOW ALL.
-Neil
Attachment | Content-Type | Size |
---|---|---|
guc_desc_cleanup-1.patch | text/x-patch | 10.7 KB |
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org, Neil Conway <neilc(at)samurai(dot)com> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: GUC description cleanup |
Date: | 2006-10-26 18:32:07 |
Message-ID: | 23339.1161887527@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
Neil Conway <neilc(at)samurai(dot)com> writes:
> BTW, should pre_auth_delay be included in SHOW ALL?
It's really just a debug aid, so I wouldn't complain if SHOW ALL didn't
show it.
regards, tom lane
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org, Neil Conway <neilc(at)samurai(dot)com> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: GUC description cleanup |
Date: | 2006-10-26 21:37:47 |
Message-ID: | 25871.1161898667@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
Neil Conway <neilc(at)samurai(dot)com> writes:
> Note that this patch breaks the translations of these strings, so I
> haven't applied it yet. Should I apply it now, or wait for 8.3 to
> branch?
BTW, unless Peter says it's OK, my advice is to wait. It's already
likely to be the case that translation updates are the critical path
for releasing 8.2.
regards, tom lane
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org, Neil Conway <neilc(at)samurai(dot)com> |
Subject: | Re: GUC description cleanup |
Date: | 2006-10-27 13:59:45 |
Message-ID: | 200610271559.46668.peter_e@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
Am Donnerstag, 26. Oktober 2006 19:47 schrieb Neil Conway:
> Note that this patch breaks the translations of these strings, so I
> haven't applied it yet. Should I apply it now, or wait for 8.3 to
> branch?
I appreciate this effort, but I think it's better to hold the patch.
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: GUC description cleanup |
Date: | 2006-10-27 14:56:43 |
Message-ID: | 1161961003.27820.50.camel@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
On Fri, 2006-10-27 at 15:59 +0200, Peter Eisentraut wrote:
> I appreciate this effort, but I think it's better to hold the patch.
Sure, I'll wait for 8.3 to branch.
-Neil
From: | "Josh Berkus" <josh(at)agliodbs(dot)com> |
---|---|
To: | Neil Conway <neilc(at)samurai(dot)com>,Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-patches(at)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [HACKERS] GUC description cleanup |
Date: | 2006-10-27 22:16:29 |
Message-ID: | web-10437873@davinci.ethosmedia.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
Neil,
> Sure, I'll wait for 8.3 to branch.
I have some cleanup I want to do for 8.3 too.
Josh Berkus
PostgreSQL @ Sun
San Francisco 415-752-2500
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: GUC description cleanup |
Date: | 2006-12-02 19:38:46 |
Message-ID: | 1165088326.23622.541.camel@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
On Fri, 2006-10-27 at 10:56 -0400, Neil Conway wrote:
> Sure, I'll wait for 8.3 to branch.
Attached is a revised version of this patch. I added a description for
pre_auth_delay (rather than removing it from SHOW ALL). I also removed
explicit mention of units from a bunch of description strings, since
after Peter's work for 8.2, some configuration variables can be
specified or displayed in several possible units. For example, the
current output from SHOW ALL is:
deadlock_timeout | 1s | The time in milliseconds to wait on lock before
checking for deadlock.
... which is plainly not a value "in milliseconds".
Barring any objections, I'll apply this to 8.3 (HEAD) tomorrow.
-Neil
Attachment | Content-Type | Size |
---|---|---|
guc_misc_cleanup-2.patch | text/x-patch | 18.8 KB |
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org, Neil Conway <neilc(at)samurai(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: GUC description cleanup |
Date: | 2006-12-03 03:51:57 |
Message-ID: | 12416.1165117917@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
Neil Conway <neilc(at)samurai(dot)com> writes:
> Attached is a revised version of this patch.
The bgwriter parameter descriptions seem still, well, not good English.
For instance
> ! gettext_noop("Background writer maximum number of LRU pages to flush per round."),
ISTM this would read better as
Maximum number of LRU pages to flush per activity round in the background writer.
or something else that betrays some understanding of English grammar ...
regards, tom lane
From: | Neil Conway <neilc(at)samurai(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: GUC description cleanup |
Date: | 2006-12-06 17:46:38 |
Message-ID: | 1165427198.6453.25.camel@localhost.localdomain |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers pgsql-patches |
On Sat, 2006-12-02 at 22:51 -0500, Tom Lane wrote:
> The bgwriter parameter descriptions seem still, well, not good English.
Indeed, there is still plenty of room for improvement, but exam period
prevents me from attempting anything more extensive at the moment...
Patch applied to HEAD.
-Neil