Re: BUG #18060: Left joining rows using random() function in join condition doesn't work as expected.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: james(dot)inform(at)pharmapp(dot)de
Cc: pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #18060: Left joining rows using random() function in join condition doesn't work as expected.
Date: 2023-08-20 16:12:21
Message-ID: 2548808.1692547941@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

PG Bug reporting form <noreply(at)postgresql(dot)org> writes:
> -- 2. Execute this query multiple time and you will see results where:
> -- * no rows from link are joined
> -- * extactly one row is joined
> -- * multiple rows are joined

> select * from source left join link on link_id = 1 + (random()*500)::int8
> order by 1

> /*
> I would expect always exactly one row to be joined.
> Instead I get 1, none or multiple.

I see no bug here: instead, your expectation is faulty. The
given join condition is unstable by definition, but it would
have to be at least stable to produce the results you expect.

Formally, the definition of a SQL join is "evaluate the join's ON
condition for each pair of rows in the cross product of the two
input relations, and return the row pair(s) that satisfy the ON".
So with a random() join condition, anywhere from none to all of
the join pairs involving a given LHS row might get returned,
because the random() function will produce a different value
for each join pair.

You could shove the random() call into a materialized CTE if
the semantics you want are that a single random() result is
used across the entire query. If you want one random() result
to be used for all join pairs involving a given LHS row, but
different ones for different LHS rows, you could probably
make that happen with some hack involving LATERAL. But I'm
too lazy to work it out for you.

regards, tom lane

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message ocean_li_996 2023-08-20 16:51:30 Re:RE: BUG #18055: logical decoding core on AllocateSnapshotBuilder()
Previous Message Michael Paquier 2023-08-20 00:20:48 Re: BUG #18057: unaccent removes intentional spaces