Re: citext_1.out, citext.out confusing comment

From: Julien Rouhaud <rjuju123(at)gmail(dot)com>
To: Kirill Reshke <reshkekirill(at)gmail(dot)com>
Cc: jian he <jian(dot)universality(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: citext_1.out, citext.out confusing comment
Date: 2025-12-09 09:55:16
Message-ID: aTfyBLhEOjdNZpuQ@jrouhaud
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Dec 09, 2025 at 12:30:51PM +0500, Kirill Reshke wrote:
> On Tue, 9 Dec 2025, 10:08 jian he, <jian(dot)universality(at)gmail(dot)com> wrote:
>
> > hi.
> >
> > While working on CAST DEFAULT, I found out citext_1.out, citext.out
> > comments are
> > confusing.
> >
> > contrib/citext/expected/citext_1.out:
> > SELECT 'a'::citext >= 'B'::text AS t; -- text wins.
> > t
> > ---
> > f
> > (1 row)
> >
> >
> > contrib/citext/expected/citext.out:
> > SELECT 'a'::citext >= 'B'::text AS t; -- text wins.
> > t
> > ---
> > t
> > (1 row)
> >
> > same input, return different result, The comment "-- text wins."
> > seems confusing.
> > (this happens in other places within citext.out, citext_1.out).
> >
>
> Well, that's because there is single sql test file, so no variation here is
> possible. We can simply delete comment, or we can rephrase it to reflect,
> under which conditions text wins

text always wins, which is why the alternative exists: the result depends on
the default collation sort order under which that test is run.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ashutosh Bapat 2025-12-09 10:21:16 Re: SQL Property Graph Queries (SQL/PGQ)
Previous Message Amit Kapila 2025-12-09 09:36:31 Re: Fix LOCK_TIMEOUT handling in slotsync worker