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

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


Reply via email to