showuon commented on a change in pull request #11337:
URL: https://github.com/apache/kafka/pull/11337#discussion_r711430770
##########
File path:
streams/src/test/java/org/apache/kafka/streams/state/internals/AbstractSessionBytesStoreTest.java
##########
@@ -121,15 +121,15 @@ public void shouldPutAndFindSessionsInRange() {
try (final KeyValueIterator<Windowed<String>, Long> values =
sessionStore.findSessions(key, 0, 1000L)
) {
- assertEquals(new HashSet<>(expected), toSet(values));
+ assertEquals(expected, toList(values));
Review comment:
before this PR, we only verify the returned data `contains` in the
expected data as **Set**. Now, we'll change to **List**, which means, not only
the data should be correct, but also the data order should be correct.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
