Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-12 Thread via GitHub
showuon merged PR #15522: URL: https://github.com/apache/kafka/pull/15522 -- 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] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-12 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2051055209 Failed tests are unrelated. -- 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

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-11 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2049237739 @soarez , there is failed test... :(

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-10 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2047096126 @soarez , oops, there's conflict due to I've just merged another PR. Please help resolve it. Thanks. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-10 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2047072476 Sorry, forgot about this PR. The jdk8 job failed to complete due to infra's issue. Re-triggering now: https://ci-builds.apache.org/job/Kafka/job/kafka-pr/job/PR-15522/6/ -- This is an

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-10 Thread via GitHub
soarez commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2046961029 Failing tests are all unrelated and tracked: * kafka.api.DelegationTokenEndToEndAuthorizationWithOwnerTest."testNoConsumeWithDescribeAclViaAssign(String).quorum=kraft"

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-02 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2033354648 ``` /home/jenkins/workspace/Kafka_kafka-pr_PR-15522/server/src/test/java/org/apache/kafka/server/AssignmentsManagerTest.java:354: error: method onAssignment in class

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-02 Thread via GitHub
soarez commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1548655446 ## core/src/test/scala/unit/kafka/server/ReplicaAlterLogDirsThreadTest.scala: ## @@ -458,7 +458,7 @@ class ReplicaAlterLogDirsThreadTest {

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-01 Thread via GitHub
showuon commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1547005718 ## core/src/test/scala/unit/kafka/server/ReplicaAlterLogDirsThreadTest.scala: ## @@ -458,7 +458,7 @@ class ReplicaAlterLogDirsThreadTest {

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-01 Thread via GitHub
soarez commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1546358504 ## core/src/main/scala/kafka/server/ReplicaAlterLogDirsThread.scala: ## @@ -105,7 +105,7 @@ class ReplicaAlterLogDirsThread(name: String, topicId <-

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-01 Thread via GitHub
soarez commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1546352789 ## core/src/main/scala/kafka/server/ReplicaManager.scala: ## @@ -624,6 +624,13 @@ class ReplicaManager(val config: KafkaConfig, errorMap } + def

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-01 Thread via GitHub
soarez commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1546351970 ## core/src/main/scala/kafka/server/ReplicaAlterLogDirsThread.scala: ## @@ -105,7 +105,7 @@ class ReplicaAlterLogDirsThread(name: String, topicId <-

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-04-01 Thread via GitHub
showuon commented on code in PR #15522: URL: https://github.com/apache/kafka/pull/15522#discussion_r1546066080 ## core/src/main/scala/kafka/server/ReplicaManager.scala: ## @@ -624,6 +624,13 @@ class ReplicaManager(val config: KafkaConfig, errorMap } + def

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-03-27 Thread via GitHub
showuon commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-2022115723 Will check it this week or next. -- 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

Re: [PR] MINOR: Improve logging in AssignmentsManager [kafka]

2024-03-12 Thread via GitHub
soarez commented on PR #15522: URL: https://github.com/apache/kafka/pull/15522#issuecomment-1991528082 @showuon PTAL -- 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