Re: citext_1.out, citext.out confusing comment - Mailing list pgsql-hackers

From Julien Rouhaud
Subject Re: citext_1.out, citext.out confusing comment
Date
Msg-id aTfyBLhEOjdNZpuQ@jrouhaud
Whole thread Raw
In response to Re: citext_1.out, citext.out confusing comment  (Kirill Reshke <reshkekirill@gmail.com>)
List 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.universality@gmail.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.



pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Fix LOCK_TIMEOUT handling in slotsync worker
Next
From: Ashutosh Bapat
Date:
Subject: Re: SQL Property Graph Queries (SQL/PGQ)