Hello. Please, review my changes [1] I fixed all conflicts after KAFKA-8885 [2] merge [3].
[1] https://github.com/apache/kafka/pull/7342 [2] https://issues.apache.org/jira/browse/KAFKA-8885 [3] https://github.com/apache/kafka/commit/0de61a4683b92bdee803c51211c3277578ab3edf В Пт, 20/09/2019 в 09:18 -0700, Colin McCabe пишет: > Hi Nikolay, > > Thanks for working on this. I think everyone agrees that we should have byte > buffer support in the generator. We just haven't had a lot of time for > reviewing it lately. I don't really mind which PR we use :) I will take a > look at your PR today and see if we can get it into shape for what we need. > > best, > Colin > > On Fri, Sep 20, 2019, at 09:18, Nikolay Izhikov wrote: > > Hello, all. > > > > Any feedback on this? > > Do we need support of ByteBuffer in RPC generated code? > > > > Which PR should be reviwed and merged? > > > > В Чт, 19/09/2019 в 10:11 +0300, Nikolay Izhikov пишет: > > > Hello, guys. > > > > > > Looks like we have duplicate tickets and PR's here. > > > > > > One from me: > > > > > > KAFKA-8584: Support of ByteBuffer for bytes field implemented. > > > ticket - https://issues.apache.org/jira/browse/KAFKA-8584 > > > pr - https://github.com/apache/kafka/pull/7342 > > > > > > and one from Colin McCabe: > > > > > > KAFKA-8628: Auto-generated Kafka RPC code should be able to use zero-copy > > > ByteBuffers > > > ticket - https://issues.apache.org/jira/browse/KAFKA-8628 > > > pr - https://github.com/apache/kafka/pull/7032 > > > > > > I want to continue work on my PR and got it merged. > > > But, it up to community to decide which changes are best for the product. > > > > > > Please, let me know, what do you think. > > > > > > > > > В Вт, 17/09/2019 в 01:52 +0300, Nikolay Izhikov пишет: > > > > Hello, Kafka team. > > > > > > > > I implemented KAFKA-8584 [1]. > > > > PR - [2] > > > > Please, do the review. > > > > > > > > [1] https://issues.apache.org/jira/browse/KAFKA-8584 > > > > [2] https://github.com/apache/kafka/pull/7342 > > > > Attachments: > > * signature.asc
signature.asc
Description: This is a digitally signed message part
