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