Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-12 Thread via GitHub
chia7712 merged PR #15670: URL: https://github.com/apache/kafka/pull/15670 -- 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:

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
chia7712 commented on PR #15670: URL: https://github.com/apache/kafka/pull/15670#issuecomment-2049994583 `JDK 8 and Scala 2.12` is stopped exceptionally. could you please trigger QA again? -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560841706 ## build.gradle: ## @@ -1828,6 +1828,7 @@ project(':storage') { implementation project(':storage:storage-api') implementation project(':server-common')

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560836816 ## build.gradle: ## @@ -1828,6 +1828,7 @@ project(':storage') { implementation project(':storage:storage-api') implementation project(':server-common')

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
OmniaGM commented on PR #15670: URL: https://github.com/apache/kafka/pull/15670#issuecomment-2049439006 > @OmniaGM sorry that I just merge a PR and it causes conflicts to this PR. Could you please fix them? just rebased -- This is an automated message from the Apache Git Service.

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560820501 ## storage/src/main/java/org/apache/kafka/storage/internals/log/ProducerStateManagerConfig.java: ## @@ -17,13 +17,12 @@ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560820501 ## storage/src/main/java/org/apache/kafka/storage/internals/log/ProducerStateManagerConfig.java: ## @@ -17,13 +17,12 @@ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
chia7712 commented on PR #15670: URL: https://github.com/apache/kafka/pull/15670#issuecomment-2049411941 @OmniaGM sorry that I just merge a PR and it causes conflicts to this PR. Could you please fix them? -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560809899 ## core/src/main/scala/kafka/server/KafkaConfig.scala: ## @@ -1062,21 +1027,21 @@ object KafkaConfig { .define(CompressionTypeProp, STRING,

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-11 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1560802777 ## core/src/main/scala/kafka/server/KafkaConfig.scala: ## @@ -1062,21 +1027,21 @@ object KafkaConfig { .define(CompressionTypeProp, STRING,

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-10 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1559526460 ## storage/src/main/java/org/apache/kafka/storage/internals/log/ProducerStateManagerConfig.java: ## @@ -17,13 +17,12 @@ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556505842 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556496340 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556491908 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556482611 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556478763 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556481647 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556478763 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556466418 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1556404908 ## transaction-coordinator/src/main/java/org/apache/kafka/coordinator/transaction/TransactionLogConfig.java: ## @@ -16,11 +16,43 @@ */ package

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-08 Thread via GitHub
OmniaGM commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1555661766 ## core/src/main/scala/kafka/server/KafkaConfig.scala: ## @@ -1062,21 +1027,21 @@ object KafkaConfig { .define(CompressionTypeProp, STRING,

Re: [PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-05 Thread via GitHub
chia7712 commented on code in PR #15670: URL: https://github.com/apache/kafka/pull/15670#discussion_r1554282432 ## core/src/main/scala/kafka/server/DynamicBrokerConfig.scala: ## @@ -1136,18 +1137,18 @@ class DynamicListenerConfig(server: KafkaBroker) extends

[PR] KAFKA-15853: Move transactions configs out of core [kafka]

2024-04-05 Thread via GitHub
OmniaGM opened a new pull request, #15670: URL: https://github.com/apache/kafka/pull/15670 ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including