Re: guc

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Liam Stewart <liams(at)redhat(dot)com>
Cc: PostgreSQL Patches List <pgsql-patches(at)postgresql(dot)org>
Subject: Re: guc
Date: 2002-02-25 03:55:54
Message-ID: 200202250355.g1P3ts101889@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


This patch was rejected. Please continue discussion on the hackers
list.

---------------------------------------------------------------------------

Liam Stewart wrote:
>
> I was playing around with adding a couple new SET commands (REAL_FORMAT
> and DOUBLE_PRECISION_FORMAT as per todo list item) and I ended up
> playing with GUC a bit.
>
> I've cleaned it up a bit - most of the changes should be fairly
> self-explanitory from the patch. Some of the configuration options
> handled in variable.c (so far, server_encoding and seed) have been moved
> into guc.c and are handled by GUC. Moving them in required adding a
> couple of fields to config_generic and friends - a read_only field and a
> display_proc field (hook for custom SHOW messages). I've removed
> GetConfigOption() and replaced it with ShowConfigOption() which does an
> elog(NOTICE) instead of returning a string.
>
> When setting REAL_FORMAT and DOUBLE_PRECISION_FORMAT, the printf-style
> format string should be checked for certain things:
>
> - field width of "*" is not allowed
> - precision of ".*" is not allowed
> - all conversion specifiers except "s" and "n" are allowed
> - only one conversion specification other than %% and variations is
> allowed
>
> Does this seem reasonable to people? I haven't written a function to
> check those yet.. will do soon.
>
> This patch isn't complete yet so don't append it to the pending patches
> list, Bruce.
>
> Liam
>
> --
> Liam Stewart :: Red Hat Canada, Ltd. :: liams(at)redhat(dot)com

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

  • guc at 2002-01-17 21:54:34 from Liam Stewart

Browse pgsql-hackers by date

  From Date Subject
Next Message Gavin Sherry 2002-02-25 04:14:53 Re: CLUSTER TODO item
Previous Message Bruce Momjian 2002-02-25 03:54:47 Re: CLUSTER TODO item

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2002-02-25 04:08:27 Re: oid2name cleanup
Previous Message Christopher Kings-Lynne 2002-02-25 03:54:02 Re: Basic DOMAIN Support