[GitHub] [kafka] satishd commented on pull request #13309: MINOR Moved a few log segment util methods from LocalLog to LogFileUtils

2023-03-04 Thread via GitHub
satishd commented on PR #13309: URL: https://github.com/apache/kafka/pull/13309#issuecomment-1455007789 > @satishd How is this related to moving `LogSegment`? It doesn't seem like that was updated at all? Methods like the below have usages in `LogSegment` class. Those will be

[jira] [Commented] (KAFKA-14754) improve consumer under example package

2023-03-04 Thread Philip Nee (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17696530#comment-17696530 ] Philip Nee commented on KAFKA-14754: Hey [~showuon] - for #1 - perhaps you mean by there no

[GitHub] [kafka] philipnee commented on a diff in pull request #13303: KAFKA-14761 Adding integration test for the prototype consumer

2023-03-04 Thread via GitHub
philipnee commented on code in PR #13303: URL: https://github.com/apache/kafka/pull/13303#discussion_r1125607022 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/PrototypeAsyncConsumer.java: ## @@ -226,11 +246,20 @@ public void

[GitHub] [kafka] philipnee commented on a diff in pull request #13303: KAFKA-14761 Adding integration test for the prototype consumer

2023-03-04 Thread via GitHub
philipnee commented on code in PR #13303: URL: https://github.com/apache/kafka/pull/13303#discussion_r1125606614 ## core/src/test/scala/integration/kafka/api/PlaintextConsumerTest.scala: ## @@ -993,7 +993,7 @@ class PlaintextConsumerTest extends BaseConsumerTest { //

[GitHub] [kafka] philipnee commented on a diff in pull request #13303: KAFKA-14761 Adding integration test for the prototype consumer

2023-03-04 Thread via GitHub
philipnee commented on code in PR #13303: URL: https://github.com/apache/kafka/pull/13303#discussion_r1125606496 ## core/src/test/scala/integration/kafka/api/AbstractConsumerTest.scala: ## @@ -97,7 +97,7 @@ abstract class AbstractConsumerTest extends BaseRequestTest { }

[GitHub] [kafka] ijuma merged pull request #13339: MINOR: Update gradle to 8.0.2 and update several gradle plugins

2023-03-04 Thread via GitHub
ijuma merged PR #13339: URL: https://github.com/apache/kafka/pull/13339 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [kafka] ijuma commented on pull request #13312: KAFKA-14766: Improve performance of VarInt encoding and decoding

2023-03-04 Thread via GitHub
ijuma commented on PR #13312: URL: https://github.com/apache/kafka/pull/13312#issuecomment-1454952744 @divijvaidya The following is also interesting https://github.com/astei/varint-writing-showdown/issues/1 -- This is an automated message from the Apache Git Service. To respond to the

[jira] [Assigned] (KAFKA-14059) Replace EasyMock and PowerMock with Mockito in WorkerSourceTaskTest

2023-03-04 Thread Hector Geraldino (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-14059?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hector Geraldino reassigned KAFKA-14059: Assignee: Hector Geraldino > Replace EasyMock and PowerMock with Mockito in

[GitHub] [kafka] ijuma commented on pull request #13309: MINOR Moved a few log segment util methods from LocalLog to LogFileUtils

2023-03-04 Thread via GitHub
ijuma commented on PR #13309: URL: https://github.com/apache/kafka/pull/13309#issuecomment-1454850372 @satishd How is this related to moving `LogSegment`? It doesn't seem like that was updated at all? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [kafka] ijuma commented on pull request #13309: MINOR Moved a few log segment util methods from LocalLog to LogFileUtils

2023-03-04 Thread via GitHub
ijuma commented on PR #13309: URL: https://github.com/apache/kafka/pull/13309#issuecomment-1454850030 I didn't see this until now. I actually think the right approach is to push some of these methods to the respective classes (eg if it's related to an index, the method should be in the

[GitHub] [kafka] dejan2609 commented on pull request #13339: MINOR: version upgrades: zinc 1.7.2 -->> 1.8.0, gradle 8.0.1 -->> 8.0.2 and gradle plugins

2023-03-04 Thread via GitHub
dejan2609 commented on PR #13339: URL: https://github.com/apache/kafka/pull/13339#issuecomment-1454847422 And i was just about to push the same change :) You got me while I was testing it. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kafka] ijuma commented on pull request #13339: MINOR: version upgrades: zinc 1.7.2 -->> 1.8.0, gradle 8.0.1 -->> 8.0.2 and gradle plugins

2023-03-04 Thread via GitHub
ijuma commented on PR #13339: URL: https://github.com/apache/kafka/pull/13339#issuecomment-1454845324 I pushed a commit that removes the gradle workaround that is no longer necessary. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [kafka] dajac commented on a diff in pull request #13240: KAFKA-14690: Add topic IDs to the OffsetCommit API version 9

2023-03-04 Thread via GitHub
dajac commented on code in PR #13240: URL: https://github.com/apache/kafka/pull/13240#discussion_r1125501983 ## clients/src/main/java/org/apache/kafka/clients/consumer/internals/ConsumerCoordinator.java: ## @@ -1328,13 +1335,16 @@ RequestFuture sendOffsetCommitRequest(final

[GitHub] [kafka] dajac commented on pull request #13240: KAFKA-14690: Add topic IDs to the OffsetCommit API version 9

2023-03-04 Thread via GitHub
dajac commented on PR #13240: URL: https://github.com/apache/kafka/pull/13240#issuecomment-1454803355 I think that we have to do 2) anyway because topic ids may not be enabled on the server side. -- This is an automated message from the Apache Git Service. To respond to the message,

[jira] [Commented] (KAFKA-13882) Dockerfile for previewing website

2023-03-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13882?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17696454#comment-17696454 ] ASF GitHub Bot commented on KAFKA-13882: lucasbru commented on PR #410: URL:

[jira] [Commented] (KAFKA-13882) Dockerfile for previewing website

2023-03-04 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13882?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17696443#comment-17696443 ] ASF GitHub Bot commented on KAFKA-13882: qingwei91 commented on PR #410: URL:

[GitHub] [kafka] Hangleton commented on pull request #13240: KAFKA-14690: Add topic IDs to the OffsetCommit API version 9

2023-03-04 Thread via GitHub
Hangleton commented on PR #13240: URL: https://github.com/apache/kafka/pull/13240#issuecomment-1454676488 Hello David, found a case where the use of OffsetCommit requests version 9 from the consumer resulted in an error in the ACL authorization tests. The reason is that when the commit