[GitHub] [kafka] cnZach commented on a change in pull request #9236: MINOR: Log warn message with details when there's kerberos login issue

2020-09-16 Thread GitBox
cnZach commented on a change in pull request #9236: URL: https://github.com/apache/kafka/pull/9236#discussion_r489215673 ## File path: clients/src/main/java/org/apache/kafka/common/security/kerberos/KerberosLogin.java ## @@ -213,6 +213,7 @@ public LoginContext login() throws L

[jira] [Created] (KAFKA-10489) Committed consumer offsets not sent to consumer on rebalance

2020-09-16 Thread AD (Jira)
AD created KAFKA-10489: -- Summary: Committed consumer offsets not sent to consumer on rebalance Key: KAFKA-10489 URL: https://issues.apache.org/jira/browse/KAFKA-10489 Project: Kafka Issue Type: Bug

[jira] [Commented] (KAFKA-9016) Warn when log dir stopped serving replicas

2020-09-16 Thread Srikantha G K (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-9016?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17196927#comment-17196927 ] Srikantha G K commented on KAFKA-9016: -- I am new to Kafka,I am working in Windows en

[GitHub] [kafka] dongjinleekr commented on pull request #7898: KAFKA-9366: Change log4j dependency into log4j2

2020-09-16 Thread GitBox
dongjinleekr commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-693414587 @hachikuji @chia7712 @abbccdda @omkreddy @tombentley Could you have a look? :pray: This is an automated mess

[GitHub] [kafka] showuon commented on pull request #9241: MINOR: Update the javadoc in GroupMetadataManager.scala

2020-09-16 Thread GitBox
showuon commented on pull request #9241: URL: https://github.com/apache/kafka/pull/9241#issuecomment-693429851 @guozhangwang , could you help review this small PR? Thanks. This is an automated message from the Apache Git Ser

[GitHub] [kafka] chia7712 commented on a change in pull request #9262: MINOR: Fix log message when tasks directory is cleaned manually

2020-09-16 Thread GitBox
chia7712 commented on a change in pull request #9262: URL: https://github.com/apache/kafka/pull/9262#discussion_r489476268 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/StateDirectory.java ## @@ -306,40 +306,55 @@ public synchronized void cle

[GitHub] [kafka] DOJI45 commented on pull request #9247: KAFKA-10362: When resuming Streams active task with EOS, the checkpoint file is deleted

2020-09-16 Thread GitBox
DOJI45 commented on pull request #9247: URL: https://github.com/apache/kafka/pull/9247#issuecomment-693451000 > > @DOJI45 Thanks for the PR! Could you also add a unit test to complete the coverage? You can find some examples in StreamTaskTest or ProcessorStateManagerTest > > Seems t

[GitHub] [kafka] ijuma commented on pull request #7898: KAFKA-9366: Change log4j dependency into log4j2

2020-09-16 Thread GitBox
ijuma commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-693453914 @dongjinleekr Has the KIP been approved? This is an automated message from the Apache Git Service. To respond to the

[jira] [Updated] (KAFKA-10481) Consider Improving the Success Criteria of Streams' System Test StreamsBrokerBounceTest

2020-09-16 Thread Matthias J. Sax (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10481?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Matthias J. Sax updated KAFKA-10481: Component/s: system tests > Consider Improving the Success Criteria of Streams' System Tes

[GitHub] [kafka] vamossagar12 opened a new pull request #9292: adding number of messages/bytes read

2020-09-16 Thread GitBox
vamossagar12 opened a new pull request #9292: URL: https://github.com/apache/kafka/pull/9292 *More detailed description of your change, if necessary. The PR title and PR message become the squashed commit message, so use a separate comment to ping reviewers.* *Summary of test

[GitHub] [kafka] vamossagar12 commented on pull request #6669: KAFKA-8238: Adding Number of messages/bytes read

2020-09-16 Thread GitBox
vamossagar12 commented on pull request #6669: URL: https://github.com/apache/kafka/pull/6669#issuecomment-693465372 @stanislavkozlovski , unfortunately the older branch of mine got corrupted. I have created a new PR here: https://github.com/apache/kafka/pull/9292 -

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489515189 ## File path: core/src/main/scala/kafka/admin/ConsumerGroupCommand.scala ## @@ -823,7 +809,7 @@ object ConsumerGroupCommand extends Logging { c

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489515189 ## File path: core/src/main/scala/kafka/admin/ConsumerGroupCommand.scala ## @@ -823,7 +809,7 @@ object ConsumerGroupCommand extends Logging { c

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489515785 ## File path: clients/src/test/java/org/apache/kafka/common/utils/UtilsTest.java ## @@ -784,4 +787,39 @@ public void testCloseAllQuietly() { asse

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489518181 ## File path: clients/src/main/java/org/apache/kafka/common/utils/Utils.java ## @@ -1271,4 +1274,34 @@ private static byte checkRange(final byte i) {

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489531741 ## File path: clients/src/main/java/org/apache/kafka/common/utils/Utils.java ## @@ -1271,4 +1274,34 @@ private static byte checkRange(final byte i) {

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489515189 ## File path: core/src/main/scala/kafka/admin/ConsumerGroupCommand.scala ## @@ -823,7 +809,7 @@ object ConsumerGroupCommand extends Logging { c

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489518181 ## File path: clients/src/main/java/org/apache/kafka/common/utils/Utils.java ## @@ -1271,4 +1274,34 @@ private static byte checkRange(final byte i) {

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489515785 ## File path: clients/src/test/java/org/apache/kafka/common/utils/UtilsTest.java ## @@ -784,4 +787,39 @@ public void testCloseAllQuietly() { asse

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489534622 ## File path: clients/src/main/java/org/apache/kafka/common/utils/Utils.java ## @@ -1271,4 +1274,34 @@ private static byte checkRange(final byte i) {

[GitHub] [kafka] manijndl7 commented on a change in pull request #9255: MINOR: Consolidate duplicated logic on reset tools

2020-09-16 Thread GitBox
manijndl7 commented on a change in pull request #9255: URL: https://github.com/apache/kafka/pull/9255#discussion_r489531741 ## File path: clients/src/main/java/org/apache/kafka/common/utils/Utils.java ## @@ -1271,4 +1274,34 @@ private static byte checkRange(final byte i) {

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489538195 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/KTableImpl.java ## @@ -182,6 +182,8 @@ public String queryableStoreN

[GitHub] [kafka] lbradstreet closed pull request #9187: MINOR: bump mockito to 3.5.0

2020-09-16 Thread GitBox
lbradstreet closed pull request #9187: URL: https://github.com/apache/kafka/pull/9187 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 t

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489539904 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/KTableKTableAbstractJoin.java ## @@ -39,9 +39,15 @@ } @O

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r487060078 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/KTableKTableAbstractJoin.java ## @@ -39,9 +39,15 @@ } @O

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489542022 ## File path: streams/src/test/java/org/apache/kafka/streams/kstream/internals/KTableImplTest.java ## @@ -304,6 +363,30 @@ public void testStateSto

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489543472 ## File path: streams/src/test/java/org/apache/kafka/streams/kstream/internals/KTableImplTest.java ## @@ -304,6 +363,30 @@ public void testStateSto

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489544014 ## File path: streams/streams-scala/src/test/scala/org/apache/kafka/streams/scala/kstream/KTableTest.scala ## @@ -39,29 +39,27 @@ class KTableTest

[GitHub] [kafka] hachikuji merged pull request #9290: KAFKA-10487; Fetch response should return diverging epoch and end offset

2020-09-16 Thread GitBox
hachikuji merged pull request #9290: URL: https://github.com/apache/kafka/pull/9290 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

[jira] [Resolved] (KAFKA-10487) Fix edge case in Raft truncation protocol

2020-09-16 Thread Jason Gustafson (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10487?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Gustafson resolved KAFKA-10487. - Resolution: Fixed > Fix edge case in Raft truncation protocol >

[GitHub] [kafka] ableegoldman commented on a change in pull request #9060: KAFKA-9274: Remove `retries` from InternalTopicManager

2020-09-16 Thread GitBox
ableegoldman commented on a change in pull request #9060: URL: https://github.com/apache/kafka/pull/9060#discussion_r489580109 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/InternalTopicManager.java ## @@ -45,33 +49,38 @@ "Please rep

[GitHub] [kafka] ableegoldman commented on a change in pull request #9060: KAFKA-9274: Remove `retries` from InternalTopicManager

2020-09-16 Thread GitBox
ableegoldman commented on a change in pull request #9060: URL: https://github.com/apache/kafka/pull/9060#discussion_r489580572 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/InternalTopicManager.java ## @@ -45,33 +49,38 @@ "Please rep

[GitHub] [kafka] kkonstantine commented on pull request #9227: KAFKA-10439: Connect's Values to parse BigInteger as Decimal with zero scale.

2020-09-16 Thread GitBox
kkonstantine commented on pull request #9227: URL: https://github.com/apache/kafka/pull/9227#issuecomment-693533434 ok to test This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [kafka] kkonstantine commented on pull request #9227: KAFKA-10439: Connect's Values to parse BigInteger as Decimal with zero scale.

2020-09-16 Thread GitBox
kkonstantine commented on pull request #9227: URL: https://github.com/apache/kafka/pull/9227#issuecomment-693534496 Closing temporarily to attempt to trigger a jenkins build. This is an automated message from the Apache Git

[GitHub] [kafka] kkonstantine closed pull request #9227: KAFKA-10439: Connect's Values to parse BigInteger as Decimal with zero scale.

2020-09-16 Thread GitBox
kkonstantine closed pull request #9227: URL: https://github.com/apache/kafka/pull/9227 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

[GitHub] [kafka] big-andy-coates commented on a change in pull request #9156: KAFKA-10077: Filter downstream of state-store results in spurious tombstones

2020-09-16 Thread GitBox
big-andy-coates commented on a change in pull request #9156: URL: https://github.com/apache/kafka/pull/9156#discussion_r489588705 ## File path: streams/src/test/java/org/apache/kafka/streams/kstream/internals/KTableImplTest.java ## @@ -304,6 +363,30 @@ public void testStateSto

[GitHub] [kafka] dongjinleekr commented on pull request #7898: KAFKA-9366: Change log4j dependency into log4j2

2020-09-16 Thread GitBox
dongjinleekr commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-693552944 @ijuma Of course not. But I hoped to reboot the discussion. (Thanks for the kind reponse! :+1:) This is an a

[GitHub] [kafka] ijuma commented on a change in pull request #9275: KAFKA-10435; Fetch protocol changes for KIP-595

2020-09-16 Thread GitBox
ijuma commented on a change in pull request #9275: URL: https://github.com/apache/kafka/pull/9275#discussion_r489612109 ## File path: core/src/main/scala/kafka/api/ApiVersion.scala ## @@ -100,7 +100,9 @@ object ApiVersion { // Introduced StopReplicaRequest V3 containing th

[GitHub] [kafka] wcarlson5 commented on a change in pull request #9273: KAFKA-9331: changes for shutdownRequest

2020-09-16 Thread GitBox
wcarlson5 commented on a change in pull request #9273: URL: https://github.com/apache/kafka/pull/9273#discussion_r489619608 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamThread.java ## @@ -570,6 +581,21 @@ void runLoop() { }

[GitHub] [kafka] hachikuji commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
hachikuji commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489569400 ## File path: clients/src/main/resources/common/message/AlterIsrRequest.json ## @@ -0,0 +1,44 @@ +// Licensed to the Apache Software Foundation (ASF) unde

[jira] [Created] (KAFKA-10490) Make constructors public for Admin API value objects

2020-09-16 Thread Noa Resare (Jira)
Noa Resare created KAFKA-10490: -- Summary: Make constructors public for Admin API value objects Key: KAFKA-10490 URL: https://issues.apache.org/jira/browse/KAFKA-10490 Project: Kafka Issue Type:

[GitHub] [kafka] vvcephei opened a new pull request #9293: POC: Allow bypassing the cache layer in IQ

2020-09-16 Thread GitBox
vvcephei opened a new pull request #9293: URL: https://github.com/apache/kafka/pull/9293 Just a quick POC of how we can optionally bypass the caching layer in IQ. Some queries that are common in IQ but rare in the PAPI perform really poorly with the caching layer involved. We should have a

[GitHub] [kafka] vvcephei commented on a change in pull request #9293: POC: Allow bypassing the cache layer in IQ

2020-09-16 Thread GitBox
vvcephei commented on a change in pull request #9293: URL: https://github.com/apache/kafka/pull/9293#discussion_r489632127 ## File path: streams/src/main/java/org/apache/kafka/streams/state/internals/MeteredKeyValueStore.java ## @@ -46,43 +46,90 @@ * @param * @param */

[GitHub] [kafka] mjsax commented on a change in pull request #9060: KAFKA-9274: Remove `retries` from InternalTopicManager

2020-09-16 Thread GitBox
mjsax commented on a change in pull request #9060: URL: https://github.com/apache/kafka/pull/9060#discussion_r489646059 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/InternalTopicManager.java ## @@ -45,33 +49,38 @@ "Please report at

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489703810 ## File path: core/src/main/scala/kafka/cluster/Partition.scala ## @@ -1210,6 +1317,37 @@ class Partition(val topicPartition: TopicPartition, } } +

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489720971 ## File path: config/log4j.properties ## @@ -76,8 +76,8 @@ log4j.additivity.kafka.request.logger=false log4j.logger.kafka.network.RequestChannel$=WARN, requ

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489721316 ## File path: core/src/main/scala/kafka/cluster/Partition.scala ## @@ -617,9 +665,9 @@ class Partition(val topicPartition: TopicPartition, // since

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489730265 ## File path: core/src/main/scala/kafka/server/AlterIsrManager.scala ## @@ -0,0 +1,173 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489740479 ## File path: core/src/main/scala/kafka/server/AlterIsrManager.scala ## @@ -0,0 +1,173 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489744561 ## File path: core/src/main/scala/kafka/server/AlterIsrManager.scala ## @@ -0,0 +1,173 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489745539 ## File path: core/src/main/scala/kafka/server/AlterIsrManager.scala ## @@ -0,0 +1,173 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489747101 ## File path: core/src/main/scala/kafka/server/AlterIsrManager.scala ## @@ -0,0 +1,173 @@ +/** + * Licensed to the Apache Software Foundation (ASF) under one

[jira] [Created] (KAFKA-10491) Check authorizations before other criteria in KafkaApis

2020-09-16 Thread David Arthur (Jira)
David Arthur created KAFKA-10491: Summary: Check authorizations before other criteria in KafkaApis Key: KAFKA-10491 URL: https://issues.apache.org/jira/browse/KAFKA-10491 Project: Kafka Issue

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489753946 ## File path: core/src/main/scala/kafka/server/KafkaApis.scala ## @@ -3045,6 +3045,24 @@ class KafkaApis(val requestChannel: RequestChannel, } } +

[GitHub] [kafka] mumrah commented on a change in pull request #9100: Add AlterISR RPC and use it for ISR modifications

2020-09-16 Thread GitBox
mumrah commented on a change in pull request #9100: URL: https://github.com/apache/kafka/pull/9100#discussion_r489756083 ## File path: core/src/main/scala/kafka/server/ReplicaManager.scala ## @@ -1317,7 +1326,9 @@ class ReplicaManager(val config: KafkaConfig, parti

[GitHub] [kafka] jolshan opened a new pull request #9294: MINOR: Fix now that kafka.apache.org resolves to 3 IP addresses

2020-09-16 Thread GitBox
jolshan opened a new pull request #9294: URL: https://github.com/apache/kafka/pull/9294 ClusterConnectStatesTest and ClientUtilsTest were failing because they expected kafka.apache.org to resolve to 2 IP addresses. This updates the tests so they reflect that DNS resolves to 3 addresses.

[jira] [Updated] (KAFKA-10476) Error when calling (kafka-configs.sh --add-config): org.apache.kafka.common.KafkaException: Failed to create new KafkaAdminClient..... Caused by: javax.crypto.BadPadding

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud updated KAFKA-10476: Attachment: (was: Error.txt) > Error when calling (kafka-configs.sh --add-conf

[jira] [Updated] (KAFKA-10476) Error when calling (kafka-configs.sh --add-config): org.apache.kafka.common.KafkaException: Failed to create new KafkaAdminClient..... Caused by: javax.crypto.BadPadding

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud updated KAFKA-10476: Description:     was: For now We created a Cluster on AWS with MSK service, we

[jira] [Updated] (KAFKA-10476) Error when calling (kafka-configs.sh --add-config): org.apache.kafka.common.KafkaException: Failed to create new KafkaAdminClient..... Caused by: javax.crypto.BadPadding

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud updated KAFKA-10476: Attachment: (was: Script.txt) > Error when calling (kafka-configs.sh --add-con

[jira] [Updated] (KAFKA-10476) ignore it

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud updated KAFKA-10476: Summary: ignore it (was: Error when calling (kafka-configs.sh --add-config): org.

[jira] [Closed] (KAFKA-10476) ignore it

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud closed KAFKA-10476. --- > ignore it > - > > Key: KAFKA-10476 > URL: http

[jira] [Resolved] (KAFKA-10476) ignore it

2020-09-16 Thread AbdulRahman Mahmoud (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10476?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] AbdulRahman Mahmoud resolved KAFKA-10476. - Resolution: Invalid > ignore it > - > > Key: KAFKA-10476

[GitHub] [kafka] ableegoldman opened a new pull request #9295: MINOR: standardize rebalance related logging for easy discovery & debugging

2020-09-16 Thread GitBox
ableegoldman opened a new pull request #9295: URL: https://github.com/apache/kafka/pull/9295 Some minor logging adjustments to standardize the grammar of rebalance related messages and make it easy to query the logs for quick debugging results -

[GitHub] [kafka] ableegoldman commented on pull request #9295: MINOR: standardize rebalance related logging for easy discovery & debugging

2020-09-16 Thread GitBox
ableegoldman commented on pull request #9295: URL: https://github.com/apache/kafka/pull/9295#issuecomment-693727341 @guozhangwang This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [kafka] jolshan commented on pull request #9294: MINOR: Fix now that kafka.apache.org resolves to 3 IP addresses

2020-09-16 Thread GitBox
jolshan commented on pull request #9294: URL: https://github.com/apache/kafka/pull/9294#issuecomment-693727406 @abbccdda I agree that they are fragile. Could you give a little more detail about checking the patterns? I'm not sure if I follow. --

[jira] [Created] (KAFKA-10492) Core Raft implementation

2020-09-16 Thread Jason Gustafson (Jira)
Jason Gustafson created KAFKA-10492: --- Summary: Core Raft implementation Key: KAFKA-10492 URL: https://issues.apache.org/jira/browse/KAFKA-10492 Project: Kafka Issue Type: Sub-task

[GitHub] [kafka] kkonstantine commented on a change in pull request #9104: KAFKA-10266: Update the connector config header.converter

2020-09-16 Thread GitBox
kkonstantine commented on a change in pull request #9104: URL: https://github.com/apache/kafka/pull/9104#discussion_r489834773 ## File path: connect/runtime/src/main/java/org/apache/kafka/connect/runtime/ConnectorConfig.java ## @@ -89,11 +89,16 @@ public static final Stri

[GitHub] [kafka] abbccdda commented on pull request #9103: Add redirection for (Incremental)AlterConfig, AlterClientQuota and CreateTopics

2020-09-16 Thread GitBox
abbccdda commented on pull request #9103: URL: https://github.com/apache/kafka/pull/9103#issuecomment-693757638 retest this please This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [kafka] abbccdda commented on a change in pull request #9103: Add redirection for (Incremental)AlterConfig, AlterClientQuota and CreateTopics

2020-09-16 Thread GitBox
abbccdda commented on a change in pull request #9103: URL: https://github.com/apache/kafka/pull/9103#discussion_r489875283 ## File path: core/src/main/scala/kafka/server/KafkaConfig.scala ## @@ -1542,6 +1542,9 @@ class KafkaConfig(val props: java.util.Map[_, _], doLog: Boolean

[GitHub] [kafka] chia7712 opened a new pull request #9296: MINOR: remove unused scala files from core module

2020-09-16 Thread GitBox
chia7712 opened a new pull request #9296: URL: https://github.com/apache/kafka/pull/9296 ```NoEpochForPartitionException``` and ```ClientIdAndTopic``` are not used anymore. ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Veri

[GitHub] [kafka] chia7712 commented on a change in pull request #9292: adding number of messages/bytes read

2020-09-16 Thread GitBox
chia7712 commented on a change in pull request #9292: URL: https://github.com/apache/kafka/pull/9292#discussion_r489931752 ## File path: core/src/main/scala/kafka/coordinator/group/GroupMetadataManager.scala ## @@ -670,12 +675,16 @@ class GroupMetadataManager(brokerId: Int,

[GitHub] [kafka] vamossagar12 commented on a change in pull request #9292: adding number of messages/bytes read

2020-09-16 Thread GitBox
vamossagar12 commented on a change in pull request #9292: URL: https://github.com/apache/kafka/pull/9292#discussion_r489956487 ## File path: core/src/main/scala/kafka/coordinator/group/GroupMetadataManager.scala ## @@ -670,12 +675,16 @@ class GroupMetadataManager(brokerId: Int

[jira] [Commented] (KAFKA-10485) Use a separate error code for replication related errors

2020-09-16 Thread Chia-Ping Tsai (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10485?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17197368#comment-17197368 ] Chia-Ping Tsai commented on KAFKA-10485: REQUEST_TIMED_OUT is viewed as fatal er

[jira] [Updated] (KAFKA-10490) Make constructors public for Admin API value objects

2020-09-16 Thread Tom Bentley (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-10490?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tom Bentley updated KAFKA-10490: Labels: needs-kip (was: ) > Make constructors public for Admin API value objects > --

[jira] [Commented] (KAFKA-6791) Add a CoordinatorNodeProvider in KafkaAdminClient

2020-09-16 Thread Jeff Widman (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-6791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17197411#comment-17197411 ] Jeff Widman commented on KAFKA-6791: Should this be closed as "wontfix"? The associ

[GitHub] [kafka] tombentley commented on pull request #9266: KAFKA-10469: Resolve logger levels hierarchically

2020-09-16 Thread GitBox
tombentley commented on pull request #9266: URL: https://github.com/apache/kafka/pull/9266#issuecomment-693952069 @ijuma @hachikuji @guozhangwang @mjsax @mimaison please could one of you could take a look? This is an automa

[GitHub] [kafka] tombentley commented on pull request #9252: KAFKA-10241: Add test for message compatibility

2020-09-16 Thread GitBox
tombentley commented on pull request #9252: URL: https://github.com/apache/kafka/pull/9252#issuecomment-693953116 @abbccdda or perhaps @cmccabe or @dajac any chance one of you could take a look? This is an automated message

[GitHub] [kafka] tombentley commented on pull request #8878: MINOR: Generator config-specific HTML ids

2020-09-16 Thread GitBox
tombentley commented on pull request #8878: URL: https://github.com/apache/kafka/pull/8878#issuecomment-693954974 @mimaison @omkreddy could one of you do the build again? This is an automated message from the Apache Git Serv

[jira] [Created] (KAFKA-10493) Ktable out-of-order updates are not being ignored

2020-09-16 Thread Pedro Gontijo (Jira)
Pedro Gontijo created KAFKA-10493: - Summary: Ktable out-of-order updates are not being ignored Key: KAFKA-10493 URL: https://issues.apache.org/jira/browse/KAFKA-10493 Project: Kafka Issue Typ