[GitHub] [kafka] divijvaidya commented on a diff in pull request #14412: KAFKA-14133: Move MeteredTimestampedKeyValueStoreTest, ReadOnlyWindowStoreFacadeTest and TimestampedWindowStoreBuilderTest to M

2023-09-22 Thread via GitHub
divijvaidya commented on code in PR #14412: URL: https://github.com/apache/kafka/pull/14412#discussion_r1334106327 ## streams/src/test/java/org/apache/kafka/streams/state/internals/TimestampedWindowStoreBuilderTest.java: ## @@ -223,38 +208,9 @@ public void shouldThrowNullPointer

[GitHub] [kafka] divijvaidya commented on a diff in pull request #14412: KAFKA-14133: Move MeteredTimestampedKeyValueStoreTest, ReadOnlyWindowStoreFacadeTest and TimestampedWindowStoreBuilderTest to M

2023-09-22 Thread via GitHub
divijvaidya commented on code in PR #14412: URL: https://github.com/apache/kafka/pull/14412#discussion_r1334103035 ## streams/src/test/java/org/apache/kafka/streams/state/internals/TimestampedWindowStoreBuilderTest.java: ## @@ -223,38 +208,9 @@ public void shouldThrowNullPointer

[GitHub] [kafka] divijvaidya commented on a diff in pull request #14412: KAFKA-14133: Move MeteredTimestampedKeyValueStoreTest, ReadOnlyWindowStoreFacadeTest and TimestampedWindowStoreBuilderTest to M

2023-09-21 Thread via GitHub
divijvaidya commented on code in PR #14412: URL: https://github.com/apache/kafka/pull/14412#discussion_r1332776819 ## streams/src/test/java/org/apache/kafka/streams/state/internals/TimestampedWindowStoreBuilderTest.java: ## @@ -223,38 +208,9 @@ public void shouldThrowNullPointer