+1 (binding) Thanks a lot for the KIP!
-Matthias On 5/14/18 10:17 AM, Guozhang Wang wrote: > +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. >>> >> > > >
signature.asc
Description: OpenPGP digital signature