[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-08-06 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1667163104 Hey @junrao - The most recent build, after merging trunk in, seems reasonably clean. Here are the broken tests, but all seem unrelated to the change ``` Build / JDK 11 and

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-08-02 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1663201841 There seems to be some weirdness to the build - jdk 17 was timedout or something. Retriggered the build... -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-08-01 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1661525036 @junrao - Thank you so much for the reviews. The latest commit addressed your comments. Let me know if you have more. -- This is an automated message from the Apache Git Service. To

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-08-01 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1660477317 latest test failures are unrelated ``` New failing - 10 Build / JDK 17 and Scala 2.13 / testBumpTransactionalEpoch(String).quorum=kraft – kafka.api.TransactionsTest 1m 17s

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-31 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1659400843 Here are the most recent failures: ``` Build / JDK 11 and Scala 2.13 / testClose() – kafka.log.remote.RemoteIndexCacheTest <1s Build / JDK 11 and Scala 2.13 /

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-31 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1659084728 Current failures: ``` New failing - 12 Build / JDK 11 and Scala 2.13 / [1] tlsProtocol=TLSv1.2, useInlinePem=false – org.apache.kafka.common.network.SslTransportLayerTest

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1656346526 Umm build failures for jdk 8 plus some non-related test failures: ``` Build / JDK 20 and Scala 2.13 / testSyncTopicConfigs() –

[GitHub] [kafka] philipnee commented on pull request #14123: MINOR: Fix committed API in the PrototypeAsyncConsumer timeout

2023-07-28 Thread via GitHub
philipnee commented on PR #14123: URL: https://github.com/apache/kafka/pull/14123#issuecomment-1656345210 We don't. Do you know when was is introduced? Is it in your inflight PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to