mjsax commented on code in PR #21580:
URL: https://github.com/apache/kafka/pull/21580#discussion_r2887467634
##########
streams/src/main/java/org/apache/kafka/streams/state/internals/KeyValueStoreWrapper.java:
##########
@@ -45,70 +48,70 @@ public class KeyValueStoreWrapper<K, V> implements
StateStore {
public static final long PUT_RETURN_CODE_IS_LATEST
= VersionedKeyValueStore.PUT_RETURN_CODE_VALID_TO_UNDEFINED;
- private TimestampedKeyValueStore<K, V> timestampedStore = null;
+ private TimestampedKeyValueStoreWithHeaders<K, V> headersStore = null;
private VersionedKeyValueStore<K, V> versionedStore = null;
- // same as either timestampedStore or versionedStore above. kept merely as
a convenience
- // to simplify implementation for methods which do not depend on store
type.
private StateStore store;
public KeyValueStoreWrapper(final ProcessorContext<?, ?> context, final
String storeName) {
+ // Try headers-aware store first, then versioned store
try {
- // first try timestamped store
- timestampedStore = context.getStateStore(storeName);
- store = timestampedStore;
+ // first try headers-aware timestamped store
+ headersStore = context.getStateStore(storeName);
+ store = headersStore;
return;
} catch (final ClassCastException e) {
- // ignore since could be versioned store instead
+ // ignore since could be regular timestamped or versioned store
instead
+ System.out.println(e.getMessage());
Review Comment:
Guess it's on left over debug statement that needs to get removed?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
