vvcephei commented on a change in pull request #11406:
URL: https://github.com/apache/kafka/pull/11406#discussion_r731306798



##########
File path: 
streams/src/main/java/org/apache/kafka/streams/state/internals/InMemoryKeyValueStore.java
##########
@@ -72,6 +74,11 @@ public boolean isOpen() {
         return open;
     }
 
+    @Override
+    public <R> QueryResult<R> query(final Query<R> query, final Map<String, 
Map<Integer, Long>> offsetBound) {
+        return StoreQueryUtils.requireKVQuery(query, this);

Review comment:
       When we do this think "for real", I'd like to create a general spec test 
so we can iterate over every possible permutation of store and config to make 
sure they all execute their queries correctly.




-- 
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


Reply via email to