Re: [RRR] Tests citext casts

From: Kenneth Marshall <ktm(at)rice(dot)edu>
To: "David E(dot) Wheeler" <david(at)kineticode(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [RRR] Tests citext casts
Date: 2008-11-05 20:34:03
Message-ID: 20081105203403.GE20501@it.is.rice.edu
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-rrreviewers

On Wed, Nov 05, 2008 at 09:04:04AM -0800, David E. Wheeler wrote:
> On Nov 5, 2008, at 6:40 AM, Kenneth Marshall wrote:
>
>> I installed and ran the citext tests both with and without
>> the patch and had failures both times. The patch applied
>> cleanly and the "make;make install" completed without errors.
>> I have attached the two regression.diffs files, one without
>> the patch applied and the other with the patch.
>
> What patch was it you applied? And is this CVS HEAD that you're testing?
> What locale/collation is your database configured with?
>
> Thanks,
>
> David
>
David,

I am using the anonymous CVS repository, it returns the following
information in pg_catalog.pg_settings:

postgres=# select * from pg_catalog.pg_settings where name like 'server_%';
name | setting | unit | category
| short_desc | extra_desc | c
ontext | vartype | source | min_val | max_val | enumvals | boot_val | reset_
val | sourcefile | sourceline
--------------------+----------+------+-----------------------------------------
-----------+----------------------------------------------------+------------+--
--------+---------+----------+---------+---------+----------+-----------+-------
----+------------+------------
server_encoding | UTF8 | | Client Connection Defaults / Locale and
Formatting | Sets the server (database) character set encoding. | | i
nternal | string | override | | | | SQL_ASCII | UTF8
| |
server_version | 8.4devel | | Preset Options
| Shows the server version. | | i
nternal | string | default | | | | 8.4devel | 8.4dev
el | |
server_version_num | 80400 | | Preset Options
| Shows the server version as an integer. | | i
nternal | integer | default | 80400 | 80400 | | 80400 | 80400
| |
(3 rows)

The patch that I used is from the link in the commitfest 2008-11 wiki
which points to:

http://archives.postgresql.org/message-id/0BD2E4E9-AF5A-4B4F-B546-027424EE8FAD@kineticode.com

Cheers,
Ken

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2008-11-05 20:41:50 Re: Toast bug in CVS HEAD
Previous Message Alvaro Herrera 2008-11-05 20:31:29 Re: A bug with ALTER TABLE SET WITHOUT OIDS in CVS HEAD

Browse pgsql-rrreviewers by date

  From Date Subject
Next Message Unicron 2008-11-06 04:53:45 "I am done"
Previous Message Josh Berkus 2008-11-05 19:34:55 Reminder: where to send reviews