[GitHub] [kafka] vvcephei commented on a change in pull request #11336: DRAFT: Add currentPosition and times to ProcessorContext

[GitBox <gi...@apache.org>]

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



##########
File path: streams/src/main/java/org/apache/kafka/streams/processor/api/ProcessorContext.java
##########
@@ -247,4 +251,45 @@ Cancellable schedule(final Duration interval,
      * @return the key/values matching the given prefix from the StreamsConfig properties.
      */
     Map<String, Object> appConfigsWithPrefix(final String prefix);
+
+    /**
+     * Return the current system timestamp (also called wall-clock time) in milliseconds.
+     *
+     * <p> Note: this method returns the internally cached system timestamp from the Kafka Stream runtime.
+     * Thus, it may return a different value compared to {@code System.currentTimeMillis()}.
+     *
+     * @return the current system timestamp in milliseconds
+     */
+    long currentSystemTimeMs();

Review comment:
       Yeah, I think it was just a miss. That KIP might have been going in parallel with the new PAPI, I don't remember.
   
   I agree adding these would just be an update to KIP-622 instead of a part of this KIP. I'll just go ahead and send a separate PR for this and a note to the KIP-622 vote thread.




-- 
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-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org