Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-20 Thread via GitHub
hachikuji merged PR #14697: URL: https://github.com/apache/kafka/pull/14697 -- 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...@kafka.apach

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-20 Thread via GitHub
hachikuji commented on PR #14697: URL: https://github.com/apache/kafka/pull/14697#issuecomment-1819609057 @junrao I triggered a few builds and didn't see much consistency in the test failures. I tried some of the tests locally and they passed. I will go ahead and merge to trunk. -- This

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-14 Thread via GitHub
hachikuji commented on PR #14697: URL: https://github.com/apache/kafka/pull/14697#issuecomment-1811585213 @ex172000 @junrao Thanks for reviewing. Please take another look when you have time. -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-14 Thread via GitHub
hachikuji commented on code in PR #14697: URL: https://github.com/apache/kafka/pull/14697#discussion_r1393453106 ## core/src/main/scala/kafka/log/UnifiedLog.scala: ## @@ -818,9 +818,7 @@ class UnifiedLog(@volatile var logStartOffset: Long, appendInfo.setMaxTimestam

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-14 Thread via GitHub
hachikuji commented on code in PR #14697: URL: https://github.com/apache/kafka/pull/14697#discussion_r1393447344 ## core/src/test/scala/unit/kafka/log/LogValidatorTest.scala: ## @@ -399,12 +402,20 @@ class LogValidatorTest { assertEquals(i, offsetCounter.value); assert

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-03 Thread via GitHub
junrao commented on code in PR #14697: URL: https://github.com/apache/kafka/pull/14697#discussion_r1382110298 ## core/src/test/scala/unit/kafka/log/LogValidatorTest.scala: ## @@ -1641,7 +1659,7 @@ class LogValidatorTest { } } - def verifyRecordConversionStats(stats: R

Re: [PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-02 Thread via GitHub
ex172000 commented on code in PR #14697: URL: https://github.com/apache/kafka/pull/14697#discussion_r1381157089 ## core/src/test/scala/unit/kafka/log/LogValidatorTest.scala: ## @@ -399,12 +402,20 @@ class LogValidatorTest { assertEquals(i, offsetCounter.value); assertE

[PR] MINOR: Small LogValidator clean ups [kafka]

2023-11-02 Thread via GitHub
hachikuji opened a new pull request, #14697: URL: https://github.com/apache/kafka/pull/14697 1. Set shallowOffsetOfMaxTimestamp consistently as the last offset in the batch for v2 compressed and non-compressed data. 2. Rename `RecordConversionStats` to `RecordValidationStats` since one of