Re: tsearch core path, v0.58

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Gregory Stark <stark(at)enterprisedb(dot)com>
Cc: "Teodor Sigaev" <teodor(at)sigaev(dot)ru>, "Patches" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: tsearch core path, v0.58
Date: 2007-08-16 14:33:45
Message-ID: 3311.1187274825@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Gregory Stark <stark(at)enterprisedb(dot)com> writes:
> "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
>> BTW, I would like to shorten some of the path names in this fileset.
>> Is there a reason not to combine src/backend/utils/adt/tsearch,
>> src/backend/utils/tsearch, and src/backend/utils/tsearch/dict_ispell
>> into one place, perhaps src/backend/tsearch?

> I for one would like to see all the data types in a file in adt. Even if it's
> just a single file containing mostly just glue functions into code in
> sc/backend/tsearch.

The existing file layout doesn't meet that expectation either, since all
the functions were for some reason dumped into a *subdirectory* of adt.
That was where my too-weird, too-much-typing reflex kicked in ...

regards, tom lane

In response to

Browse pgsql-patches by date

  From Date Subject
Next Message Julius Stroffek 2007-08-16 16:10:13 Optimizer hook
Previous Message Gregory Stark 2007-08-16 08:07:49 Re: tsearch core path, v0.58