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