Thanks Jason, I have added a public constructor to TransactionDescription class and updated the KIP.
On Thu, Jan 11, 2024 at 9:33 AM Jason Gustafson <ja...@confluent.io.invalid> wrote: > > HI Raman, > > Thanks for the KIP! +1 from me. > > One small thing: we will probably have to overload the constructor for > TransactionDescription in order to add the new update time field to avoid > breaking the API. We might consider whether we need the overload to be > public or not. > > Best, > Jason > > On Tue, Jan 9, 2024 at 10:41 AM Justine Olshan <jols...@confluent.io.invalid> > wrote: > > > Thanks Raman. > > > > +1 (binding) from me as well. > > > > Justine > > > > On Tue, Jan 9, 2024 at 10:12 AM Jun Rao <j...@confluent.io.invalid> wrote: > > > > > Hi, Raman, > > > > > > Thanks for the KIP. +1 from me. > > > > > > Jun > > > > > > On Tue, Dec 26, 2023 at 11:32 AM Raman Verma <rve...@confluent.io.invalid > > > > > > wrote: > > > > > > > I would like to start a Vote on KIP-994 > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/KAFKA/KIP-994%3A+Minor+Enhancements+to+ListTransactions+and+DescribeTransactions+APIs > > > > > > > > >
