+1 (binding)
On 10/19/21 10:37 AM, Israel Ekpo wrote:
Thanks for the KIP
+1 from me (non binding)
On Tue, Oct 19, 2021 at 12:39 PM Guozhang Wang <wangg...@gmail.com> wrote:
Thanks for the KIP Sophie, I'm big +1 on this idea :)
On Tue, Oct 19, 2021 at 6:48 AM Bill Bejeck <bbej...@gmail.com> wrote:
Thanks for the KIP Sophie, I think this will be helpful.
+1(binding)
-Bill
On Mon, Oct 18, 2021 at 5:04 PM John Roesler <vvcep...@apache.org>
wrote:
Thanks, Sophie,
+1 from me.
I was a little apprehensive about the potential to break
exception handler logic, but I agree with your comment in
the "compatibility" section: Identifying the causes of
exceptions is always kind of chaotic and this seems like
more of an improvement than anything.
Regardless, it'll be really nice to get some information
about the context of the exception.
Thanks!
-John
On Mon, 2021-10-18 at 13:56 -0500, Walker Carlson wrote:
Hey Sophie,
+1 for me, I think that is would only help.
Walker
On Mon, Oct 18, 2021 at 1:45 AM Luke Chen <show...@gmail.com> wrote:
Hi Sophie,
Add taskId to make the exception much clear is a good improvement.
+ 1 (non-binding)
Thank you.
Luke
On Mon, Oct 18, 2021 at 12:10 PM Sophie Blee-Goldman
<sop...@confluent.io.invalid> wrote:
Hey all,
I'd like to kick off the vote on this small KIP which adds a
TaskId
field
to the StreamsException class. Please take a look and cast your
vote
when
you have a chance.
Links:
- KIP-783: Add TaskId field to StreamsException
<https://cwiki.apache.org/confluence/x/VIlnCw>
- PR #11405 <https://cwiki.apache.org/confluence/x/VIlnCw>
Thanks!
Sophie
--
-- Guozhang
