[GitHub] [kafka] SpacRocket commented on pull request #13382: KAFKA-14722: Make BooleanSerde public

2023-04-10 Thread via GitHub
SpacRocket commented on PR #13382: URL: https://github.com/apache/kafka/pull/13382#issuecomment-1502183087 @mjsax Sure thanks for the suggestion. I've pushed a new commit in this pull request: https://github.com/apache/kafka/pull/13491 -- This is an automated message from the Apache

[GitHub] [kafka] SpacRocket commented on pull request #13382: KAFKA-14722: Make BooleanSerde public

2023-03-29 Thread via GitHub
SpacRocket commented on PR #13382: URL: https://github.com/apache/kafka/pull/13382#issuecomment-1489249233 @wcarlson5 Thank you for your time, a question what do you mean exactly by upgrade notes? I've searched for it in wiki and couldn't find it: The KIP is currently in this

[GitHub] [kafka] SpacRocket commented on pull request #13382: KAFKA-14722: Make BooleanSerde public

2023-03-23 Thread via GitHub
SpacRocket commented on PR #13382: URL: https://github.com/apache/kafka/pull/13382#issuecomment-1481980186 Apple M1: JDK 17, Scala 2.13 testDeleteInternalTopic [1st test] **(2/2 tests passed)** testDescribeQuorumStatusSuccesful [2nd test] **(6/6 tests passed)**

[GitHub] [kafka] SpacRocket commented on pull request #13382: KAFKA-14722: Make BooleanSerde public

2023-03-23 Thread via GitHub
SpacRocket commented on PR #13382: URL: https://github.com/apache/kafka/pull/13382#issuecomment-1480696323 Hello I've looked into the pr-merge and the only warning that I see is this: [2023-03-22T22:21:56.566Z] Recording test results [2023-03-22T22:22:14.166Z] [Checks API] No suitable