Thanks Steven, I believe I addressed all the comments. If the it looks good to you let's move forward on the vote.
On Sat, Dec 9, 2017 at 12:50 AM Steven Aerts <steven.ae...@gmail.com> wrote: > Hello Xavier, > > for me it is perfect to take it along. > I made a few small remarks in your PR. > > Thanks > > Op za 9 dec. 2017 om 01:29 schreef Xavier Léauté <xav...@confluent.io>: > > > Hi Steve, I just posted in the discussion thread, there's just one tiny > fix > > I think would be useful to add while we're making changes to this API. > > Do you mind having a look? > > > > On Fri, Dec 8, 2017 at 11:37 AM Mickael Maison <mickael.mai...@gmail.com > > > > wrote: > > > > > +1 (non binding) > > > Thanks for the KIP > > > > > > On Fri, Dec 8, 2017 at 6:53 PM, Tom Bentley <t.j.bent...@gmail.com> > > wrote: > > > > +1 > > > > > > > > On 8 December 2017 at 18:34, Ted Yu <yuzhih...@gmail.com> wrote: > > > > > > > >> +1 > > > >> > > > >> On Fri, Dec 8, 2017 at 3:49 AM, Steven Aerts < > steven.ae...@gmail.com> > > > >> wrote: > > > >> > > > >> > Hello everybody, > > > >> > > > > >> > > > > >> > I think KIP-218 is crystallized enough to start voting. > > > >> > > > > >> > KIP documentation: > > > >> > > > > >> > https://cwiki.apache.org/confluence/display/KAFKA/KIP- > > > >> > 218%3A+Make+KafkaFuture.Function+java+8+lambda+compatible > > > >> > > > > >> > > > > >> > Thanks, > > > >> > > > > >> > > > > >> > Steven > > > >> > > > > >> > > > > > >
