Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
chia7712 commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1518695740 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,19 +352,15 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
ijuma commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1518681734 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,19 +352,15 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
chia7712 commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1518649558 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,19 +352,15 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
ijuma commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1518648733 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,19 +352,15 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
chia7712 merged PR #15488: URL: https://github.com/apache/kafka/pull/15488 -- 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: remove the copy constructor of LogSegment [kafka]

2024-03-09 Thread via GitHub
chia7712 commented on PR #15488: URL: https://github.com/apache/kafka/pull/15488#issuecomment-1986950699 the failed tests pass on my local. ```sh ./gradlew cleanTest core:test --tests QuorumControllerTest --tests ReplicaManagerTest ``` -- This is an automated message from the

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-08 Thread via GitHub
johnnychhsu commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1517940750 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,7 +353,10 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-08 Thread via GitHub
johnnychhsu commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1517906750 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,7 +353,10 @@ class LogLoaderTest { // Intercept all segment read calls val

Re: [PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-07 Thread via GitHub
chia7712 commented on code in PR #15488: URL: https://github.com/apache/kafka/pull/15488#discussion_r1516370592 ## core/src/test/scala/unit/kafka/log/LogLoaderTest.scala: ## @@ -352,7 +353,10 @@ class LogLoaderTest { // Intercept all segment read calls val

[PR] MINOR: remove the copy constructor of LogSegment [kafka]

2024-03-07 Thread via GitHub
johnnychhsu opened a new pull request, #15488: URL: https://github.com/apache/kafka/pull/15488 ## Context In the `LogSegmnet`, the copy constructor is only used in LogLoaderTest ## Solution Remove the copy constructor and build the logic in test ## Test Run `./gradlew