Re: [DISCUSS] KIP-747 Add support for basic aggregation APIs

2021-08-15 Thread Alexandre Brasil
; fewer parameters is usually better... > > > I am not sure why we would want to pass `Function> > func` into `min()`? > > > > -Matthias > > > > On 6/21/21 11:23 AM, Mohan Parthasarathy wrote: > > Alex, > > > > > > On Wed, Jun 16, 2021 a

Re: [DISCUSS] KIP-747 Add support for basic aggregation APIs

2021-06-16 Thread Alexandre Brasil
to require a ` > Number>`. > > > > I guess these are the two possibilities: > > Long sum(Function func) > Long sum(Function func) > > Both should work. "func" can return any subtypes of Number and I don't see > any advantages with the fi

Re: [DISCUSS] KIP-747 Add support for basic aggregation APIs

2021-06-08 Thread Alexandre Brasil
tion of those use cases. Best regards, Alexandre On Sat, Jun 5, 2021 at 10:17 PM Mohan Parthasarathy wrote: > Hi Alex, > > Responses below. > > On Fri, Jun 4, 2021 at 9:27 AM Alexandre Brasil < > alexandre.bra...@gmail.com> > wrote: > > > Hi Mohan, > > &

Re: [DISCUSS] KIP-747 Add support for basic aggregation APIs

2021-06-04 Thread Alexandre Brasil
Hi Mohan, I like the idea of adding those methods to the API, but I'd like to make a suggestion: Although the most used scenario for min() / max() might possibly be for numeric values, I think they could also be useful on other objects like Dates, LocalDates or Strings. Why limit the API to