Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2024-04-10 Thread via GitHub
jolshan commented on code in PR #14607: URL: https://github.com/apache/kafka/pull/14607#discussion_r1560454922 ## storage/src/main/java/org/apache/kafka/storage/internals/checkpoint/PartitionMetadataFile.java: ## @@ -0,0 +1,141 @@ +/* + * Licensed to the Apache Software

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-11-01 Thread via GitHub
junrao merged PR #14607: URL: https://github.com/apache/kafka/pull/14607 -- 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-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-11-01 Thread via GitHub
junrao commented on PR #14607: URL: https://github.com/apache/kafka/pull/14607#issuecomment-1789288736 Thanks for triaging the transient test failures, @alok123t. Will merge the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-11-01 Thread via GitHub
alok123t commented on PR #14607: URL: https://github.com/apache/kafka/pull/14607#issuecomment-1788463950 @ijuma @junrao the latest build succeeds for JDK 8/Scala 2.12. The previous failures should be unrelated to this PR

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-11-01 Thread via GitHub
alok123t commented on PR #14607: URL: https://github.com/apache/kafka/pull/14607#issuecomment-1788460578 @junrao created following JIRA https://issues.apache.org/jira/browse/KAFKA-15770 For rest of the failed tests, JIRA already exists -- This is an automated message from the Apache

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-28 Thread via GitHub
ijuma commented on PR #14607: URL: https://github.com/apache/kafka/pull/14607#issuecomment-1783833262 I think this doesn't compile with Scala 2.12 - that needs to be fixed before we can proceed. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-26 Thread via GitHub
alok123t commented on PR #14607: URL: https://github.com/apache/kafka/pull/14607#issuecomment-1781801313 @junrao I think the test failures should be unrelated to the PR from a quick look - not sure if these are known flaky tests, I will wait for another run from the latest commit --

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-26 Thread via GitHub
alok123t commented on code in PR #14607: URL: https://github.com/apache/kafka/pull/14607#discussion_r1373738648 ## storage/src/main/java/org/apache/kafka/storage/internals/checkpoint/PartitionMetadata.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-26 Thread via GitHub
junrao commented on code in PR #14607: URL: https://github.com/apache/kafka/pull/14607#discussion_r1373572075 ## storage/src/main/java/org/apache/kafka/storage/internals/checkpoint/PartitionMetadata.java: ## @@ -0,0 +1,43 @@ +/* + * Licensed to the Apache Software Foundation

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-26 Thread via GitHub
alok123t commented on code in PR #14607: URL: https://github.com/apache/kafka/pull/14607#discussion_r1372811735 ## storage/src/main/java/org/apache/kafka/storage/internals/log/PartitionMetadataReadBuffer.java: ## @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache Software

Re: [PR] KAFKA-14484: [1/N] Move PartitionMetadataFile to storage module [kafka]

2023-10-25 Thread via GitHub
junrao commented on code in PR #14607: URL: https://github.com/apache/kafka/pull/14607#discussion_r1372393891 ## storage/src/main/java/org/apache/kafka/storage/internals/log/PartitionMetadataReadBuffer.java: ## @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache Software Foundation