vpapavas commented on a change in pull request #11682:
URL: https://github.com/apache/kafka/pull/11682#discussion_r791953273



##########
File path: 
streams/src/main/java/org/apache/kafka/streams/state/internals/ChangeLoggingKeyValueBytesStore.java
##########
@@ -145,4 +145,9 @@ public void putAll(final List<KeyValue<Bytes, byte[]>> 
entries) {
     void log(final Bytes key, final byte[] value) {
         context.logChange(name(), key, value, context.timestamp(), position);
     }
+
+    @Override
+    public Position getPosition() {
+        return position;

Review comment:
       The changelogging stores don't have a position anymore since I added the 
public `getPosition`. The merging now is necessary only in the caching layer




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