kafka-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Roesler <j...@confluent.io>
Subject Re: Posted a new article about Kafka Streams
Date Mon, 17 Jun 2019 21:40:00 GMT
Hi all,

Thanks for the heads-up! I've added the article to my reading list.

And thanks for the reminder about your KIP, Paul. I just visited the
discussion and voting threads.

Also, another plug, you might both be interested in my
freshly-proposed KIP-478
(https://cwiki.apache.org/confluence/x/2SkLBw) to add more type safety
to the Processor API.

When I did the POC PR for it, I found it was _super_ useful in
spotting and preventing bugs in the internal code for the Streams DSL.
Maybe you can share some thoughts about whether it would also be
useful as users of the Processor API, what it would need to change to
be useful, or whether I should just propose it as an internal change.

Thanks!
-John

On Mon, Jun 17, 2019 at 1:06 PM Development <dev@yeralin.net> wrote:
>
> Hey Paul,
>
> Thank you so much for your input! :)
> Just updated my article about the iterator closing.
>
> Thank you!
>
> Best,
> Daniyar Yeralin
>
> > On Jun 16, 2019, at 4:11 PM, Paul Whalen <pgwhalen@gmail.com> wrote:
> >
> > I've only skimmed it so far, but great job!  The community is in serious
> > need of more examples of the Processor API, there really isn't that much
> > out there.
> >
> > One thing I did notice: the iterator you get from kvStore.all() ought to be
> > closed to release resources when you're done with it.  This matters when
> > the underlying store is RocksDB, which as I understand it, allocates
> > additional memory off heap to iterate.  I see this bug everywhere, after
> > writing it many times myself over the course of many months :). it's too
> > bad the API can't be more clear, but I guess there's not a ton you can do
> > in Java.  People think about this kind of thing for DB calls, but when
> > you're using something that's basically a HashMap you really don't think of
> > it at all.
> >
> > Side plug for KIP-401 since you're using the Processor API, it would be
> > interesting to hear on that discussion thread if you find it useful (
> > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=97553756).
> > It seems like there's soft interest, but maybe not yet enough to push it
> > over the finish line.
> >
> > Again, great job!
> >
> > Paul
> >
> > On Fri, Jun 14, 2019 at 10:33 AM Development <dev@yeralin.net> wrote:
> >
> >> Bad link:
> >>
> >> https://medium.com/@daniyaryeralin/utilizing-kafka-streams-processor-api-and-implementing-custom-aggregator-6cb23d00eaa7
> >>
> >>> On Jun 14, 2019, at 11:07 AM, Development <dev@yeralin.net> wrote:
> >>>
> >>> Hello Kafka Dev community,
> >>>
> >>> I wrote an article on implementing a custom transformer using Processor
> >> API for Kafka Streams!
> >>>
> >> https://medium.com/@daniyaryeralin/utilizing-kafka-streams-processor-api-and-implementing-custom-aggregation-f6a4a6c376be
> >> <
> >> https://medium.com/@daniyaryeralin/utilizing-kafka-streams-processor-api-and-implementing-custom-aggregation-f6a4a6c376be
> >>>
> >>> Feel free to leave a feedback and/or corrections if I wrote something
> >> silly :)
> >>>
> >>> Thank you!
> >>>
> >>> Best,
> >>> Daniyar Yeralin
> >>
> >>
>

Mime
View raw message