Hi Matthias

Thank you again for previous suggestion.
I will start a VOTE soon.

Best regards,
TengYao

Matthias J. Sax <mj...@apache.org> 於 2024年8月22日 週四 下午12:02寫道:

> I think the KIP is fine as-is.
>
> Seems it's also not very controversial. Feel free to start a VOTE.
>
>
> -Matthias
>
> On 8/6/24 7:54 PM, TengYao Chi wrote:
> > Hi Matthias,
> > Thank you for pointing that out. I will keep it in mind.
> > Do I need to update the content of the KIP, or can we leave it as is?
> >
> > Best regards,
> > TengYao
> >
> > Matthias J. Sax <mj...@apache.org> 於 2024年8月7日 週三 上午10:45寫道:
> >
> >> I have already an open PR for the internal cleanup:
> >> https://github.com/apache/kafka/pull/14551
> >>
> >> While I did this cleanup, I filed the Jira for the necessary public
> >> changes. Did not expect that the KIP would get picked up so quickly :)
> >>
> >> In general, a KIP should only concern itself with public API changes,
> >> and thus, does not need to mention internal changes.
> >>
> >>
> >> -Matthias
> >>
> >> On 8/4/24 5:50 PM, 黃竣陽 wrote:
> >>> I just trace these code, I think this proposal is clear enough.
> >>>
> >>>> Kuan Po Tseng <brandb...@apache.org> 於 2024年8月5日 凌晨12:45 寫道:
> >>>>
> >>>> Thank you ! LGTM !
> >>>>
> >>>> Best regards,
> >>>> Kuan-Po Tseng
> >>>>
> >>>> On 2024/08/04 16:40:12 TengYao Chi wrote:
> >>>>> Hello Kuan Po
> >>>>>
> >>>>> I just reviewed the code, and you are right.
> >>>>>
> >>>>> Currently, Joined#gracePeriod is not in the JoinedInternal. I will
> >> slightly
> >>>>> modify the content of the KIP to avoid any misunderstandings.
> >>>>>
> >>>>>
> >>>>> Thanks for pointing it out.
> >>>>>
> >>>>> Best Regards,
> >>>>> TengYao
> >>>>>
> >>>>> Kuan Po Tseng <brandb...@apache.org> 於 2024年8月5日 週一 上午12:30寫道:
> >>>>>
> >>>>>> Thank you for the nice KIP !
> >>>>>>
> >>>>>> About deprecating Joined#gracePeriod, did you mean we should move
> the
> >>>>>> Joined#gracePeriod to JoinedInternal class as I didn't see the
> >>>>>> corresponding method in JoinedInternal you mentioned.
> >>>>>>
> >>>>>> Best regards,
> >>>>>>
> >>>>>> Kuan-Po Tseng
> >>>>>>
> >>>>>> On 2024/08/04 16:13:31 TengYao Chi wrote:
> >>>>>>> Hello everyone,
> >>>>>>>
> >>>>>>>
> >>>>>>> I would like to start a discussion about KIP-1078
> >>>>>>> <
> >>>>>>
> >>
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-1078%3A+Remove+Leaking+Getter+Methods+in+Joined+Helper+Class
> >>>>>>> .
> >>>>>>> We plan to deprecate the four public APIs in
> >>>>>>> *org.apache.kafka.streams.kstream.Joined* that already have
> internal
> >>>>>>> versions and are intended for internal use only.
> >>>>>>>
> >>>>>>>
> >>>>>>> Please take a look, and if you have any suggestions or thoughts,
> feel
> >>>>>> free
> >>>>>>> to reply to this email thread.
> >>>>>>>
> >>>>>>> Thank you!
> >>>>>>>
> >>>>>>> Best regards,
> >>>>>>> TengYao
> >>>>>>>
> >>>>>>
> >>>>>
> >>>
> >>
> >
>

Reply via email to