[GitHub] [kafka] showuon commented on a change in pull request #11705: KAFKA-9847: add config to set default store type

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11705: URL: https://github.com/apache/kafka/pull/11705#discussion_r811658823 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/SlidingWindowedCogroupedKStreamImpl.java ## @@ -113,14 +114,23 @@

[GitHub] [kafka] showuon commented on a change in pull request #11705: KAFKA-9847: add config to set default store type

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11705: URL: https://github.com/apache/kafka/pull/11705#discussion_r811658823 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/SlidingWindowedCogroupedKStreamImpl.java ## @@ -113,14 +114,23 @@

[GitHub] [kafka] lkokhreidze commented on a change in pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
lkokhreidze commented on a change in pull request #10851: URL: https://github.com/apache/kafka/pull/10851#discussion_r811652014 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/ClientTagAwareStandbyTaskAssignor.java ## @@ -0,0 +1,296

[GitHub] [kafka] showuon commented on a change in pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
showuon commented on a change in pull request #10851: URL: https://github.com/apache/kafka/pull/10851#discussion_r811651144 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/ClientTagAwareStandbyTaskAssignor.java ## @@ -0,0 +1,296 @@ +

[GitHub] [kafka] showuon commented on a change in pull request #11705: KAFKA-9847: add config to set default store type

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11705: URL: https://github.com/apache/kafka/pull/11705#discussion_r811648912 ## File path: streams/src/main/java/org/apache/kafka/streams/kstream/internals/MaterializedInternal.java ## @@ -56,6 +69,10 @@ public String storeName()

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

2022-02-21 Thread GitBox
Indupa commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-1047501444 Currently in kafka we have not used Socketserver,SMTPAppender,JDBCAppender and JMSSink features from log4j,unless user explicitly use it in their Custom changes.?

[GitHub] [kafka] lkokhreidze commented on a change in pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
lkokhreidze commented on a change in pull request #10851: URL: https://github.com/apache/kafka/pull/10851#discussion_r811645496 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/ClientTagAwareStandbyTaskAssignor.java ## @@ -0,0 +1,296

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

2022-02-21 Thread GitBox
Indupa commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-1047496080 Ohhh OKk, In that case none of the 4 vulnerable things ,used in kafka as of now right.? -- This is an automated message from the Apache Git Service. To respo

[GitHub] [kafka] showuon commented on a change in pull request #11788: KAFKA-13673: disable idempotence when config conflicts

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11788: URL: https://github.com/apache/kafka/pull/11788#discussion_r811623527 ## File path: clients/src/main/java/org/apache/kafka/clients/producer/ProducerConfig.java ## @@ -276,8 +276,8 @@

[GitHub] [kafka] showuon commented on pull request #11788: KAFKA-13673: disable idempotence when config conflicts

2022-02-21 Thread GitBox
showuon commented on pull request #11788: URL: https://github.com/apache/kafka/pull/11788#issuecomment-1047478002 @mimaison , updated the PR in this commit: https://github.com/apache/kafka/pull/11788/commits/29bb358a2e63046ea332981e8a5864caa9b25e78. Thanks. -- This is an automated messa

[GitHub] [kafka] showuon commented on a change in pull request #11788: KAFKA-13673: disable idempotence when config conflicts

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11788: URL: https://github.com/apache/kafka/pull/11788#discussion_r811623527 ## File path: clients/src/main/java/org/apache/kafka/clients/producer/ProducerConfig.java ## @@ -276,8 +276,8 @@

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

2022-02-21 Thread GitBox
dongjinleekr commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-1047473190 @Indupa As long as the user explicitly configure the loggers to use them, they are not used yet. -- This is an automated message from the Apache Git Service. To respond to

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

2022-02-21 Thread GitBox
Indupa commented on pull request #7898: URL: https://github.com/apache/kafka/pull/7898#issuecomment-1047472293 Hi @dongjinleekr , Can you please provide me Some information on the Vulenerabiities regarding log4j features , Is this following vulnerable features of log4j are using in Kafka 2

[GitHub] [kafka] showuon edited a comment on pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
showuon edited a comment on pull request #11493: URL: https://github.com/apache/kafka/pull/11493#issuecomment-1047469644 Failed tests are flaky tests that also failed in trunk branch: ``` [Build / JDK 17 and Scala 2.13 / kafka.server.DynamicBrokerReconfigurationTest.testThreadPoolResi

[GitHub] [kafka] showuon commented on pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
showuon commented on pull request #11493: URL: https://github.com/apache/kafka/pull/11493#issuecomment-1047469644 Failed tests are flaky tests: ``` [Build / JDK 17 and Scala 2.13 / kafka.server.DynamicBrokerReconfigurationTest.testThreadPoolResize()](https://ci-builds.apache.org/job/K

[GitHub] [kafka] showuon edited a comment on pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
showuon edited a comment on pull request #11493: URL: https://github.com/apache/kafka/pull/11493#issuecomment-1047469153 @tim-patterson , could we rename the PR title? It seems that we don't just assign standby tasks to `threads without active tasks`. Please help update it. Thanks. -- T

[GitHub] [kafka] showuon commented on pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
showuon commented on pull request #11493: URL: https://github.com/apache/kafka/pull/11493#issuecomment-1047469153 @tim-patterson , could we rename the PR title? It seems that we not just assign standby tasks to `threads without active tasks`. Please help update it. Thanks. -- This is an

[GitHub] [kafka] showuon commented on a change in pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11493: URL: https://github.com/apache/kafka/pull/11493#discussion_r811606664 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamsPartitionAssignor.java ## @@ -1028,116 +1045,113 @@ private boole

[GitHub] [kafka] showuon commented on a change in pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
showuon commented on a change in pull request #10851: URL: https://github.com/apache/kafka/pull/10851#discussion_r811556641 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/ClientTagAwareStandbyTaskAssignor.java ## @@ -0,0 +1,296 @@ +

[GitHub] [kafka] showuon commented on a change in pull request #11642: MINOR: Improve Connect docs

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11642: URL: https://github.com/apache/kafka/pull/11642#discussion_r811545842 ## File path: connect/runtime/src/main/java/org/apache/kafka/connect/tools/PredicateDoc.java ## @@ -61,7 +61,7 @@ private static void printPredicateHtml(

[GitHub] [kafka] showuon commented on a change in pull request #11642: MINOR: Improve Connect docs

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11642: URL: https://github.com/apache/kafka/pull/11642#discussion_r811545842 ## File path: connect/runtime/src/main/java/org/apache/kafka/connect/tools/PredicateDoc.java ## @@ -61,7 +61,7 @@ private static void printPredicateHtml(

[GitHub] [kafka] showuon merged pull request #11794: MINOR: Add links to connector configs in TOC

2022-02-21 Thread GitBox
showuon merged pull request #11794: URL: https://github.com/apache/kafka/pull/11794 -- 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: jira-unsubscr.

[GitHub] [kafka] showuon commented on a change in pull request #11794: MINOR: Add links to connector configs in TOC

2022-02-21 Thread GitBox
showuon commented on a change in pull request #11794: URL: https://github.com/apache/kafka/pull/11794#discussion_r811537471 ## File path: docs/toc.html ## @@ -45,6 +45,10 @@ 3.3 Producer Configs 3.4 Consumer Configs 3.5 Kafka C

[jira] [Updated] (KAFKA-13679) Superfluous node disconnected log messages

2022-02-21 Thread Philip Bourke (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13679?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Philip Bourke updated KAFKA-13679: -- Summary: Superfluous node disconnected log messages (was: Superfluous node disconnected logs)

[jira] [Created] (KAFKA-13679) Superfluous node disconnected logs

2022-02-21 Thread Philip Bourke (Jira)
Philip Bourke created KAFKA-13679: - Summary: Superfluous node disconnected logs Key: KAFKA-13679 URL: https://issues.apache.org/jira/browse/KAFKA-13679 Project: Kafka Issue Type: Bug

[GitHub] [kafka] cadonna commented on a change in pull request #11760: KAFKA-13600: Kafka Streams - Fall back to most caught up client if no caught up clients exist

2022-02-21 Thread GitBox
cadonna commented on a change in pull request #11760: URL: https://github.com/apache/kafka/pull/11760#discussion_r811402366 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/TaskMovement.java ## @@ -56,25 +54,34 @@ private int numCaugh

[GitHub] [kafka] cadonna commented on a change in pull request #11760: KAFKA-13600: Kafka Streams - Fall back to most caught up client if no caught up clients exist

2022-02-21 Thread GitBox
cadonna commented on a change in pull request #11760: URL: https://github.com/apache/kafka/pull/11760#discussion_r811394037 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/TaskMovement.java ## @@ -102,17 +109,26 @@ static int assignA

[GitHub] [kafka] tim-patterson commented on a change in pull request #11760: KAFKA-13600: Kafka Streams - Fall back to most caught up client if no caught up clients exist

2022-02-21 Thread GitBox
tim-patterson commented on a change in pull request #11760: URL: https://github.com/apache/kafka/pull/11760#discussion_r811393001 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/TaskMovement.java ## @@ -102,17 +109,26 @@ static int a

[GitHub] [kafka] mimaison commented on a change in pull request #11575: KAFKA-13511: Add support for different unix precisions in TimestampConverter SMT

2022-02-21 Thread GitBox
mimaison commented on a change in pull request #11575: URL: https://github.com/apache/kafka/pull/11575#discussion_r811318744 ## File path: connect/transforms/src/main/java/org/apache/kafka/connect/transforms/TimestampConverter.java ## @@ -83,12 +78,36 @@ private static fi

[jira] [Updated] (KAFKA-13673) disable idempotent producer when acks/retries/max.in.flight config conflicts

2022-02-21 Thread Mickael Maison (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13673?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mickael Maison updated KAFKA-13673: --- Fix Version/s: 3.0.1 > disable idempotent producer when acks/retries/max.in.flight config co

[GitHub] [kafka] mimaison commented on a change in pull request #11788: KAFKA-13673: disable idempotence when config conflicts

2022-02-21 Thread GitBox
mimaison commented on a change in pull request #11788: URL: https://github.com/apache/kafka/pull/11788#discussion_r811310081 ## File path: docs/upgrade.html ## @@ -73,7 +74,8 @@ Notable changes in 3 A bug prevented the producer idempotence default from being applied whi

[jira] [Created] (KAFKA-13678) 2nd punctuation using STREAM_TIME does not respect scheduled interval

2022-02-21 Thread Lorenzo Cagnatel (Jira)
Lorenzo Cagnatel created KAFKA-13678: Summary: 2nd punctuation using STREAM_TIME does not respect scheduled interval Key: KAFKA-13678 URL: https://issues.apache.org/jira/browse/KAFKA-13678 Project

[GitHub] [kafka] cadonna commented on a change in pull request #11760: KAFKA-13600: Kafka Streams - Fall back to most caught up client if no caught up clients exist

2022-02-21 Thread GitBox
cadonna commented on a change in pull request #11760: URL: https://github.com/apache/kafka/pull/11760#discussion_r811184710 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/TaskMovement.java ## @@ -102,17 +109,26 @@ static int assignA

[GitHub] [kafka] rhauch merged pull request #11758: MINOR: Clarify logging behavior with errors.log.include.messages property

2022-02-21 Thread GitBox
rhauch merged pull request #11758: URL: https://github.com/apache/kafka/pull/11758 -- 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: jira-unsubscr..

[jira] [Commented] (KAFKA-13675) Null pointer exception with kafka streams application reset tool with --to-datetime

2022-02-21 Thread Bruno Cadonna (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-13675?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17495485#comment-17495485 ] Bruno Cadonna commented on KAFKA-13675: --- [~pcallahan] thank you for the report, T

[GitHub] [kafka] mimaison commented on pull request #11642: MINOR: Improve Connect docs

2022-02-21 Thread GitBox
mimaison commented on pull request #11642: URL: https://github.com/apache/kafka/pull/11642#issuecomment-1046772930 @showuon @tombentley Can you take another look? Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [kafka] mimaison commented on a change in pull request #11794: MINOR: Add links to connector configs in TOC

2022-02-21 Thread GitBox
mimaison commented on a change in pull request #11794: URL: https://github.com/apache/kafka/pull/11794#discussion_r811030427 ## File path: docs/toc.html ## @@ -45,6 +45,10 @@ 3.3 Producer Configs 3.4 Consumer Configs 3.5 Kafka

[GitHub] [kafka] lkokhreidze commented on pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
lkokhreidze commented on pull request #10851: URL: https://github.com/apache/kafka/pull/10851#issuecomment-1046758399 Thanks @showuon for the valuable feedback. I've addressed your comments, please have a look when you got time. -- This is an automated message from the Apache Git Servi

[GitHub] [kafka] tim-patterson commented on a change in pull request #11493: KAFKA-12959: Prioritise assigning standby tasks to threads without any active tasks

2022-02-21 Thread GitBox
tim-patterson commented on a change in pull request #11493: URL: https://github.com/apache/kafka/pull/11493#discussion_r810985840 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamsPartitionAssignor.java ## @@ -1028,116 +1045,113 @@ private

[jira] [Updated] (KAFKA-13677) version 3.0.0 Processor implementation uses parameter record (restricted identifier)

2022-02-21 Thread Jira
[ https://issues.apache.org/jira/browse/KAFKA-13677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Aljoša Vrbanac updated KAFKA-13677: --- Affects Version/s: 2.7.0 > version 3.0.0 Processor implementation uses parameter record (res

[jira] [Created] (KAFKA-13677) version 3.0.0 Processor implementation uses parameter record (restricted identifier)

2022-02-21 Thread Jira
Aljoša Vrbanac created KAFKA-13677: -- Summary: version 3.0.0 Processor implementation uses parameter record (restricted identifier) Key: KAFKA-13677 URL: https://issues.apache.org/jira/browse/KAFKA-13677

[GitHub] [kafka] alozano3 commented on pull request #10803: KAFKA-12874; Increase default consumer session timeout to 45s

2022-02-21 Thread GitBox
alozano3 commented on pull request #10803: URL: https://github.com/apache/kafka/pull/10803#issuecomment-1046622683 ¿Shouldn't this be also applied to Connect consumer configurations ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Commented] (KAFKA-12874) Increase default consumer session timeout to 45s (KIP-735)

2022-02-21 Thread Albert Lozano (Jira)
[ https://issues.apache.org/jira/browse/KAFKA-12874?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17495399#comment-17495399 ] Albert Lozano commented on KAFKA-12874: --- ¿Shouldn't this be also applied to Connec

[GitHub] [kafka] lkokhreidze commented on a change in pull request #10851: KAFKA-6718 / Rack aware standby task assignor

2022-02-21 Thread GitBox
lkokhreidze commented on a change in pull request #10851: URL: https://github.com/apache/kafka/pull/10851#discussion_r810894883 ## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/assignment/ClientTagAwareStandbyTaskAssignor.java ## @@ -0,0 +1,295