[GitHub] [kafka] dajac commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
dajac commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r858303688 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[GitHub] [kafka] guozhangwang commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
guozhangwang commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r858263213 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[jira] [Commented] (KAFKA-13716) add tests for `DeleteRecordsCommand` class

2022-04-25 Thread Luke Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527866#comment-17527866 ] Luke Chen commented on KAFKA-13716: --- [~divijvaidya] , I saw you have another PR to address this issue:

[GitHub] [kafka] philipnee closed pull request #12088: KAFKA-12841 bug fix and tests

2022-04-25 Thread GitBox
philipnee closed pull request #12088: KAFKA-12841 bug fix and tests URL: https://github.com/apache/kafka/pull/12088 -- 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

[jira] [Commented] (KAFKA-13856) MirrorCheckpointTask meets ConcurrentModificationException

2022-04-25 Thread Justinwins (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527841#comment-17527841 ] Justinwins commented on KAFKA-13856: HashMap cannot be "put" and traversed by two threads

[GitHub] [kafka] arunmathew88 commented on pull request #11438: KAFKA-13403 Fix KafkaServer crashes when deleting topics due to the race in log deletion

2022-04-25 Thread GitBox
arunmathew88 commented on PR #11438: URL: https://github.com/apache/kafka/pull/11438#issuecomment-1109219047 retest this please -- 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

[jira] [Created] (KAFKA-13856) MirrorCheckpointTask meets ConcurrentModificationException

2022-04-25 Thread Justinwins (Jira)
Justinwins created KAFKA-13856: -- Summary: MirrorCheckpointTask meets ConcurrentModificationException Key: KAFKA-13856 URL: https://issues.apache.org/jira/browse/KAFKA-13856 Project: Kafka Issue

[GitHub] [kafka] dengziming commented on a diff in pull request #12072: KAFKA-13854 Refactor ApiVersion to MetadataVersion

2022-04-25 Thread GitBox
dengziming commented on code in PR #12072: URL: https://github.com/apache/kafka/pull/12072#discussion_r858159547 ## server-common/src/main/java/org/apache/kafka/server/common/MetadataVersionValidator.java: ## @@ -0,0 +1,40 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] [kafka] junrao commented on a diff in pull request #12029: KAFKA-13815: Avoid reinitialization for a replica that is being deleted

2022-04-25 Thread GitBox
junrao commented on code in PR #12029: URL: https://github.com/apache/kafka/pull/12029#discussion_r858099615 ## core/src/main/scala/kafka/log/UnifiedLog.scala: ## @@ -674,21 +682,29 @@ class UnifiedLog(@volatile var logStartOffset: Long, } /** - * Rename the directory

[jira] [Commented] (KAFKA-13403) KafkaServer crashes when deleting topics due to the race in log deletion

2022-04-25 Thread Haruki Okada (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13403?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527800#comment-17527800 ] Haruki Okada commented on KAFKA-13403: -- [~showuon] Hi, could you help reviewing the PR

[jira] [Commented] (KAFKA-13855) FileNotFoundException: Error while rolling log segment for topic partition in dir

2022-04-25 Thread Haruki Okada (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527797#comment-17527797 ] Haruki Okada commented on KAFKA-13855: -- I guess that's same cause as

[GitHub] [kafka] junrao commented on pull request #12064: KAFKA-12841: Remove an additional call of onAcknowledgement

2022-04-25 Thread GitBox
junrao commented on PR #12064: URL: https://github.com/apache/kafka/pull/12064#issuecomment-1109120987 @cadonna : Could we merge this PR to 3.2? It fixes an issue in 3.2. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [kafka] junrao merged pull request #12064: KAFKA-12841: Remove an additional call of onAcknowledgement

2022-04-25 Thread GitBox
junrao merged PR #12064: URL: https://github.com/apache/kafka/pull/12064 -- 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] jsancio merged pull request #11732: MINOR: enable KRaft in ConfigCommandIntegrationTest

2022-04-25 Thread GitBox
jsancio merged PR #11732: URL: https://github.com/apache/kafka/pull/11732 -- 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] mt206 opened a new pull request, #12094: MINOR: fix html generation syntax errors

2022-04-25 Thread GitBox
mt206 opened a new pull request, #12094: URL: https://github.com/apache/kafka/pull/12094 *More detailed description of your change, The html document generation has some errors in it, specifically related to protocols. The two issues identified and resolved are: - Missing ``

[GitHub] [kafka] dajac commented on a diff in pull request #12029: KAFKA-13815: Avoid reinitialization for a replica that is being deleted

2022-04-25 Thread GitBox
dajac commented on code in PR #12029: URL: https://github.com/apache/kafka/pull/12029#discussion_r858031776 ## core/src/main/scala/kafka/log/UnifiedLog.scala: ## @@ -674,21 +682,29 @@ class UnifiedLog(@volatile var logStartOffset: Long, } /** - * Rename the directory

[GitHub] [kafka] dajac commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
dajac commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r858024381 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[GitHub] [kafka] dajac commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
dajac commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r858019261 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[GitHub] [kafka] ahuang98 commented on a diff in pull request #12072: KAFKA-13854 Refactor ApiVersion to MetadataVersion

2022-04-25 Thread GitBox
ahuang98 commented on code in PR #12072: URL: https://github.com/apache/kafka/pull/12072#discussion_r858015440 ## server-common/src/test/java/org/apache/kafka/server/common/MetadataVersionTest.java: ## @@ -0,0 +1,361 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] [kafka] dajac merged pull request #12081: MINOR: Refactor `kafka.cluster.Replica`

2022-04-25 Thread GitBox
dajac merged PR #12081: URL: https://github.com/apache/kafka/pull/12081 -- 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] mumrah commented on a diff in pull request #12072: KAFKA-13854 Refactor ApiVersion to MetadataVersion

2022-04-25 Thread GitBox
mumrah commented on code in PR #12072: URL: https://github.com/apache/kafka/pull/12072#discussion_r857970386 ## server-common/src/main/java/org/apache/kafka/server/common/MetadataVersion.java: ## @@ -0,0 +1,280 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under

[jira] [Commented] (KAFKA-13716) add tests for `DeleteRecordsCommand` class

2022-04-25 Thread Richard Joerger (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13716?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527730#comment-17527730 ] Richard Joerger commented on KAFKA-13716: - Hi Team,   I'd like to work on this particular

[GitHub] [kafka] guozhangwang commented on pull request #12041: MINOR: ignore unused configuration when ConsumerCoordinator is not constructed

2022-04-25 Thread GitBox
guozhangwang commented on PR #12041: URL: https://github.com/apache/kafka/pull/12041#issuecomment-1108911987 @C0urante Yes I agree this is a fair question to debate on, and I personally think that it could be very subjective --- as you can see in

[GitHub] [kafka] guozhangwang commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
guozhangwang commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r857910667 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[GitHub] [kafka] guozhangwang commented on a diff in pull request #12035: KAFKA-13217: Reconsider skipping the LeaveGroup on close() or add an overload that does so

2022-04-25 Thread GitBox
guozhangwang commented on code in PR #12035: URL: https://github.com/apache/kafka/pull/12035#discussion_r857908050 ## streams/src/main/java/org/apache/kafka/streams/KafkaStreams.java: ## @@ -1498,6 +1516,37 @@ public synchronized boolean close(final Duration timeout) throws

[jira] [Updated] (KAFKA-13855) FileNotFoundException: Error while rolling log segment for topic partition in dir

2022-04-25 Thread Sergey Ivanov (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13855?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Sergey Ivanov updated KAFKA-13855: -- Description: Hello, We faced an issue when one of Kafka broker in cluster has failed with an

[jira] [Created] (KAFKA-13855) FileNotFoundException: Error while rolling log segment for topic partition in dir

2022-04-25 Thread Sergey Ivanov (Jira)
Sergey Ivanov created KAFKA-13855: - Summary: FileNotFoundException: Error while rolling log segment for topic partition in dir Key: KAFKA-13855 URL: https://issues.apache.org/jira/browse/KAFKA-13855

[GitHub] [kafka] junrao commented on a diff in pull request #12029: KAFKA-13815: Avoid reinitialization for a replica that is being deleted

2022-04-25 Thread GitBox
junrao commented on code in PR #12029: URL: https://github.com/apache/kafka/pull/12029#discussion_r857864434 ## core/src/test/scala/unit/kafka/log/UnifiedLogTest.scala: ## @@ -3325,6 +3325,32 @@ class UnifiedLogTest { assertEquals(1, log.numberOfSegments) } + /** +

[GitHub] [kafka] hachikuji commented on a diff in pull request #11951: KAFKA-13836: Improve KRaft broker heartbeat logic

2022-04-25 Thread GitBox
hachikuji commented on code in PR #11951: URL: https://github.com/apache/kafka/pull/11951#discussion_r857852045 ## core/src/main/scala/kafka/server/BrokerLifecycleManager.scala: ## @@ -333,7 +353,11 @@ class BrokerLifecycleManager(val config: KafkaConfig, } private def

[GitHub] [kafka] hachikuji merged pull request #12032: MINOR: Change `AlterPartition` validation order in `KafkaController`

2022-04-25 Thread GitBox
hachikuji merged PR #12032: URL: https://github.com/apache/kafka/pull/12032 -- 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] ahuang98 commented on a diff in pull request #12072: KAFKA-13854 Refactor ApiVersion to MetadataVersion

2022-04-25 Thread GitBox
ahuang98 commented on code in PR #12072: URL: https://github.com/apache/kafka/pull/12072#discussion_r857807415 ## server-common/src/main/java/org/apache/kafka/server/common/MetadataVersionValidator.java: ## @@ -0,0 +1,40 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] [kafka] C0urante commented on pull request #11773: KAFKA-10000: Add new source connector APIs related to exactly-once support (KIP-618)

2022-04-25 Thread GitBox
C0urante commented on PR #11773: URL: https://github.com/apache/kafka/pull/11773#issuecomment-1108752944 @tombentley @mimaison I'd really appreciate another pass if you can find the time this week. Thanks! -- This is an automated message from the Apache Git Service. To respond to the

[jira] [Created] (KAFKA-13854) Refactor ApiVersion to MetadataVersion

2022-04-25 Thread David Arthur (Jira)
David Arthur created KAFKA-13854: Summary: Refactor ApiVersion to MetadataVersion Key: KAFKA-13854 URL: https://issues.apache.org/jira/browse/KAFKA-13854 Project: Kafka Issue Type: Sub-task

[GitHub] [kafka] mumrah commented on pull request #12072: [WIP] KRaft upgrade - MetadataVersion

2022-04-25 Thread GitBox
mumrah commented on PR #12072: URL: https://github.com/apache/kafka/pull/12072#issuecomment-1108659307 I fixed a conflict from `trunk` so the build could run -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [kafka] mumrah commented on a diff in pull request #12072: [WIP] KRaft upgrade - MetadataVersion

2022-04-25 Thread GitBox
mumrah commented on code in PR #12072: URL: https://github.com/apache/kafka/pull/12072#discussion_r857673849 ## core/src/test/scala/unit/kafka/api/ApiVersionTest.scala: ## @@ -184,6 +184,37 @@ class ApiVersionTest { assertEquals("3.2", KAFKA_3_2_IV0.shortVersion) } +

[GitHub] [kafka] divijvaidya commented on pull request #12045: KAFKA-12319: Change calculation of window size used to calculate `Rate`

2022-04-25 Thread GitBox
divijvaidya commented on PR #12045: URL: https://github.com/apache/kafka/pull/12045#issuecomment-1108621881 Hey @apovzner @dajac, did you get a chance to take a look at this? Please let me know if I can make explanation simpler or if you have any questions. -- This is an automated

[GitHub] [kafka] viktorsomogyi commented on a diff in pull request #10738: KAFKA-6945: KIP-373, allow users to create delegation token for others

2022-04-25 Thread GitBox
viktorsomogyi commented on code in PR #10738: URL: https://github.com/apache/kafka/pull/10738#discussion_r857632186 ## core/src/test/scala/unit/kafka/security/token/delegation/DelegationTokenManagerTest.scala: ## @@ -315,13 +314,20 @@ class DelegationTokenManagerTest extends

[GitHub] [kafka] C0urante commented on a diff in pull request #11983: KAFKA-13763 (2): Refactor IncrementalCooperativeAssignor for improved unit testing

2022-04-25 Thread GitBox
C0urante commented on code in PR #11983: URL: https://github.com/apache/kafka/pull/11983#discussion_r857610529 ## connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/IncrementalCooperativeAssignorTest.java: ## @@ -1375,24 +1171,6 @@ private void

[GitHub] [kafka] viktorsomogyi commented on a diff in pull request #10738: KAFKA-6945: KIP-373, allow users to create delegation token for others

2022-04-25 Thread GitBox
viktorsomogyi commented on code in PR #10738: URL: https://github.com/apache/kafka/pull/10738#discussion_r857571909 ## core/src/main/scala/kafka/security/authorizer/AclEntry.scala: ## @@ -100,9 +100,10 @@ object AclEntry { resourceType match { case

[GitHub] [kafka] vamossagar12 commented on pull request #11211: KAFKA-12960: Enforcing strict retention time for WindowStore and Sess…

2022-04-25 Thread GitBox
vamossagar12 commented on PR #11211: URL: https://github.com/apache/kafka/pull/11211#issuecomment-1108243111 @showuon i have pushed the commit as discussed. Plz review whenever you get the chance -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [kafka] vamossagar12 commented on a diff in pull request #11211: KAFKA-12960: Enforcing strict retention time for WindowStore and Sess…

2022-04-25 Thread GitBox
vamossagar12 commented on code in PR #11211: URL: https://github.com/apache/kafka/pull/11211#discussion_r857375620 ## streams/src/test/java/org/apache/kafka/streams/state/internals/AbstractDualSchemaRocksDBSegmentedBytesStoreTest.java: ## @@ -157,43 +157,34 @@ public void

[GitHub] [kafka] vamossagar12 commented on a diff in pull request #11211: KAFKA-12960: Enforcing strict retention time for WindowStore and Sess…

2022-04-25 Thread GitBox
vamossagar12 commented on code in PR #11211: URL: https://github.com/apache/kafka/pull/11211#discussion_r857375173 ## streams/src/main/java/org/apache/kafka/streams/state/internals/AbstractRocksDBSegmentedBytesStore.java: ## @@ -234,7 +275,11 @@ public void put(final Bytes key,

[GitHub] [kafka] divijvaidya commented on pull request #12087: KAFKA-13851: Add integration tests for DeleteRecords API

2022-04-25 Thread GitBox
divijvaidya commented on PR #12087: URL: https://github.com/apache/kafka/pull/12087#issuecomment-1108235535 Known flaky tests failing unrelated to this change ``` Build / JDK 17 and Scala 2.13 / shouldBootstrapTwoBrokersWithFollowerThrottle() – kafka.server.ReplicationQuotasTest

[jira] [Commented] (KAFKA-13840) KafkaConsumer is unable to recover connection to group coordinator after commitOffsetsAsync exception

2022-04-25 Thread Luke Chen (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13840?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17527348#comment-17527348 ] Luke Chen commented on KAFKA-13840: --- [~kyle.stehbens] , thanks for the reply. If it is using

[GitHub] [kafka] showuon commented on a diff in pull request #11983: KAFKA-13763 (2): Refactor IncrementalCooperativeAssignor for improved unit testing

2022-04-25 Thread GitBox
showuon commented on code in PR #11983: URL: https://github.com/apache/kafka/pull/11983#discussion_r857284813 ## connect/runtime/src/test/java/org/apache/kafka/connect/runtime/distributed/IncrementalCooperativeAssignorTest.java: ## @@ -1375,24 +1171,6 @@ private void