From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | "David G(dot) Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> |
Cc: | Martin Kalcher <martin(dot)kalcher(at)aboutsource(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: [PATCH] Introduce array_shuffle() and array_sample() |
Date: | 2022-07-18 22:43:32 |
Message-ID: | 847968.1658184212@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
"David G. Johnston" <david(dot)g(dot)johnston(at)gmail(dot)com> writes:
> On Mon, Jul 18, 2022 at 3:18 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Independently of the dimensionality question --- I'd imagined that
>> array_sample would select a random subset of the array elements
>> but keep their order intact. If you want the behavior shown
>> above, you can do array_shuffle(array_sample(...)). But if we
>> randomize it, and that's not what the user wanted, she has no
>> recourse.
> And for those that want to know in what order those elements were chosen
> they have no recourse in the other setup.
Um ... why is "the order in which the elements were chosen" a concept
we want to expose? ISTM sample() is a black box in which notionally
the decisions could all be made at once.
> I really think this function needs to grow an algorithm argument that can
> be used to specify stuff like ordering, replacement/without-replacement,
> etc...just some enums separated by commas that can be added to the call.
I think you might run out of gold paint somewhere around here. I'm
still not totally convinced we should bother with the sample() function
at all, let alone that it needs algorithm variants. At some point we
say to the user "here's a PL, write what you want for yourself".
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Martin Kalcher | 2022-07-18 22:52:47 | Re: [PATCH] Introduce array_shuffle() and array_sample() |
Previous Message | David G. Johnston | 2022-07-18 22:32:00 | Re: [PATCH] Introduce array_shuffle() and array_sample() |
From | Date | Subject | |
---|---|---|---|
Next Message | Martin Kalcher | 2022-07-18 22:52:47 | Re: [PATCH] Introduce array_shuffle() and array_sample() |
Previous Message | Justin Pryzby | 2022-07-18 22:32:40 | Re: Commitfest Update |