hanyuzheng7 commented on code in PR #14570: URL: https://github.com/apache/kafka/pull/14570#discussion_r1378447978
########## streams/src/main/java/org/apache/kafka/streams/state/internals/MeteredTimestampedKeyValueStore.java: ########## @@ -102,4 +140,230 @@ static class RawAndDeserializedValue<ValueType> { this.value = value; } } + + @SuppressWarnings("unchecked") + @Override + public <R> QueryResult<R> query(final Query<R> query, + final PositionBound positionBound, + final QueryConfig config) { + + final long start = time.nanoseconds(); + final QueryResult<R> result; + + final StoreQueryUtils.QueryHandler handler = queryHandlers.get(query.getClass()); + if (handler == null) { + result = wrapped().query(query, positionBound, config); + if (config.isCollectExecutionInfo()) { + result.addExecutionInfo( + "Handled in " + getClass() + " in " + (time.nanoseconds() - start) + "ns"); + } + } else { + result = (QueryResult<R>) handler.apply( + query, + positionBound, + config, + this + ); + if (config.isCollectExecutionInfo()) { + result.addExecutionInfo( + "Handled in " + getClass() + " with serdes " + + serdes + " in " + (time.nanoseconds() - start) + "ns"); + } + } + return result; + } + + + + @SuppressWarnings("unchecked") + protected <R> QueryResult<R> runTimestampKeyQuery(final Query<R> query, + final PositionBound positionBound, + final QueryConfig config) { + final QueryResult<R> result; + final TimestampedKeyQuery<K, V> typedKeyQuery = (TimestampedKeyQuery<K, V>) query; + final KeyQuery<Bytes, byte[]> rawKeyQuery = + KeyQuery.withKey(keyBytes(typedKeyQuery.getKey())); + final QueryResult<byte[]> rawResult = + wrapped().query(rawKeyQuery, positionBound, config); + if (rawResult.isSuccess()) { + final Function<byte[], ValueAndTimestamp<V>> deserializer = getDeserializeValue(serdes, wrapped()); + final ValueAndTimestamp<V> value = deserializer.apply(rawResult.getResult()); + final QueryResult<ValueAndTimestamp<V>> typedQueryResult = + InternalQueryResultUtil.copyAndSubstituteDeserializedResult(rawResult, value); + result = (QueryResult<R>) typedQueryResult; + } else { + // the generic type doesn't matter, since failed queries have no result set. + result = (QueryResult<R>) rawResult; + } + return result; + } + + @SuppressWarnings("unchecked") + protected <R> QueryResult<R> runTimestampRangeQuery(final Query<R> query, + final PositionBound positionBound, + final QueryConfig config) { + + final QueryResult<R> result; + final TimestampedRangeQuery<K, V> typedQuery = (TimestampedRangeQuery<K, V>) query; + + final RangeQuery<Bytes, byte[]> rawRangeQuery; + if (typedQuery.getLowerBound().isPresent() && typedQuery.getUpperBound().isPresent()) { + rawRangeQuery = RangeQuery.withRange( + keyBytes(typedQuery.getLowerBound().get()), + keyBytes(typedQuery.getUpperBound().get()) + ); + } else if (typedQuery.getLowerBound().isPresent()) { + rawRangeQuery = RangeQuery.withLowerBound(keyBytes(typedQuery.getLowerBound().get())); + } else if (typedQuery.getUpperBound().isPresent()) { + rawRangeQuery = RangeQuery.withUpperBound(keyBytes(typedQuery.getUpperBound().get())); + } else { + rawRangeQuery = RangeQuery.withNoBounds(); + } + final QueryResult<KeyValueIterator<Bytes, byte[]>> rawResult = + wrapped().query(rawRangeQuery, positionBound, config); + if (rawResult.isSuccess()) { + final KeyValueIterator<Bytes, byte[]> iterator = rawResult.getResult(); + final KeyValueIterator<K, V> resultIterator = new MeteredKeyValueTimestampedIterator( + iterator, + getSensor, + getDeserializeValue(serdes, wrapped()), + false + ); + final QueryResult<KeyValueIterator<K, V>> typedQueryResult = + InternalQueryResultUtil.copyAndSubstituteDeserializedResult( + rawResult, + resultIterator + ); + result = (QueryResult<R>) typedQueryResult; + } else { + // the generic type doesn't matter, since failed queries have no result set. + result = (QueryResult<R>) rawResult; + } + return result; + } + + @SuppressWarnings("unchecked") + protected <R> QueryResult<R> runKeyQuery2(final Query<R> query, Review Comment: solved -- 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