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 > > > > > > > > > >
