Hey all,
Whilst working on some UDAFs, I noticed I essentially had to reimplement
GroupByScalar to use scalars as HashMap keys inside accumulator struct state,
as ScalarValue (correctly!) doesn't implement Eq/Hash.
A simple fix to ease this process would be to remove the crate-only access
qual
Hi Ruan,
I am not sure about any stability guidelines, I didn't hear of it.
Aside from this, I think there might be some arguments against making
GroupByScalar public for a number of reasons:
* We have both ScalarValue and GroupByValue, with some duplication/mapping
between them. I think it make
ns on this topic.
Cheers,
Ruan
> -Original Message-
> From: Daniël Heres
> Sent: 23 February 2021 18:19
> To: dev@arrow.apache.org
> Subject: Re: [DataFusion] Promoting GroupByScalar to public API
>
> Hi Ruan,
>
> I am not sure about any stability guidelines,