Re: Couple document fixes

From: David Fetter <david(at)fetter(dot)org>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Kevin Grittner <kevin(dot)grittner(at)wicourts(dot)gov>, Thom Brown <thom(at)linux(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Couple document fixes
Date: 2011-09-06 04:22:18
Message-ID: 20110906042218.GM3924@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-docs pgsql-hackers

On Mon, Sep 05, 2011 at 10:07:29PM -0300, Alvaro Herrera wrote:
> Excerpts from David Fetter's message of lun sep 05 21:05:10 -0300 2011:
>
> > On brief inspection, it appears that each of these would be better
> > served, at least functionally, with some kind of enumerated type.
> > Might it be worth trying to micro-optimize this case for a one-byte
> > enum? Or maybe something like the varvarlena pattern?
>
> What would be the point?

Removing the legacy "char" type, per original post. :)

> It works pretty well already. It doesn't need fixing.

We've made changes as big on aesthetic grounds before, and if the
change results in an enum type optimized for space efficiency, that's
all to the good.

Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

In response to

Responses

Browse pgsql-docs by date

  From Date Subject
Next Message Bruce Momjian 2011-09-06 15:08:51 Re: number of semaphores and semaphore sets
Previous Message Bruce Momjian 2011-09-06 02:58:57 Re: Add strftime escapes to Log config

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2011-09-06 04:41:36 Re: [v9.1] sepgsql - userspace access vector cache
Previous Message Robert Haas 2011-09-06 02:58:53 Re: PATCH: regular logging of checkpoint progress