vvcephei commented on a change in pull request #11513: URL: https://github.com/apache/kafka/pull/11513#discussion_r764279791
########## File path: streams/src/main/java/org/apache/kafka/streams/processor/internals/ProcessorContextImpl.java ########## @@ -110,17 +121,27 @@ public RecordCollector recordCollector() { public void logChange(final String storeName, final Bytes key, final byte[] value, - final long timestamp) { + final long timestamp, + final Position position) { throwUnsupportedOperationExceptionIfStandby("logChange"); final TopicPartition changelogPartition = stateManager().registeredChangelogPartitionFor(storeName); - // Sending null headers to changelog topics (KIP-244) + Headers headers = new RecordHeaders(); Review comment: nit: I'd make this final with no assignment, then assign it in both branches below. -- 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