+1 from me

One more comment on the wiki: while reviewing the PR I realized that in `
MockProcessorContext.java
<https://github.com/apache/kafka/pull/4955/files#diff-d5440e7338f775230019a86e6bcacccb>`
we are also adding one additional API plus modifying the existing
`setRecordMetadata` API. Since this class is part of the public test-utils
package we should claim it in the wiki as well.


Guozhang

On Mon, May 14, 2018 at 8:43 AM, Ted Yu <yuzhih...@gmail.com> wrote:

> +1
>
> On Mon, May 14, 2018 at 8:31 AM, Jorge Esteban Quilcate Otoya <
> quilcate.jo...@gmail.com> wrote:
>
> > Hi everyone,
> >
> > I would like to start a vote on KIP-244: Add Record Header support to
> Kafka
> > Streams
> >
> > KIP wiki page:
> > https://cwiki.apache.org/confluence/display/KAFKA/KIP-
> > 244%3A+Add+Record+Header+support+to+Kafka+Streams+Processor+API
> >
> > The discussion thread is here:
> > http://mail-archives.apache.org/mod_mbox/kafka-dev/201805.
> > mbox/%3CCAC3UcJvrgcBfe6%3DiW6%2BuTWsLB%2B4CsHgRmDx9TvCzJQrWvfg7_w%
> > 40mail.gmail.com%3E
> >
> > Cheers,
> > Jorge.
> >
>



-- 
-- Guozhang

Reply via email to