hanyuzheng7 commented on code in PR #14523:
URL: https://github.com/apache/kafka/pull/14523#discussion_r1356041608
##########
streams/src/test/java/org/apache/kafka/streams/integration/IQv2StoreIntegrationTest.java:
##########
@@ -880,25 +888,113 @@ private <T> void shouldHandleWindowKeyQueries(final
Function<T, Integer> extract
extractor,
mkSet()
);
+
+ shouldHandleWindowKeyQuery(
Review Comment:
While adding tests, I observed an intriguing scenario. If we have three
windows: 0-5, 5-10, and 10-15, any effective query must encompass the window's
start time. For instance, with a key-value pair at time 6, the query range must
include 5. We could have ranges like query(5, 6) or query(5,7). If the query's
lower bound exceeds 5, we won't retrieve the value for that key-value pair.
When selecting a query range, I ensure the lower bound is less than or equal
to the window's start time. For instance, if I want to retrieve the data at
timestamp 6 (represented as <1,1>), I would set the lower bound to a value less
than or equal to 5. The upper bound, on the other hand, is selected randomly
from values greater than or equal to 5.
--
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
